aboutsummaryrefslogtreecommitdiffhomepage
path: root/tools/cpp/cc_configure.bzl
Commit message (Collapse)AuthorAge
...
* cc_configure: use the wrapper only if on Darwin and ship the environment ↵Gravatar Damien Martin-Guillerez2016-05-09
| | | | | | | | | | | | | | | | only if in homebrew This will fix the sandboxing issues for rules_rust and rules_go while still allowing to build inside homebrew. Should unblock #1238 Fixes bazelbuild/rules_go#20. Fixes bazelbuild/rules_rust#4. Tested under homebrew + classic tests. -- MOS_MIGRATED_REVID=121834186
* Rollback of commit 5e6218ea4e83a7fadfe54c35488781da550ce13e.Gravatar Damien Martin-Guillerez2016-05-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | *** Reason for rollback *** Roll-forward with fix: - Previous test did let the all_files group with the wrapper script, and it got removed when creating the actual change. Added it back Tested with `./compile.sh determinism` *** Original change description *** Automated [] rollback of commit 87a58589d6770a6e9a0f998274bfe69f1feb6014. *** Reason for rollback *** The sandbox fix did not fix. More work is needed. Fixes #1239 *** Original change description *** Automated [] rollback of commit b4549fe8dfb29f87fd37b38bf21a5b4bef818b12. *** Reason for rollback *** Rollforward with fixes: - Quote single-quote in the environment exports - Exclude the following environment variables: _ and dotted names - Add the wrapper to the dependency of the crosstool chain so it gets shipped to the sandbox. Hopefu... -- MOS_MIGRATED_REVID=121667505
* Rollback of commit 87a58589d6770a6e9a0f998274bfe69f1feb6014.Gravatar Damien Martin-Guillerez2016-05-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | *** Reason for rollback *** The sandbox fix did not fix. More work is needed. Fixes #1239 *** Original change description *** Automated [] rollback of commit b4549fe8dfb29f87fd37b38bf21a5b4bef818b12. *** Reason for rollback *** Rollforward with fixes: - Quote single-quote in the environment exports - Exclude the following environment variables: _ and dotted names - Add the wrapper to the dependency of the crosstool chain so it gets shipped to the sandbox. Hopefully after that we can cut a new release and have a homebrew package again #1177. *** Original change description *** Automated [] rollback of commit f1f24fc4b5aa83b7a4b872ec8f1c3a369799c081. *** Reason... *** -- MOS_MIGRATED_REVID=121665313
* Rollback of commit b4549fe8dfb29f87fd37b38bf21a5b4bef818b12.Gravatar Damien Martin-Guillerez2016-05-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | *** Reason for rollback *** Rollforward with fixes: - Quote single-quote in the environment exports - Exclude the following environment variables: _ and dotted names - Add the wrapper to the dependency of the crosstool chain so it gets shipped to the sandbox. Hopefully after that we can cut a new release and have a homebrew package again #1177. *** Original change description *** Automated [] rollback of commit f1f24fc4b5aa83b7a4b872ec8f1c3a369799c081. *** Reason for rollback *** Broke ci.bazel.io See #1231. *** Original change description *** Ship the environment fixed at configure time to the C++ compiler Use a wrapper script on all platform to ship the environment to the C++ compiler. This should enable building with Homebrew special setup and will likely reduce the number of hard corner cases to solve. Should... *** -- MOS_MIGRATED_REVID=121497195
* Rollback of commit f1f24fc4b5aa83b7a4b872ec8f1c3a369799c081.Gravatar Damien Martin-Guillerez2016-05-04
| | | | | | | | | | | | | | | | | | | | | | | *** Reason for rollback *** Broke ci.bazel.io See #1231. *** Original change description *** Ship the environment fixed at configure time to the C++ compiler Use a wrapper script on all platform to ship the environment to the C++ compiler. This should enable building with Homebrew special setup and will likely reduce the number of hard corner cases to solve. Should unblock #1177. -- MOS_MIGRATED_REVID=121468271
* Ship the environment fixed at configure time to the C++ compilerGravatar Damien Martin-Guillerez2016-05-04
| | | | | | | | | | | | | | Use a wrapper script on all platform to ship the environment to the C++ compiler. This should enable building with Homebrew special setup and will likely reduce the number of hard corner cases to solve. Should unblock #1177. -- Change-Id: I36f09edaf131f65c730bdd626ce521478ff587c7 Reviewed-on: https://bazel-review.googlesource.com/3560 MOS_MIGRATED_REVID=121462120
* cc_configure: add -lm to the default linker flagsGravatar Damien Martin-Guillerez2016-04-28
| | | | | | | | | | | | | | Some C++ toolchain requires -lm when passing -lstdc++. Always add it. We should probably make that depend on what kind of binary we actually want to build but that's beyond the scope of cc_configure. Fixes #1190. -- MOS_MIGRATED_REVID=121015578
* cc_configure: do not assume the message from gcc is in EnglishGravatar Damien Martin-Guillerez2016-04-28
| | | | | | | Fixes #1204. -- MOS_MIGRATED_REVID=120997826
* cc_configure.bzl: support spaces in CC environmentGravatar Damien Martin-Guillerez2016-04-27
| | | | | | | Also support CC to be empty and fallback to gcc. -- MOS_MIGRATED_REVID=120907744
* cc_configure.bzl: Add -Wl,-no-as-needed as default link optionsGravatar Damien Martin-Guillerez2016-04-26
| | | | | -- MOS_MIGRATED_REVID=120791766
* cc_configure: Add -B to compiler flag tooGravatar Damien Martin-Guillerez2016-04-22
| | | | | | | | | On some target platform gcc invokes external tools during compilation, see #1152. Also support absolute path when looking for gcc. This needs to be cherry-picked to fix #1177. -- MOS_MIGRATED_REVID=120581887
* cc_configure.bzl: strip end of line when looking for the cpuGravatar Damien Martin-Guillerez2016-04-20
| | | | | | | As noticed in #1122. -- MOS_MIGRATED_REVID=120315240
* Added configuration for wrapper scripts of msvc toolchainGravatar Yun Peng2016-04-20
| | | | | | | | | Now we only need to specify --cpu=x64_windows_msvc when using msvc toolchain -- Change-Id: Id501dd9ef2fd6553285677605ec75e80499b9ef7 Reviewed-on: https://bazel-review.googlesource.com/#/c/3441 MOS_MIGRATED_REVID=120260588
* cc_configure: uses which on the CC environment variableGravatar Damien Martin-Guillerez2016-04-20
| | | | | | | | | It's wrong to assume it points to an absolute path. Discovered in issue #1152. -- MOS_MIGRATED_REVID=120242469
* cc_configure: Add piii to the list of supported cpu_valueGravatar Damien Martin-Guillerez2016-04-19
| | | | | | | Fixes #1122. -- MOS_MIGRATED_REVID=120223970
* cc_configure: always add -B/usr/bin to the list of gcc optionGravatar Damien Martin-Guillerez2016-04-19
| | | | | | | Fixes #760. -- MOS_MIGRATED_REVID=120217217
* Fix cc_configure include path for Frameworks on OS X.Gravatar Damien Martin-Guillerez2016-04-19
| | | | | | | | | | | | cc_configure assumed the paths returned by $(CC) -E -xc++ -v where not containing any extra information but clang on OS X adds " (framework directory)" for path from Frameworks. This change strip this out. -- Change-Id: I90617e825100f86a1f0991e128755802da2c7afd Reviewed-on: https://bazel-review.googlesource.com/3389 MOS_MIGRATED_REVID=120210800
* Fix issues with cc_configureGravatar Alpha Lam2016-04-01
| | | | | | | | | | | | | | Fixing a few issues with cc_configure that I encountered. * The argument for rpath should be -Wl,-rpath. * cc_configure should consider CPLUS_INCLUDE_PATH. I ran into this problem when using a crosstool gcc which needs -I/usr/include. * Check if -Wunused-but-set-parameter is available. -- Change-Id: I73198b5b17674ecbf1b511e23fcc9331ca96c8e0 Reviewed-on: https://bazel-review.googlesource.com/#/c/3210/ MOS_MIGRATED_REVID=118763218
* cc_configure.bzl: fix path to static crosstoolGravatar Damien Martin-Guillerez2016-03-31
| | | | | | | | | | | | | | | | Static crosstool where copied in the wrong path, leading to not found package and breaking the Windows build. Also the BUILD.bazel file contains now a reference to @cc_configure// package, so we need another copy. Tested: manually patched the change on the Windows slave and ran ./compile.sh Fixes #1101. -- MOS_MIGRATED_REVID=118674922
* Adds Skylark Remote Repository documentationGravatar Damien Martin-Guillerez2016-03-24
| | | | | | | Fixes #1043 -- MOS_MIGRATED_REVID=118039426
* cc_configure: Fix fallout of renaming ctx to repository_ctx.Gravatar Damien Martin-Guillerez2016-03-18
| | | | | -- MOS_MIGRATED_REVID=117478721
* cc_configure: fallback to static crosstool for unsupported platformGravatar Damien Martin-Guillerez2016-03-15
| | | | | -- MOS_MIGRATED_REVID=117258091
* Use repository_ctx for each usage of repository context.Gravatar Damien Martin-Guillerez2016-03-14
| | | | | | | | This will avoid confusion with the normal Skylark rule context. Also fixed indentation of cc_configure.bzl -- MOS_MIGRATED_REVID=117011107
* cc_configure: fix issues with CentOS 6.7 and custom gccGravatar Damien Martin-Guillerez2016-03-08
| | | | | | | This setup was provided on issue #760. -- MOS_MIGRATED_REVID=116656431
* cc_configure: fix various small errorsGravatar Damien Martin-Guillerez2016-03-04
| | | | | | | | | | | - Skylark does not allow implicit concatenation that was added by the linter - osx_cc_wrapper renaming was not applied everywhere - arm toolchain was missing in the toolchain_suite Also fix the test to use the correct cc_configure.bzl file. -- MOS_MIGRATED_REVID=116362768
* Add a cc_configure rule to auto-configure C++ crosstoolGravatar Damien Martin-Guillerez2016-03-02
This is the last step of http://goo.gl/fD4ZsY (issue #893). Tests are in a separate change because they requires pretty complex setup. -- MOS_MIGRATED_REVID=116141979