From: Thomas Sachau <tommy@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] Usage of econf with an additional || die
Date: Tue, 30 Sep 2008 18:47:58 +0200 [thread overview]
Message-ID: <48E2583E.5060404@gentoo.org> (raw)
In-Reply-To: <48E215AD.3070309@gentoo.org>
[-- Attachment #1: Type: text/plain, Size: 1078 bytes --]
Jeremy Olexa schrieb:
> Ben de Groot wrote:
>
>> So, can we have a nice little table of which functions die by themselves
>> and which ones need || die added in ebuilds? Please?
>>
>> Thanks,
>>
>
> A quick grep of /usr/lib/portage/bin clues you in that every function
> that is an external file does *not* die by itself. So, emake, do*, etc..
>
> %% grep -c die * |grep -v ":0"
> ebuild.sh:61
> etc-update:14
> isolated-functions.sh:18
> misc-functions.sh:26
> repoman:2
>
> I believe this is because you can have those functions in a subshell and
> then die won't behave predictably. I'm sure some PM people will correct
> me if I am wrong. ;)
>
> -Jeremy
>
>
From my knowledge and experience with sunrise:
some functions that dont need "|| die":
epatch, econf, eqmake3, eqmake4
some functions that need "|| die":
emake, do*
Afaik die wont work in a subshell independent of how it is called, so the die from econf wont work,
but also the "emake || die" one would also not work.
--
Thomas Sachau
Gentoo Linux Developer
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 315 bytes --]
next prev parent reply other threads:[~2008-09-30 16:48 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <bcwiO-7zO-1@gated-at.bofh.it>
[not found] ` <be8YL-33V-3@gated-at.bofh.it>
[not found] ` <be9Lf-478-5@gated-at.bofh.it>
2008-09-21 6:18 ` [gentoo-dev] Re: Default src_install for EAPI-2 or following EAPI Vaeth
2008-09-21 11:44 ` Thomas Anderson
2008-09-21 12:03 ` [gentoo-dev] " Steve Long
2008-09-21 13:04 ` [gentoo-dev] " Ulrich Mueller
2008-09-21 17:30 ` Kent Fredric
2008-09-21 18:50 ` Ulrich Mueller
2008-09-21 20:23 ` Steve Long
2008-09-21 20:46 ` Ulrich Mueller
2008-09-21 21:10 ` Steve Long
2008-09-23 19:39 ` Thomas Sachau
2008-09-23 23:21 ` Bo Ørsted Andresen
2008-09-24 5:28 ` Alec Warner
2008-09-24 9:01 ` Bo Ørsted Andresen
2008-09-24 7:46 ` Nirbheek Chauhan
2008-09-27 10:17 ` Thomas Sachau
2008-09-28 18:24 ` [gentoo-dev] Usage of econf with an additional || die Thomas Sachau
2008-09-28 18:28 ` Vlastimil Babka
2008-09-30 6:55 ` Peter Volkov
2008-09-30 10:36 ` Ben de Groot
2008-09-30 12:03 ` Jeremy Olexa
2008-09-30 16:47 ` Thomas Sachau [this message]
2008-09-30 17:10 ` Matthias Schwarzott
2008-09-30 10:39 ` Zac Medico
2008-09-29 5:16 ` [gentoo-dev] Re: Default src_install for EAPI-2 or following EAPI Nirbheek Chauhan
2008-09-30 17:05 ` Petteri Räty
2008-10-05 8:52 ` [gentoo-dev] " Ulrich Mueller
2008-10-05 14:15 ` Robert Buchholz
2008-10-05 15:45 ` Ulrich Mueller
2008-10-05 16:47 ` Robert Buchholz
2008-10-05 17:03 ` Ulrich Mueller
2008-10-05 17:58 ` Thomas Sachau
2008-09-24 0:35 ` [gentoo-dev] " Robert Buchholz
2008-09-24 8:26 ` Santiago M. Mola
2008-09-24 1:34 ` [gentoo-dev] " Steve Long
2008-09-24 6:38 ` [gentoo-dev] " Ulrich Mueller
2008-09-24 16:21 ` [gentoo-dev] OT: " Steve Long
2008-09-24 16:33 ` [gentoo-dev] " Steve Long
2008-09-24 6:40 ` [gentoo-dev] " Duncan
2008-09-22 1:35 ` Alec Warner
2008-09-22 8:22 ` Duncan
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=48E2583E.5060404@gentoo.org \
--to=tommy@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