This is the mail archive of the
mailing list for the GCC project.
[Darwin, committed] Pick up SDKROOT as the sysroot fallback.
- From: Iain Sandoe <iain at sandoe dot co dot uk>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Cc: Mike Stump <mrs at gcc dot gnu dot org>
- Date: Thu, 3 Oct 2019 21:23:30 +0100
- Subject: [Darwin, committed] Pick up SDKROOT as the sysroot fallback.
For compatibility with xcrun (and the behaviour of the clang driver), make use
of the setting of the SDKROOT environment variable when it is available.
This applies to both finding headers and libraries (i.e. it is also passed to
1. User's command-line specified --sysroot= or -isysroot.
2. The SDKROOT variable when set, and validated.
3. Any sysroot provided by --with-sysroot= configuration parameter.
SDKROOT is checked thus:
2. That it starts with "/" (i.e. 'absolute').
3. That it is not "/" only (since that's the default).
4. That it is readable by the process executing the driver.
This is pretty much the same rule set as used by the clang driver (modulo the
rule for —with-sysroot= configuration).
NOTE: (3) might turn out to be overly restrictive in the case that we
have configured with --with-sysroot= and then we want to run on a system
with an installation of the headers/libraries in /. We can revisit this
if that turns out to be an important use-case.
So one can do:
xcrun --sdk macosx /path/to/gcc ....
and that provides the SDK path as the sysroot to GCC as expected.
CAVEAT: An unfortunate effect of the fact that "gcc" (and "g++") are
executables in the Xcode installation, which are found ahead of any such
named in the $PATH:
xcrun --sdk macosx gcc ....
does *not* work, instead that executes the clang from the xcode/commmand
line tools installation.
xcrun --sdk macosx x64_64-apple-darwinXX-gcc ...
does work as expected, however.
tested on x86_64-darwin16, applied to mainline,
2019-10-03 Iain Sandoe <email@example.com>
* config/darwin-driver.c (maybe_get_sysroot_from_sdkroot): New.
(darwin_driver_init): Use the sysroot provided by SDKROOT when that
is available and the user has not set one on the command line.
diff --git a/gcc/config/darwin-driver.c b/gcc/config/darwin-driver.c
index 49716fabf2..82ac231dc4 100644
@@ -210,6 +210,28 @@ darwin_default_min_version (void)
+/* See if we can find the sysroot from the SDKROOT environment variable. */
+static const char *
+ const char *maybe_sysroot = getenv ("SDKROOT");
+ /* We'll use the same rules as the clang driver, for compatibility.
+ 1) The path must be absolute
+ 2) Ignore "/", that is the default anyway and we do not want the
+ sysroot semantics to be applied to it.
+ 3) It must exist (actually, we'll check it's readable too). */
+ if (maybe_sysroot == NULL
+ || *maybe_sysroot != '/'
+ || strlen (maybe_sysroot) == 1
+ || access (maybe_sysroot, R_OK) == -1)
+ return NULL;
+ return xstrndup (maybe_sysroot, strlen (maybe_sysroot));
/* Translate -filelist and -framework options in *DECODED_OPTIONS
(size *DECODED_OPTIONS_COUNT) to use -Xlinker so that they are
considered to be linker inputs in the case that no other inputs are
@@ -234,6 +256,7 @@ darwin_driver_init (unsigned int *decoded_options_count,
bool appendM64 = false;
const char *vers_string = NULL;
bool seen_version_min = false;
+ bool seen_sysroot_p = false;
for (i = 1; i < *decoded_options_count; i++)
@@ -314,6 +337,11 @@ darwin_driver_init (unsigned int *decoded_options_count,
+ case OPT__sysroot_:
+ case OPT_isysroot:
+ seen_sysroot_p = true;
@@ -375,6 +403,22 @@ darwin_driver_init (unsigned int *decoded_options_count,
&(*decoded_options)[*decoded_options_count - 1]);
+ if (! seen_sysroot_p)
+ /* We will pick up an SDKROOT if we didn't specify a sysroot and treat
+ it as overriding any configure-time --with-sysroot. */
+ const char *sdkroot = maybe_get_sysroot_from_sdkroot ();
+ if (sdkroot)
+ *decoded_options = XRESIZEVEC (struct cl_decoded_option,
+ generate_option (OPT__sysroot_, sdkroot, 1, CL_DRIVER,
+ &(*decoded_options)[*decoded_options_count - 1]);
/* We will need to know the OS X version we're trying to build for here
so that we can figure out the mechanism and source for the sysroot to
be used. */