From: Fabian Groffen <grobian@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] RFC: adding support for running eautoconf to base.eclass
Date: Wed, 13 Feb 2008 10:19:37 +0100 [thread overview]
Message-ID: <20080213091937.GA6925@gentoo.org> (raw)
In-Reply-To: <47B2A13B.2090406@gentoo.org>
On 13-02-2008 08:50:19 +0100, Rémi Cardona wrote:
> Petteri Räty a écrit :
>> What do you think about adding support to base.eclass for running
>> eautoreconf?
> In most of the ebuilds where we need to run eautoreconf, we usually apply
> patches. I can't remember of an ebuild where we just run eautoreconf on its
> own.
+1
If you need to run eautoreconf without adding patches, it may be worth
adding a comment explaining why.
> In the end, that won't help us at all (which is no reason not to have this
> :) ) but I'm afraid that adding options for semi-hidden stuff can come and
> bite us later, as we add more and more of those.
I think it should not be added as it hides something quite important.
- it takes a lot of time on most platforms I run
- it may break (especially during bootstrapping, eautoreconfs are hell)
- it may introduce extra deps/caution (e.g. gettext macros being available)
So I'd prefer to keep it quite clear that this is happening, instead of
hiding it somewhere in an obscure corner of the ebuild.
--
Fabian Groffen
Gentoo on a different level
--
gentoo-dev@lists.gentoo.org mailing list
next prev parent reply other threads:[~2008-02-13 9:19 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-12 23:44 [gentoo-dev] RFC: adding support for running eautoconf to base.eclass Petteri Räty
2008-02-13 4:52 ` Paul de Vrieze
2008-02-13 6:22 ` Alec Warner
2008-02-13 12:02 ` Petteri Räty
2008-02-13 6:37 ` Ciaran McCreesh
2008-02-13 7:50 ` Rémi Cardona
2008-02-13 9:19 ` Fabian Groffen [this message]
2008-02-13 12:00 ` Petteri Räty
2008-02-13 12:22 ` Matthias Schwarzott
2008-02-13 12:35 ` Peter Volkov
2008-02-13 19:30 ` Petteri Räty
2008-02-13 22:46 ` Vlastimil Babka
2008-02-14 0:03 ` [gentoo-dev] " Ryan Hill
2008-02-14 17:39 ` [gentoo-dev] " Bo Ørsted Andresen
2008-02-13 12:01 ` Petteri Räty
2008-02-13 12:19 ` Samuli Suominen
2008-02-13 19:31 ` Petteri Räty
2008-02-13 12:54 ` Marius Mauch
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=20080213091937.GA6925@gentoo.org \
--to=grobian@gentoo.org \
--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