Looks good to me.
/Erik
On 2015-11-10 03:56, Ioi Lam wrote:
Please review a very very small fix:
http://cr.openjdk.java.net/~iklam/8080940-remove-solaris-mcs/
Bug: 8080940 Remove mcs post-hook from hotspot solaris builds
https://bugs.openjdk.java.net/browse/JDK-8080940
Summary of fix:
I am getting a heart attack every time I look at the
Solaris build log and it tells me libjvm.so has failed
when it hasn't. So here's a patch.
The call to mcs has been removed from LINK_LIB.CXX/POST_HOOK
According to Magnus's analysis in the bug report, the removal does
not cause any issue.
I verified the word "mcs" no longer appear in the SPARC build logs,
and there's no more spurious report of libjvm.so failure.
Tests:
JPRT with all solaris targets.:
Build Stats: 5 pass, 0 fail, 0 killed, 0 working, 0
initializing, 0 not started
Test Stats: 57 pass, 0 fail, 0 killed, 0 working, 0
initializing, 0 not started
Thanks
- Ioi
