Looks good to me. /Magnus
> 8 mars 2018 kl. 18:46 skrev Erik Joelsson <[email protected]>: > > The change in JDK-8198859 unfortunately broke solaris x64. While we (at > Oracle) aren't building this platform regularly anymore, we still have a > weird dependency on it so need to keep it building for a while more. > > The simplest fix is to only do the elfedit on sparc. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8199339 > > Patch: > > diff -r 28729cf7e4bc make/hotspot/lib/CompileDtracePostJvm.gmk > --- a/make/hotspot/lib/CompileDtracePostJvm.gmk > +++ b/make/hotspot/lib/CompileDtracePostJvm.gmk > @@ -160,7 +160,9 @@ > > $(DTRACE_SUPPORT_DIR)/$(@F).d)) > $(call ExecuteWithLog, $@, $(DTRACE) $(DTRACE_FLAGS) -o $@ \ > -s $(DTRACE_SUPPORT_DIR)/$(@F).d) > - $(call ExecuteWithLog, [email protected], $(ELFEDIT) $(call GetElfeditCommands) > $@) > + ifeq ($(OPENJDK_TARGET_CPU_ARCH), sparc) > + $(call ExecuteWithLog, [email protected], $(ELFEDIT) $(call > GetElfeditCommands) $@) > + endif > > ############################################################################ > # Build the stand-alone dtrace libraries > > > /Erik >
