Date: Tue, 1 May 2007 23:24:48 -0700 From: Brian Somers <brians@ca.sophos.com> To: Brian Somers <brians@ca.sophos.com> Cc: gnome@FreeBSD.org Subject: Re: Fix for x11/gnome-screensaver Message-ID: <20070501232448.1239bbd3@dev.lan.Awfulhak.org> In-Reply-To: <20070501123616.6b43b4de@conflict.ca.sophos.com>
index | next in thread | previous in thread | raw e-mail
On Tue, 1 May 2007 12:36:16 -0700 Brian Somers <brians@ca.sophos.com> wrote:
> Hi,
>
> Could someone apply the attached patch for x11/gnome-screensaver
> or approve it for me to commit?
>
> TIA.
>
> --
> Brian Somers Tel: +1 604 484 6434
> Mob: +1 604 315 1343
> Sophos - security and control Web: www.sophos.com
>
> Index: Makefile
> ===================================================================
> RCS file: /usr/home/ncvs/ports/x11/gnome-screensaver/Makefile,v
> retrieving revision 1.23
> diff -u -r1.23 Makefile
> --- Makefile 11 Apr 2007 13:32:59 -0000 1.23
> +++ Makefile 1 May 2007 19:34:17 -0000
> @@ -47,7 +47,7 @@
> CONFIGURE_ARGS+= --disable-pam
> .endif
>
> -pre-install:
> +pre-su-install:
> @${CP} ${.CURDIR}/pkg-plist ${PLIST}
> @cd ${XSCREENSAVER_DIR} && ${FIND} -s * -maxdepth 1 -type f -name "*.xml" | \
> ${GREP} -Ev ${XSCREENSAVER_EXCLUDE} | \
> @@ -55,7 +55,7 @@
> ${SED} -e 's:[.]xml$$:.desktop:' >> ${PLIST}
> @${ECHO_CMD} @dirrm ${SCREENSAVER_DIR} >> ${PLIST}
>
> -post-install:
> +post-su-install:
> ${CHOWN} root ${PREFIX}/libexec/gnome-screensaver-dialog
> ${CHMOD} +s ${PREFIX}/libexec/gnome-screensaver-dialog
> ${MKDIR} ${PREFIX}/${SCREENSAVER_DIR}
Hmm, it seems that post-su-install doesn't exist and
post-install is run with root credentials, so please
disregard the second part of the patch.
The pre-install -> pre-su-install change is still
required though...
Cheers.
--
Brian Somers Tel: +1 604 484 6434
Mob: +1 604 315 1343
Sophos - security and control Web: www.sophos.com
home |
help
Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20070501232448.1239bbd3>