From: Tomas Touceda <chiiph@gmail.com>
To: "gentoo-dev@lists.gentoo.org" <gentoo-dev@lists.gentoo.org>
Subject: Re: [gentoo-dev] Calling unknown commands in an ebuild
Date: Sun, 7 Feb 2010 21:45:28 -0400 [thread overview]
Message-ID: <99DED219-E3E1-459D-AB54-560CF60084E5@gmail.com> (raw)
In-Reply-To: <4B6F3C7F.700@gentoo.org>
El 07/02/2010, a las 18:19, Zac Medico <zmedico@gentoo.org> escribió:
> On 02/07/2010 01:10 PM, Stelian Ionescu wrote:
>> Wouldn't it be a good idea to use "set -e" in the ebuild
>> environment ?
>> I've seen cases of ebuilds calling epatch without inheriting from
>> eutils
>> which compiled and installed (apparently) fine but possibly broken
>> binaries. Examples of cases where "set -e" would have helped: 303849,
>> 297063, 260279, 221257,
>> https://bugs.gentoo.org/buglist.cgi?quicksearch=command+not+found
>> and perhaps others I haven't managed to find in bugzilla
>
> I don't know what kind of side-effects set -e would introduce, but
> we can easily add a repoman check for epatch calls without eutils
> inherit.
>
> Portage already does a search of the build log for 'command not
> found' messages and generates a QA warnings. Set
> PORTAGE_ELOG_CLASSES="${PORTAGE_ELOG_CLASSES} qa" in /etc/make.conf
> if you want to have those warnings logged.
But, shouldn't it die when a command isn't found? Not only with epatch.
> --
> Thanks,
> Zac
>
next prev parent reply other threads:[~2010-02-08 0:47 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-07 21:10 [gentoo-dev] Calling unknown commands in an ebuild Stelian Ionescu
2010-02-07 22:19 ` Zac Medico
2010-02-08 0:56 ` Stelian Ionescu
2010-02-08 1:45 ` Tomas Touceda [this message]
2010-02-08 2:24 ` Mike Frysinger
2010-02-08 7:48 ` Rémi Cardona
2010-02-08 10:18 ` Peter Volkov
2010-02-08 2:22 ` Mike Frysinger
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=99DED219-E3E1-459D-AB54-560CF60084E5@gmail.com \
--to=chiiph@gmail.com \
--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