On 00:31 Wed 31 Oct     , Mike Frysinger wrote:
> On Monday 15 October 2007, Donnie Berkholz wrote:
> > On 18:33 Mon 15 Oct     , Micheal Marineau (marineam) wrote:
> > > 1.1                  app-emulation/xen/xen-3.1.1.ebuild
> > >
> > > file :
> > > http://sources.gentoo.org/viewcvs.py/gentoo-x86/app-emulation/xen/xen-3.1
> > >.1.ebuild?rev=1.1&view=markup plain:
> > > http://sources.gentoo.org/viewcvs.py/gentoo-x86/app-emulation/xen/xen-3.1
> > >.1.ebuild?rev=1.1&content-type=text/plain
> > >
> > > src_compile() {
> > >   local myopt
> > >   use debug && myopt="${myopt} debug=y"
> > >   use pae && myopt="${myopt} pae=y"
> > >
> > >   if use custom-cflags; then
> > >           filter-flags -fPIE -fstack-protector
> > >   else
> > >           unset CFLAGS
> > >   fi
> > >
> > >   # Send raw LDFLAGS so that --as-needed works
> > >   emake LDFLAGS="$(raw-ldflags)" -C xen ${myopt} || die "compile failed"
> > > }
> > >
> > > src_install() {
> > >   local myopt
> > >   use debug && myopt="${myopt} debug=y"
> > >   use pae && myopt="${myopt} pae=y"
> >
> > The environment should be preserved across phases, so you shouldn't have
> > to repeat this stuff.
> 
> "myopt" is declared "local"

Sure, but did you notice it's set to the exact thing in both phases? 
Seems a bit pointless.

Thanks,
Donnie
-- 
[EMAIL PROTECTED] mailing list

Reply via email to