Gentoo Logo
Gentoo Spaceship




Note: Due to technical difficulties, the Archives are currently not up to date. GMANE provides an alternative service for most mailing lists.
c.f. bug 424647
List Archive: gentoo-alt
Navigation:
Lists: gentoo-alt: < Prev By Thread Next > < Prev By Date Next >
Headers:
To: gentoo-alt@g.o
From: Burcin Erocal <burcin@...>
Subject: Re: permission test
Date: Tue, 18 Oct 2011 20:34:12 +0200
Hi Fabian,

Thanks for the quick response.

On Tue, 18 Oct 2011 20:13:14 +0200
Fabian Groffen <grobian@g.o> wrote:

> On 18-10-2011 20:10:24 +0200, Burcin Erocal wrote:
> > Attached trivial patch puts this check in an if statement depending
> > on an environment variable QA_SKIP_WRITABLE. Setting this in
> > make.conf makes prefix usable again.
> 
> > diff --git a/bin/misc-functions.sh b/bin/misc-functions.sh
> > --- a/bin/misc-functions.sh
> > +++ b/bin/misc-functions.sh
> > @@ -168,6 +168,7 @@
> >  	fi
> >  
> >  	# Now we look for all world writable files.
> > +	if [ "${QA_SKIP_WRITABLE-unset}" == unset ] ; then
> >  	local i
> >  	for i in $(find "${D}/" -type f -perm -2); do
> 
> How would this work, if you changed the D into ED here?  Checking
> files outside of our control is indeed not really useful.

In that context, printing $D gives $PORTAGE_TMP/$CATEGORY/$P/image in
the prefix. Since these are the new files introduced by the ebuild, I
don't think we need to change that line. Note that this is already in
the portage sources and I didn't touch it. :)

Cheers,
Burcin


Replies:
Re: permission test
-- Fabian Groffen
References:
permission test
-- Burcin Erocal
Re: permission test
-- Fabian Groffen
Navigation:
Lists: gentoo-alt: < Prev By Thread Next > < Prev By Date Next >
Previous by thread:
Re: permission test
Next by thread:
Re: permission test
Previous by date:
Re: permission test
Next by date:
Re: permission test


Updated Apr 03, 2012

Summary: Archive of the gentoo-alt mailing list.

Donate to support our development efforts.

Copyright 2001-2013 Gentoo Foundation, Inc. Questions, Comments? Contact us.