Date: Mon, 15 Nov 2010 11:21:06 -0700 From: John Hein <jhein@symmetricom.com> To: Oliver Fromme <olli@lurza.secnetix.de> Cc: Anonymous <swell.k@gmail.com>, python@FreeBSD.org, bug-followup@FreeBSD.org Subject: Re: ports/152224: [patch] fix installed permissions for ?lang/python27 Message-ID: <19681.31250.541714.180410@gossamer.timing.com> In-Reply-To: <201011151735.oAFHZeY6014846@lurza.secnetix.de> References: <201011140410.oAE4ABUJ070464@freefall.freebsd.org> <201011151735.oAFHZeY6014846@lurza.secnetix.de>
index | next in thread | previous in thread | raw e-mail
[-- Attachment #1 --]
Oliver Fromme wrote at 18:35 +0100 on Nov 15, 2010:
> Anonymous <swell.k@gmail.com> wrote:
> > John Hein <jhein@symmetricom.com> writes:
> > > +post-extract:
> > > +# The distribution tarball for python 2.7 has permission bits for 'others'
> > > +# set to 0. Later during install, we copy Tools and Demo to the installed
> > > +# prefix, so set them right here.
> > > + ${FIND} ${WRKSRC}/Tools ${WRKSRC}/Demo -type d | ${XARGS} ${CHMOD} a+rx
> > > + ${FIND} ${WRKSRC}/Tools ${WRKSRC}/Demo -type f | ${XARGS} ${CHMOD} a+r
> > > +
> >
> > This can be reduced to one command
> >
> > ${FIND} ${WRKSRC}/Tools ${WRKSRC}/Demo \
> > -type d -exec ${CHMOD} a+rx {} + \
> > -or -type f -exec ${CHMOD} a+r {} +
>
> It's unclear to me why you have to use find(1) at all.
> The following simple command should work equally well:
>
> ${CHMOD} -R og=u-w ${WRKSRC}/Tools ${WRKSRC}/Demo
Yes, that's better still and will work fine since the user bits are
good in the tarball. Updated patch (with a fix for the path, too)...
[-- Attachment #2 --]
Index: Makefile
===================================================================
RCS file: /base/FreeBSD-CVS/ports/lang/python27/Makefile,v
retrieving revision 1.169
diff -u -p -r1.169 Makefile
--- Makefile 6 Sep 2010 00:25:04 -0000 1.169
+++ Makefile 15 Nov 2010 18:16:55 -0000
@@ -146,6 +146,12 @@ CONFIGURE_ARGS+= --disable-ipv6
CONFIGURE_ARGS+= --with-fpectl
.endif
+post-extract:
+# The distribution tarball for python 2.7 has permission bits for 'others'
+# set to 0. Later during install, we copy Tools and Demo to the installed
+# prefix, so set them right here.
+ ${CHMOD} og=u-w ${PYTHON_WRKSRC}/Tools ${PYTHON_WRKSRC}/Demo
+
pre-patch:
${CP} -r ${PATCH_WRKSRC}/Lib/plat-freebsd8 \
${PATCH_WRKSRC}/Lib/plat-freebsd9
@@ -260,12 +266,12 @@ post-install:
.if !defined(NOPORTDATA)
@${MKDIR} ${DATADIR}
@cd ${PYTHON_WRKSRC}; ${TAR} -cf - Tools | \
- (cd ${DATADIR}; ${TAR} -xf -)
+ (cd ${DATADIR}; ${TAR} --no-same-owner -xf -)
.endif
.if !defined(NOPORTEXAMPLES)
@${MKDIR} ${EXAMPLESDIR}
@cd ${PYTHON_WRKSRC}/Demo; ${TAR} -cf - * | \
- (cd ${EXAMPLESDIR}; ${TAR} -xf -)
+ (cd ${EXAMPLESDIR}; ${TAR} --no-same-owner -xf -)
.endif
@${CAT} ${PKGMESSAGE}
help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?19681.31250.541714.180410>