Bug 315527 – dev-lang/swig-1.3.40-r1 ebuild dies with USE="-doc"

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description William Throwe 2010-04-16 01:18:08 UTC]

The line

( use doc && dohtml -r Doc/{Devel,Manual} ) || die

doesn't do what you want.

Reproducible: Always

[Comment 1 frilled 2010-04-16 05:31:19 UTC]

It's also confusing, since it dies with 

 * ERROR: dev-lang/swig-1.3.40-r1 failed:
 *   (no error message)

[Comment 2 thanasis 2010-04-16 05:42:01 UTC]

on ~amd64:
...<snip>...
Installing /var/tmp/portage/dev-lang/swig-1.3.40-r1/image//usr/share/swig/1.3.40/std/std_wstring.i
Installing /var/tmp/portage/dev-lang/swig-1.3.40-r1/image//usr/share/swig/1.3.40/std/std_carray.swg
Installation complete
 * ERROR: dev-lang/swig-1.3.40-r1 failed:
 *   (no error message)
 *
 * Call stack:
 *     ebuild.sh, line  54:  Called src_install
 *   environment, line 223:  Called die
 * The specific snippet of code:
 *       ( use doc && dohtml -r Doc/{Devel,Manual} ) || die
 *
 * If you need support, post the output of 'emerge --info =dev-lang/swig-1.3.40-r1',
 * the complete build log and the output of 'emerge -pqv =dev-lang/swig-1.3.40-r1'.
 * The complete build log is located at '/var/tmp/portage_logs/dev-lang:swig-1.3.40-r1:20100416-053244.log'.
 * The ebuild environment file is located at '/var/tmp/portage/dev-lang/swig-1.3.40-r1/temp/environment'.
 * S: '/var/tmp/portage/dev-lang/swig-1.3.40-r1/work/swig-1.3.40'

[Comment 3 Panagiotis Christopoulos (RETIRED) 2010-04-16 06:59:38 UTC]

true, I'll fix it

[Comment 4 vp 2010-04-16 07:26:34 UTC]

should be 

use doc && ( dohtml -r Doc/{Devel,Manual} || die )

[Comment 5 Panagiotis Christopoulos (RETIRED) 2010-04-16 07:38:28 UTC]

  16 Apr 2010; Panagiotis Christopoulos <[e-mail]>
  swig-1.3.40-r1.ebuild:
  fix doc handling, bug 315527, thanks to William Throwe < wthrowe at
  mit.edu > , for reporting it


Fixed, InCVS. I'll keep the bug open for a while. This was my mistake. Thank you all, for reporting this.

[Comment 6 Xavier Miller (RETIRED) 2010-04-16 08:13:31 UTC]

If I change the ebuild as describe, it works for me.

[Comment 7 Panagiotis Christopoulos (RETIRED) 2010-04-16 08:45:42 UTC]

(In reply to comment #6)
> If I change the ebuild as describe, it works for me.
> 
It's already fixed, just resync your portage.

[Comment 8 William Throwe 2010-04-16 13:19:33 UTC]

(In reply to comment #4)
> should be 
> 
> use doc && ( dohtml -r Doc/{Devel,Manual} || die )
> 

That's better, but now the die won't work because it's in a subshell.  You need to use a real if:
if use doc ; then
    dohtml blah || die
fi

[Comment 9 Arfrever Frehtes Taifersar Arahesis (RETIRED) 2010-04-16 14:15:56 UTC]

(In reply to comment #8)

I have applied this change.

[Comment 10 Panagiotis Christopoulos (RETIRED) 2010-04-17 10:06:06 UTC]

*** Bug 315705 has been marked as a duplicate of this bug. ***