[PATCH][RFC] Extend locations where to seach for Fortran pre-include.
Joseph Myers
joseph@codesourcery.com
Fri Nov 23 18:08:00 GMT 2018
On Fri, 23 Nov 2018, Martin Liška wrote:
> Looks the problematic is quite complex as I can understand. I prepared a patch
> that should hopefully follow advises provided.
I don't see how this version ensures that NATIVE_SYSTEM_HEADER_DIR is
properly sysrooted. Note there's add_sysrooted_prefix separate from
add_prefix (but that's *not* the correct thing to use here because it uses
target_sysroot_suffix whereas you need target_sysroot_hdrs_suffix).
The last argument to add_prefix, which you're setting to true, is
os_multilib. But using the OS multilib scheme seems wrong here, because
the OS multilib names are names like "../lib64", which only make sense in
directories called lib - you don't want to end up searching a directory
<something>/include/finclude/../lib64.
In the multiarch case, do you want
<something>/include/finclude/<multiarch> or
<something>/include/<multiarch>/finclude? (This is where I'd hope Debian
/ Ubuntu GCC people would comment.)
--
Joseph S. Myers
joseph@codesourcery.com
More information about the Fortran
mailing list