[gentoo-dev] Nested die error

[gentoo-dev] Nested die error

[Caleb Tennis]

So it seems repoman doesn't like nested die calls like this:

use blah && ( emake foo || die )


But, without the parenthesis

use blah && emake foo || die

always dies (the emake functions just fine, but it returns an error once emake 
is completed).

What's the trick?  Wrap the emake in a if/then call?  Ignore repoman at the 
risk of potential finger breakage by mr_bones'?

--
gentoo-dev@gentoo.org mailing list

Re: [gentoo-dev] Nested die error

[Stefan Schweizer]

if useq blah; then
    emake foo || die "emake foo failed"
fi

we use useq because it does not output blah.

regards,
Stefan

--
gentoo-dev@gentoo.org mailing list

[gentoo-dev] Re: Nested die error

[Michael Sterrett -Mr. Bones.-]

On Thu, 14 Apr 2005, Stefan Schweizer wrote:

> we use useq because it does not output blah.

"use" doesn't output anything anymore either and anything that depends
on it doing so is already broken.

Michael Sterrett
   -Mr. Bones.-
mr_bones_@gentoo.org

--
gentoo-dev@gentoo.org mailing list

Re: [gentoo-dev] Nested die error

[Stephen Bennett]

On Thu, 2005-04-14 at 16:42 -0500, Caleb Tennis wrote:
> So it seems repoman doesn't like nested die calls like this:
> 
> use blah && ( emake foo || die )
> 
Yep, because that doesn't work.

> What's the trick?  Wrap the emake in a if/then call?  
Yes.

> Ignore repoman at the 
> risk of potential finger breakage by mr_bones'?
Well, you can as long as you don't mind your ebuild not working. :)

--
gentoo-dev@gentoo.org mailing list

Re: [gentoo-dev] Nested die error

[Caleb Tennis]

On Thursday 14 April 2005 04:54 pm, Stephen Bennett wrote:
> > use blah && ( emake foo || die )
>
> Yep, because that doesn't work.

Wow.  I've been doing it for years.  What's broken about it, the nested die ro 
the "use blah &&" part?

Caleb
--
gentoo-dev@gentoo.org mailing list

Re: [gentoo-dev] Nested die error

[Mike Frysinger]

On Thursday 14 April 2005 05:58 pm, Caleb Tennis wrote:
> On Thursday 14 April 2005 04:54 pm, Stephen Bennett wrote:
> > > use blah && ( emake foo || die )
> >
> > Yep, because that doesn't work.
>
> Wow.  I've been doing it for years.  What's broken about it, the nested die
> ro the "use blah &&" part?

as hinted elsewhere, the problem is that ( ) spawns a subshell ... that means 
the call to 'die' aborts the sub (i.e. forked) shell, not the parent (i.e. 
the ebuild) shell ... so when emake fails, portage wont notice
-mike
--
gentoo-dev@gentoo.org mailing list

Re: [gentoo-dev] Nested die error

[Olivier Crête]

[-- Attachment #1: Type: text/plain, Size: 706 bytes --]

On Thu, 2005-14-04 at 16:42 -0500, Caleb Tennis wrote:
> So it seems repoman doesn't like nested die calls like this:
> 
> use blah && ( emake foo || die )
> 
> 
> But, without the parenthesis
> 
> use blah && emake foo || die
> 
> always dies (the emake functions just fine, but it returns an error once emake 
> is completed).
> 
> What's the trick?  Wrap the emake in a if/then call?  Ignore repoman at the 
> risk of potential finger breakage by mr_bones'?

You can do if/then/fi or
use blah && { emake foo || die; } 

{} will not spawn a sub-shell... just execute them in the same shell so
the die will work properly

-- 
Olivier Crête
tester@gentoo.org
x86 Security Liaison

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: [gentoo-dev] Nested die error

[Mike Frysinger]

On Thursday 14 April 2005 05:57 pm, Olivier Crête wrote:
> You can do if/then/fi or
> use blah && { emake foo || die; }
>
> {} will not spawn a sub-shell... just execute them in the same shell so
> the die will work properly

both syntaxes are ugly imho but the best thing would be to use if/then/fi 
because many more developers understand that than the 1 line { } code
-mike

--
gentoo-dev@gentoo.org mailing list

Re: [gentoo-dev] Nested die error

[Ciaran McCreesh]

[-- Attachment #1: Type: text/plain, Size: 1029 bytes --]

On Thu, 14 Apr 2005 16:42:22 -0500 Caleb Tennis <caleb@gentoo.org>
wrote:
| use blah && ( emake foo || die )

Here's what The Doc will have to say about this:

``die`` and Subshells
---------------------

.. Warning:: ``die`` **will not work in a subshell**.

The following code will not work as expected, since the ``die`` is
inside a subshell: ::

    [[ -f foorc ]] && ( update_foorc || die "couldn't update foorc" )

The correct way to rewrite this is to use an ``if`` block: ::

    if [[ -f foorc ]] ; then
        update_foorc || die "couldn't update foorc"
    fi

When using pipes, a subshell is introduced, so the following is unsafe:
::

    cat list | while read file ; do epatch ${file} ; done

Using input redirection (see `Abuse of cat`_) avoids this problem: ::

    while read file ; do epatch ${file} ; done < list


-- 
Ciaran McCreesh : Gentoo Developer (Vim, Fluxbox, shell tools)
Mail            : ciaranm at gentoo.org
Web             : http://dev.gentoo.org/~ciaranm


[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [gentoo-dev] Nested die error

[Vibhav Garg]

Caleb Tennis wrote:

>So it seems repoman doesn't like nested die calls like this:
>
>use blah && ( emake foo || die )
>
>  
>
i believe this is a question in the developer end quiz too:-)

>But, without the parenthesis
>
>use blah && emake foo || die
>
>always dies (the emake functions just fine, but it returns an error once emake 
>is completed).
>
>What's the trick?  Wrap the emake in a if/then call?  Ignore repoman at the 
>risk of potential finger breakage by mr_bones'?
>
>--
>gentoo-dev@gentoo.org mailing list
>
>
>  
>

--
gentoo-dev@gentoo.org mailing list

Re: [gentoo-dev] Nested die error

[Ciaran McCreesh]

[-- Attachment #1: Type: text/plain, Size: 509 bytes --]

On Thu, 14 Apr 2005 17:05:25 -0500 Vibhav Garg <vgarg@gentoo.org> wrote:
| Caleb Tennis wrote:
| >So it seems repoman doesn't like nested die calls like this:
| >
| >use blah && ( emake foo || die )
| >
| i believe this is a question in the developer end quiz too:-)

It is now. It wasn't back in the good old days when Caleb joined :)

-- 
Ciaran McCreesh : Gentoo Developer (Vim, Fluxbox, shell tools)
Mail            : ciaranm at gentoo.org
Web             : http://dev.gentoo.org/~ciaranm


[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]