cc_configure: removes -no-canonical-prefixes from the list of default flags
This might causes problem in a future remote execution services but
in that kind of service it would make more sense to not use the
auto-configured C++ toolchain.
Tested with bazel test //tools/cpp/test:*
Fixes #1204.
--
MOS_MIGRATED_REVID=123643630
diff --git a/tools/cpp/cc_configure.bzl b/tools/cpp/cc_configure.bzl
index 364305e..d31a355 100644
--- a/tools/cpp/cc_configure.bzl
+++ b/tools/cpp/cc_configure.bzl
@@ -180,8 +180,7 @@
"-lstdc++",
"-lm", # Some systems expect -lm in addition to -lstdc++
# Anticipated future default.
- ] + _add_option_if_supported(repository_ctx, cc, "-no-canonical-prefixes") +
- _add_option_if_supported(repository_ctx, cc, "-Wl,-no-as-needed") + (
+ ] + _add_option_if_supported(repository_ctx, cc, "-Wl,-no-as-needed") + (
[
"-undefined",
"dynamic_lookup",
@@ -203,8 +202,6 @@
"cxx_builtin_include_directory": _get_cxx_inc_directories(repository_ctx, cc),
"objcopy_embed_flag": ["-I", "binary"],
"unfiltered_cxx_flag":
- # Anticipated future default.
- _add_option_if_supported(repository_ctx, cc, "-no-canonical-prefixes") +
_add_option_if_supported(repository_ctx, cc, "-fno-canonical-system-headers") + [
# Make C++ compilation deterministic. Use linkstamping instead of these
# compiler symbols.
