Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 600488 - www-client/firefox-50.0 - src_install() failes when bash isn't current shell
Summary: www-client/firefox-50.0 - src_install() failes when bash isn't current shell
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Mozilla Gentoo Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: nonbash
  Show dependency tree
 
Reported: 2016-11-22 15:11 UTC by Ian Stakenvicius (RETIRED)
Modified: 2022-08-27 18:45 UTC (History)
1 user (show)

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

Attachments
build.log (firefox-50.0.1-build.log.tgz,504.95 KB, application/x-gtar)
2016-11-29 20:22 UTC, William 		Details
ff.build.50.0.2.log (ff.build.50.0.2.log.tgz,501.48 KB, application/x-gtar)
2016-12-06 18:26 UTC, William 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ian Stakenvicius (RETIRED) gentoo-dev 2016-11-22 15:11:17 UTC 
As alerted via https://github.com/gentoo/gentoo/pull/2891 .

More investigation is required but it would seem that the mozilla build system isn't honouring the SHELL it detects at configure time.  This should likely be fixed via a build system patch rather than an override?
Comment 1 William 2016-11-29 20:22:18 UTC 
Created attachment 454712 [details]
build.log

So I'm running into this bug with mksh as /bin/sh.
This is with www-client/firefox-50.0.1
I attached the build.
Comment 2 William 2016-12-06 18:26:59 UTC 
Created attachment 455288 [details]
ff.build.50.0.2.log

same thing with 50.0.2
Comment 3 William 2016-12-23 11:28:30 UTC 
Same thing with: firefox-50.1.0

It worked by forcing SHELL to bash on emake install, do you want me to submit that patch again?
Comment 4 Ian Stakenvicius (RETIRED) gentoo-dev 2017-01-27 16:43:30 UTC 
commit 61fe4f891af6a622ebb4a3754f07ac5a54f5c629
Author: Ian Stakenvicius <axs@gentoo.org>
Date:   Fri Jan 27 11:03:08 2017 -0500

    www-client/firefox: adjust SHELL, other var specs on calls to emake
    
    Aligned the use of forcing various variables on calls to emake, so that they
    are common across all current versions of firefox.  This should address
    two bugs related to SHELL: bug 602370 and bug 600488
    
    Package-Manager: portage-2.3.0


Note, this commit only adjusted firefox-45.7 and firefox-51, as previous versions are or will shortly be removed from the gentoo repo.