Open Bug 1052560 Opened 10 years ago Updated 2 years ago

Shell doesn't run scripts with a very long #! interpreter

Categories

(Firefox Build System :: General, defect)

x86_64
Linux
defect

Tracking

(Not tracked)

People

(Reporter: oleksiy.avramchenko, Unassigned)

References

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:31.0) Gecko/20100101 Firefox/31.0 (Beta/Release)
Build ID: 20140715215003

Steps to reproduce:

Running config.status scripts from subconfigure.py via shell -c won't work for the python scripts with very long #! interpteter name, causing FXOS gecko build to fail. Seems to start after bug #1046533 merge. This happens on Ubuntu 12.04 and most likely bash is to blaim.

It's hidden on normal runs when source tree is located close to the /, but our build servers got affected where tree is checked out into the quite long directory name where final #! /*/_virtualenv/bin/python is like 170 chars long.

For the local quick fix we changed the subconfigure.py to read the first line of the script and use the specified interpreter instead of data['shell'] -c.
Sample patch of the quick local workaround.
CC for review.
Flags: needinfo?(mh+mozilla)
Comment on attachment 8471701 [details] [diff] [review]
0001-Call-exact-interpreter-for-the-config.status.patch

Review of attachment 8471701 [details] [diff] [review]:
-----------------------------------------------------------------

That will break on windows because /bin/sh won't be recognized.
Attachment #8471701 - Flags: review-
Flags: needinfo?(mh+mozilla)
(In reply to Loic from comment #2)
> CC for review.

Loic, it's the second time you needinfo? me for a review. Please use the review? flag on the patch to review instead.
(In reply to Mike Hommey [:glandium] from comment #4)
> (In reply to Loic from comment #2)
> > CC for review.
> 
> Loic, it's the second time you needinfo? me for a review. Please use the
> review? flag on the patch to review instead.

Yeah, sorry, I'm not very familiar with reviewing, and as I only sort bugs (so I use needinfo? a lot), I forgot to use the review? flag. :)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Product: Core → Firefox Build System
Severity: normal → S3

The severity field for this bug is relatively low, S3. However, the bug has 3 duplicates.
:ahochheiden, could you consider increasing the bug severity?

For more information, please visit auto_nag documentation.

Flags: needinfo?(ahochheiden)

The last needinfo from me was triggered in error by recent activity on the bug. I'm clearing the needinfo since this is a very old bug and I don't know if it's still relevant.

Flags: needinfo?(ahochheiden)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: