Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 350081 - dev-lang/spidermonkey-1.9.2.13 missing DEPEND on app-arch/zip
Summary: dev-lang/spidermonkey-1.9.2.13 missing DEPEND on app-arch/zip
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: All Linux
: High normal
Assignee: Dirkjan Ochtman (RETIRED)
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2010-12-29 21:16 UTC by Stelian Ionescu
Modified: 2010-12-29 23:26 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Add DEPEND on app-arch/zip and app-arch/unzip (spidermonkey-1.9.2.13.ebuild.patch,284 bytes, patch)
2010-12-29 21:17 UTC, Stelian Ionescu
Details | Diff
Add DEPEND on app-arch/zip (spidermonkey-1.9.2.13.ebuild.patch,267 bytes, patch)
2010-12-29 21:27 UTC, Stelian Ionescu
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Stelian Ionescu 2010-12-29 21:16:50 UTC
The configure script checks for zip and unzip and errs if any not found.
Comment 1 Stelian Ionescu 2010-12-29 21:17:37 UTC
Created attachment 258393 [details, diff]
Add DEPEND on app-arch/zip and app-arch/unzip
Comment 2 Stelian Ionescu 2010-12-29 21:24:24 UTC
On a closer look, only app-arch/zip is required; on line 4284 of configure, there's this:

if test -z "$ZIP" || test "$ZIP" = ":"; then
    { echo "configure: error: zip not found in \$PATH" 1>&2; exit 1; }
fi

No such code for unzip.

Btw, all ebuilds of net-libs/xulrunner and www-client/firefox appear to have the same bug
Comment 3 Stelian Ionescu 2010-12-29 21:27:41 UTC
Created attachment 258394 [details, diff]
Add DEPEND on app-arch/zip

Fixes last patch
Comment 4 Panagiotis Christopoulos (RETIRED) gentoo-dev 2010-12-29 21:33:38 UTC
True.
<snip>
checking for full perl installation... yes
checking for python2.7... /usr/bin/python2.7
checking for doxygen... :
checking for whoami... /usr/bin/whoami
checking for autoconf... /usr/bin/autoconf
checking for unzip... /usr/bin/unzip
checking for zip... no
configure: error: zip not found in $PATH

!!! Please attach the following file when seeking support:
!!! /var/tmp/portage/dev-lang/spidermonkey-1.9.2.13/work/mozilla-1.9.2/js/src/config.log
 * ERROR: dev-lang/spidermonkey-1.9.2.13 failed:
 *   econf failed
 * 
 * Call stack:
 *     ebuild.sh, line   56:  Called src_configure
 *   environment, line 4646:  Called econf
 *     ebuild.sh, line  555:  Called die
 * The specific snippet of code:
 *                      die "econf failed"
 * 
 * If you need support, post the output of 'emerge --info =dev-lang/spidermonkey-1.9.2.13',
 * the complete build log and the output of 'emerge -pqv =dev-lang/spidermonkey-1.9.2.13'.
 * The complete build log is located at '/var/tmp/portage/dev-lang/spidermonkey-1.9.2.13/temp/build.log'.
 * The ebuild environment file is located at '/var/tmp/portage/dev-lang/spidermonkey-1.9.2.13/temp/environment'.
 * S: '/var/tmp/portage/dev-lang/spidermonkey-1.9.2.13/work/mozilla-1.9.2'
</snip>
Comment 5 Jory A. Pratt gentoo-dev 2010-12-29 23:26:20 UTC
(In reply to comment #2)
> On a closer look, only app-arch/zip is required; on line 4284 of configure,
> there's this:
> 
> if test -z "$ZIP" || test "$ZIP" = ":"; then
>     { echo "configure: error: zip not found in \$PATH" 1>&2; exit 1; }
> fi
> 
> No such code for unzip.
> 
> Btw, all ebuilds of net-libs/xulrunner and www-client/firefox appear to have
> the same bug
> 

These pick up the dep from the eclasses, so they are not effected.