public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
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 --]

  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