Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5)

Unified Diff: toolchain_build/toolchain_build.py

Issue 11462002: toolchain_build: Force GCC build to recognize "system" limits.h (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client
Patch Set: Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: toolchain_build/toolchain_build.py
diff --git a/toolchain_build/toolchain_build.py b/toolchain_build/toolchain_build.py
index bab3ab4691d08f441306991bc47e4ec9f3c200a8..caf56bcc21c8dff171d80cd59452cbf4858a0262 100755
--- a/toolchain_build/toolchain_build.py
+++ b/toolchain_build/toolchain_build.py
@@ -306,7 +306,14 @@ def HostTools(target):
]],
'commands': ConfigureTargetPrep(target) + [
ConfigureGccCommand(target),
- GccCommand(target, MAKE_PARALLEL_CMD + ['all-gcc']),
+ # gcc/Makefile's install rules ordinarily look at the
+ # installed include directory for a limits.h to decide
+ # whether the lib/gcc/.../include-fixed/limits.h header
+ # should be made to expect a libc-supplied limits.h or not.
+ # Since we're doing this build in a clean environment without
+ # any libc installed, we need to force its hand here.
+ GccCommand(target, MAKE_PARALLEL_CMD + ['all-gcc',
+ 'LIMITS_H_TEST=true']),
# gcc/Makefile's install targets populate this directory
# only if it already exists.
command.Mkdir(os.path.join('%(output)s', target + '-nacl', 'bin'),
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698