RFR: JDK-8199606 Set -lc as global LIBS on solstudio
Erik Joelsson
erik.joelsson at oracle.com
Wed Mar 14 14:43:40 UTC 2018
Hello,
Nice to see this cleaned up!
The differences you mention were the reason I meticulously kept all the
-lc in the exact same places as the old build did way back when we
rewrote the build system. I highly doubt they matter, but they needed to
be eliminated for a clean switch of build systems. This change should
however be fine as it's controlled and we know what we are changing.
libraries.m4: 153: This line is now redundant.
Otherwise this looks good.
/Erik
On 2018-03-14 04:17, Magnus Ihse Bursie wrote:
> We should always use -lc when linking with solstudio. All other
> toolchains do this automatically. This also means that -lc is
> unnecessary for other platforms, where it has been set in some cases
> (due to the old confusion where "solaris" really meant "unix").
>
> I have tested this using COMPARE_BUILD. Unsurprisingly, there are
> differences, but they are expected. On macosx and linux, libunpack has
> disassembly differences (they had previously an explicit -lc). On
> linux, the same also goes for libjli, libawt_xawt, libfontmanager and
> libsunec.
>
> On solaris there are even more differences. Now all libraries get
> explicitely linked with -lc, which means binary/disassembly
> differences in multiple libraries. We also get a deps difference for
> libjsig and librmi, which were previously compiled (successfully, for
> a change! :-)) without -lc. Finally, we get a symbol difference for
> libjava.We're missing symbols like: "a s_finite.c" and "t finite", and
> we have one less duplicate of "r two54". I can't say I understand why,
> but it does seem harmless.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8199606
> WebRev:
> http://cr.openjdk.java.net/~ihse/JDK-8199606-set-lc-as-GLOBAL_LIBS/webrev.01
>
> /Magnus
More information about the build-dev
mailing list