Re: strange error when running jtreg tests

2016-08-17 Thread Gustavo Romero
Hi David, On 17-08-2016 00:38, David Holmes wrote: > On 16/08/2016 8:41 AM, Gustavo Romero wrote: >> On Ubuntu 16.04 PPC64 LE replacing -lpthread by -pthread in >> ./hotspot/make/test/JtregNative.gmk:82 solves the issue on pthread_* symbols >> and >> placing -ljvm (for instance) in front of ".../

Re: strange error when running jtreg tests

2016-08-17 Thread Gustavo Romero
Sorry, again: (reading from left to right the link command). Regards, Gustavo On 17-08-2016 11:15, Gustavo Romero wrote: > Hi David, > > On 17-08-2016 00:38, David Holmes wrote: >> On 16/08/2016 8:41 AM, Gustavo Romero wrote: >>> On Ubuntu 16.04 PPC64 LE replacing -lpthread by -pthread in >>> ./

Re: strange error when running jtreg tests

2016-08-17 Thread Erik Joelsson
I finally took a look at this and yes, there is a bug in the makefile. This line in hotspot/make/test/JtregNative.gmk: BUILD_HOTSPOT_JTREG_EXECUTABLES_LDFLAGS_exeinvoke := -ljvm -lpthread should be: BUILD_HOTSPOT_JTREG_EXECUTABLES_LIBS_exeinvoke := -ljvm -lpthread Could someone who is able to

Re: strange error when running jtreg tests

2016-08-17 Thread Gustavo Romero
Hi Erik, I applied your change: diff -r 397565766eb4 make/test/JtregNative.gmk --- a/make/test/JtregNative.gmk Thu Aug 11 16:22:08 2016 -0700 +++ b/make/test/JtregNative.gmk Wed Aug 17 09:54:20 2016 -0500 @@ -79,7 +79,7 @@ ifeq ($(OPENJDK_TARGET_OS), linux) BUILD_HOTSPOT_JTREG_LIBRARIES_LDF

Re: strange error when running jtreg tests

2016-08-17 Thread Erik Joelsson
I'm sorry, you also need this patch, which I for some reason I can't remember had in my local forest. diff -r a24702d4d5ab make/common/TestFilesCompilation.gmk --- a/make/common/TestFilesCompilation.gmk +++ b/make/common/TestFilesCompilation.gmk @@ -86,6 +86,7 @@ LANG := C, \ C

RFR: 8164071: Default.policy file missing content for solaris

2016-08-17 Thread Sean Mullan
Please review this simple fix to append the solaris default.policy file to the overall default.policy file. Bug: https://bugs.openjdk.java.net/browse/JDK-8164071 diff -r 551f7617b2c0 make/copy/Copy-java.base.gmk --- a/make/copy/Copy-java.base.gmk Wed Aug 17 10:08:18 2016 +0800 +++ b/make/c

Re: RFR: 8164071: Default.policy file missing content for solaris

2016-08-17 Thread Erik Joelsson
Hello Sean, The change looks ok, but it could also be expressed like this to avoid duplication: ifneq ($(filter $(OPENJDK_TARGET_OS), windows solaris), ) DEF_POLICY_SRC_LIST += $(JDK_TOPDIR)/src/java.base/$(OPENJDK_TARGET_OS)/lib/security/default.policy endif /Erik On 2016-08-17 18:18, S

Re: RFR: 8164071: Default.policy file missing content for solaris

2016-08-17 Thread Sean Mullan
On 8/17/2016 12:33 PM, Erik Joelsson wrote: Hello Sean, The change looks ok, but it could also be expressed like this to avoid duplication: ifneq ($(filter $(OPENJDK_TARGET_OS), windows solaris), ) DEF_POLICY_SRC_LIST += $(JDK_TOPDIR)/src/java.base/$(OPENJDK_TARGET_OS)/lib/security/defaul

Re: RFR: 8164071: Default.policy file missing content for solaris

2016-08-17 Thread Erik Joelsson
On 2016-08-17 18:43, Sean Mullan wrote: On 8/17/2016 12:33 PM, Erik Joelsson wrote: Hello Sean, The change looks ok, but it could also be expressed like this to avoid duplication: ifneq ($(filter $(OPENJDK_TARGET_OS), windows solaris), ) DEF_POLICY_SRC_LIST += $(JDK_TOPDIR)/src/java.base

Re: strange error when running jtreg tests

2016-08-17 Thread Gustavo Romero
Hi Erik, On 17-08-2016 12:23, Erik Joelsson wrote: > I'm sorry, you also need this patch, which I for some reason I can't remember > had in my local forest. > > diff -r a24702d4d5ab make/common/TestFilesCompilation.gmk > --- a/make/common/TestFilesCompilation.gmk > +++ b/make/common/TestFilesCom

Re: strange error when running jtreg tests

2016-08-17 Thread Andrew Hughes
- Original Message - > Hi Erik, > > On 17-08-2016 12:23, Erik Joelsson wrote: > > I'm sorry, you also need this patch, which I for some reason I can't > > remember had in my local forest. > > > > diff -r a24702d4d5ab make/common/TestFilesCompilation.gmk > > --- a/make/common/TestFilesCo

Fwd: RFR: 8061842: Package jurisdiction policy files as something other than JAR

2016-08-17 Thread Bradford Wetmore
Forwarding from security-dev to build-dev, as there is a change proposed for the way the cryptographic policy files are built. https://bugs.openjdk.java.net/browse/JDK-8061842 http://cr.openjdk.java.net/~wetmore/8061842/webrev.01/ The proposal is to move the configuration files from the jar fi