On 9/25/24 9:22 PM, Stephanos Ioannidis wrote:
configure script was adding the target directory flags, including the
'-B' flags for the executable prefix and the '-isystem' flags for the
pre-installed header directories, to the target flags only for
non-Canadian builds under the premise that the host binaries under the
executable prefix will not be able to execute on the build system for
Canadian builds.

While that is true for the '-B' flags specifying the executable prefix,
the '-isystem' flags specifying the pre-installed header directories are
not affected by this and do not need special handling.

This patch updates the configure script to always add the 'include' and
'sys-include' pre-installed header directories to the target search
path, in order to ensure that the availability of the pre-installed
header directories in the search path is consistent across non-Canadian
and Canadian builds.

When '--with-headers' flag is specified, this effectively ensures that
the libc headers, that are copied from the specified header directory to
the sys-include directory, are used by libstdc++.

ChangeLog:

     * configure.ac: Always add pre-installed heades to search path.
     * configure: Regenerate.
Thanks.  I've pushed this to the trunk.

FTR, this has been in my tester for quite a while, so it's been bootstrapped on aarch64, riscv64, ppc64 etc etc. I don't do any canadian crosses, so the primary purpose of the patch is not getting tested. But it shouldn't cause any problems for natives or traditional crosses.

jeff

Reply via email to