1061388 - subconfigure.py should not rely on shebang for invoking config.status

Status: RESOLVED DUPLICATE of bug 1052560

(Firefox Build System :: General, defect)

Description

[Michael Vines [:m1] [:evilmachines]]

Given a very deep build root directory, build/autoconf/config.status.m4 will construct a #! for config.status that exceeds the ~128 character limit for the path to the python interpreter on some systems (eg. Linux).

This causes a failed build with an error similar to
---
...
checking for posix_fadvise... no
checking for posix_fallocate... no
checking for setlocale... yes
checking for localeconv... (cached) no
updating cache /local/mnt/workspace/lnxbuild/project/trees_in_use/free_tree_b2g_manifest_b2g_toaster_grease_commander_4330630/checkout/out/target/product/toaster/obj/objdir-gecko/config.cache
creating ./config.status
/bin/sh: ./config.status: /local/mnt/workspace/lnxbuild/project/trees_in_use/free_tree_b2g_manifest_b2g_: bad interpreter: No such file or directory
*** Fix above errors and then restart with\
               "make -f client.mk build"
make[3]: *** [configure] Error 1
---

Other places in the build system seem to always take care to explicitly invoke |$PYTHON ./config.status| to avoid this problem.

Comment 1

[Michael Vines [:m1] [:evilmachines]]

Attachment: + $PYTHON

Comment 2

[Mike Hommey [:glandium]]

Comment on attachment 8482484 [details] [diff] [review]
+ $PYTHON

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

Not all config.statuses subconfigure.py invokes are python scripts.

Comment 3

[Mike Hommey [:glandium]]

Also, there's already a bug on file on that issue... if I can find it.