From: Steve Long <slong@rathaus.eclipse.co.uk>
To: gentoo-dev@lists.gentoo.org
Subject: [gentoo-dev] Re: Re: [RFC] New keywords for non-Gentoo Linux platforms
Date: Wed, 15 Oct 2008 11:20:12 +0100 [thread overview]
Message-ID: <gd4gqt$fa7$1@ger.gmane.org> (raw)
In-Reply-To: 1223970507.18595.7.camel@sapc154.salomon.at
Michael Haubenwallner wrote:
> Fabian Groffen wrote:
>> Ciaran McCreesh wrote:
>> > Steve Long wrote:
>> > > Unless someone can say what using PROPERTIES=prefix would break, I'd
>> > > go with that. It's a /classic/ usage of that variable, as it's simply
>> > > a boolean; PROPERTIES is well-defined and I'm hoping all the manglers
>> > > support it. It'd be great to see the prefix branch finally merged so
>> > > we all get to play with it.
>> >
>> > It would break. Prefix ebuilds won't work unless ED is set, and a non
>> > PROPERTIES aware or non-prefix-property aware package manager won't set
>> > ED. Unless prefix is reimplemented to require no package manager
>> > changes for the install to / case, PROPERTIES is out.
Nonsense; we just need some bash changes to integrate it iff we want to
allow prefix ebuilds into the main tree; we're a while away from actually
being at the stage where that would be feasible, even if it were desired.
By the time we do get there, we should be able to fulfil your last
condition, given a sane bash implementation for the mangler in question.
TBH I'm not really worried about alternative manglers atm; they can catch up
w/e afaic. IIRC PROPERTIES are part of EAPI-2, so one would hope they've
already done that by now.
>>
>> Just to comment on this possibility; I see an option, given the
>> definition of ED and EROOT to do Prefix without them, by e.g. using
>> ${D}${EPREFIX} instead of ${ED} as shorthand. When ${EPREFIX} would be
>> unset, this would result in simple ${D}, which is "backwards
>> compatible". This is not all what is necessary, but a big deal of it.
>>
>> Question here, however, is whether this is worth it. Personally, I
>> prefer the shorthands, as they give a lot of readability.
>
Yeah the shorthands are nice, agreed. Ideally we wouldn't need to change
anything in the ebuild though. (I appreciate we're a while away from there,
but if we can at least get the ones which go via emake and econf, that's a
start. We can worry about {scons,distutils,..} later; pro-audio have a nice
exteutils.eclass we can 'borrow' for a start ;)
From the prefix docs, the only place D should be retained over ED is in a
DESTDIR="$D" (or "${D}") when configure --prefix=.. has been run. Is that
correct?
> Could it also work to initialize them in profile.bashrc if they are
> unset?
>
> Something like
> : ${EPREFIX=}
> : ${ED=${D}}
> : ${EROOT=${ROOT}}
>
That would work yeah, though I think it would be better if we just
integrated that into ebuild.sh. We can check PROPERTIES to see if prefix is
in there, and then enable additional logic, and eg use a different PATH for
external helpers (most of which we can do internally in any case.) I
realise the latter is happening already, but if we're integrating, it
should be happening in the mainline code when applicable, imo. (Not saying
that is the applicable case, just that we can change w/e you see fit.)
next prev parent reply other threads:[~2008-10-15 10:32 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-09 18:11 [gentoo-dev] [RFC] New keywords for non-Gentoo Linux platforms Fabian Groffen
2008-10-09 22:05 ` Marius Mauch
2008-10-10 0:16 ` [gentoo-dev] " Duncan
2008-10-10 2:21 ` Marius Mauch
2008-10-10 7:15 ` Fabian Groffen
2008-10-10 12:40 ` Diego 'Flameeyes' Pettenò
2008-10-10 12:48 ` Fabian Groffen
2008-10-10 15:56 ` Marius Mauch
2008-10-10 16:13 ` Robert Bridge
2008-10-10 15:21 ` Ryan Hill
2008-10-10 8:56 ` [gentoo-dev] " Michael Haubenwallner
2008-10-10 12:56 ` Jeremy Olexa
2008-10-13 5:16 ` [gentoo-dev] " Steve Long
2008-10-13 14:27 ` Ciaran McCreesh
2008-10-13 17:59 ` Fabian Groffen
2008-10-14 7:48 ` Michael Haubenwallner
2008-10-15 10:20 ` Steve Long [this message]
2008-10-17 14:22 ` [gentoo-dev] " Michael Haubenwallner
2008-10-21 14:09 ` [gentoo-dev] " Tiziano Müller
2008-10-21 19:01 ` Fabian Groffen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='gd4gqt$fa7$1@ger.gmane.org' \
--to=slong@rathaus.eclipse.co.uk \
--cc=gentoo-dev@lists.gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox