RFR: 8323667: Library debug files contain non-reproducible full gcc include paths [v2]

Magnus Ihse Bursie ihse at openjdk.org
Tue Jan 16 12:58:20 UTC 2024


On Mon, 15 Jan 2024 14:38:58 GMT, Andrew Leonard <aleonard at openjdk.org> wrote:

>> For gcc toolchains in ALLOW_ABSOLUTE_PATHS_IN_OUTPUT=False builds, this PR finds the location of the gcc system include paths, and adds -fdebug-prefix-map flags to map them to a standard location. Thus making the debuginfo and resulting libraries reproducible when using DevKits in different path locations.
>
> Andrew Leonard has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
> 
>  - 8323667: Library debug files contain non-reproducible full gcc include paths
>    
>    Signed-off-by: Andrew Leonard <anleonar at redhat.com>
>  - 8323667: Library debug files contain non-reproducible full gcc include paths
>    
>    Signed-off-by: Andrew Leonard <anleonar at redhat.com>
>  - Merge branch 'master' of github.com:openjdk/jdk into gcc_debug_map
>  - 8323667: Library debug files contain non-reproducible full gcc include paths
>    
>    Signed-off-by: Andrew Leonard <anleonar at redhat.com>

Have you checked with `strings` if there are any other hard-coded paths, or are these really the only ones left?

make/autoconf/flags-cflags.m4 line 194:

> 192:     # Add gcc system include mapping => /usr/local/gcc_include
> 193:     #   Find location of stddef.h using build C compiler
> 194:     GCC_SYSTEM_INCLUDE=`$ECHO "#include <stddef.h>" | $CC $GCC_SYSROOT_PARAM -v -E - 2>&1 | $GREP stddef | $TAIL -1 | $TR -s " " | $CUT -d'"' -f2`

Don't we have a properly setup `CFLAGS` here that you can use, instead of having to specify the sysroot manually?

-------------

PR Comment: https://git.openjdk.org/jdk/pull/17399#issuecomment-1893691865
PR Review Comment: https://git.openjdk.org/jdk/pull/17399#discussion_r1453387790


More information about the build-dev mailing list