hg: jigsaw/jake/jdk: Minor clean up on StackWalker::getCallerClass spec

[mandy . chung]

Changeset: cbf1e85f7c11
Author:mchung
Date:  2017-01-20 19:58 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/cbf1e85f7c11

Minor clean up on StackWalker::getCallerClass spec

! src/java.base/share/classes/java/lang/StackWalker.java

Re: Review Request: JDK-8173096 jmod files are not world-readable

[Mandy Chung]

Updated webrev to put the temporary file in the same containing directory of 
the target file:
  http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8173096/webrev.01

I want to fix this regression in jdk-9+154.  We can revisit the approach after 
this fix if necessary.

Mandy

> On Jan 20, 2017, at 8:55 AM, Mandy Chung  wrote:
> 
> Webrev:
>   http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8173096/webrev.00/index.html
> 
> A recent change in jmod tool [1] creates the JMOD file with no group
> and other readable permission as that’s the default permission when
> creating a temporary file with Files::createTempFile.  This fixes
> the permission issue by creating the JMOD file in a temporary directory.
> 
> Mandy
> [1] https://bugs.openjdk.java.net/browse/JDK-8173096

What is the purpose of --inherit-runtime-environment

[Nicolai Parlog]

 Hi!

Pretty much what the subject line says... I don't quite get what exactly
--inherit-runtime-environment does and what it is good for. Is this
something that non-JDK developers are likely to use?

 so long ... Nicolai



-- 

PGP Key:
http://keys.gnupg.net/pks/lookup?op=vindex=0xCA3BAD2E9CCCD509

Web:
http://codefx.org
a blog about software development
https://www.sitepoint.com/java
high-quality Java/JVM content
http://do-foss.de
Free and Open Source Software for the City of Dortmund

Twitter:
https://twitter.com/nipafx

Re: RFR 8172527: Rename jdk.crypto.token to jdk.crypto.cryptoki

[Mandy Chung]

> On Jan 20, 2017, at 10:22 AM, Anthony Scarpino  
> wrote:
> 
> Good catch.. that'll teach me for trusting the graphical tool to rename a 
> directory when I used 'Rename'.
> 
> Also I found Brad's issue as it was a new changeset that just showed up in 
> that file.
> 
> I updated the webrev, which looks better now:
> http://cr.openjdk.java.net/~ascarpino/8172527/webrev-jdk.01/


+1

Mandy

Re: Review Request: JDK-8173096 jmod files are not world-readable

[Mandy Chung]

> On Jan 20, 2017, at 1:22 PM, Alan Bateman  wrote:
> 
> Mandy - a while back then Erik changed the build so that JMOD files are 
> created in JMODS_TEMPDIR rather than JMODS_DIR. I think this was to reduce 
> interference with concurrent execution of the `jmod` tool. This might mean 
> you can avoid using /tmp.

The issue is jmod —-hash-modules resolves a module graph from a given module 
path which is the containing directory of the JMOD file being created:

$ jmod create -—module-path dir —-hash-modules “.*” dir/m.jmod

It seems one solution is to have the temporary file name be a hidden file.
Mandy

Re: Review Request: JDK-8173096 jmod files are not world-readable

[Alan Bateman]

On 20/01/2017 20:37, Peter Levart wrote:



I guess the JmodTask is creating a temp file because it wants the file 
to not be observed by other processes while it is being written to, 
right? At the end when the tempTarget is fully written, it is renamed 
to its intended name with:


Files.move(tempTarget, target);


...but that operation will not be atomic since you didn't pass 
ATOMIC_MOVE to the CopyOption... options. Even if you did, the atomic 
move will succeed only when both tempTarget and target point to the 
same filesystem. Since you are creating tempTarget in default 
temporary files location (/tmp on UNIXes) this might not be the case 
(depending on where the real target is located).


So why don't you simply append a .tmp suffix to the intended target 
file name when opening the JmodOutputStream. This would ensure the 
file location will be on the same filesystem as the final target name 
as it will be created in the same directory. Module path scanning 
should skip files with .tmp extension then.
Module path scanning doesn't skip unrecognized files, this is because it 
allows for implementation specific packaging formats.


Mandy - a while back then Erik changed the build so that JMOD files are 
created in JMODS_TEMPDIR rather than JMODS_DIR. I think this was to 
reduce interference with concurrent execution of the `jmod` tool. This 
might mean you can avoid using /tmp.


-Alan.

Re: Review Request: JDK-8173096 jmod files are not world-readable

[Mandy Chung]

> On Jan 20, 2017, at 12:37 PM, Peter Levart  wrote:
> 
> I guess the JmodTask is creating a temp file because it wants the file to not 
> be observed by other processes while it is being written to, right? At the 
> end when the tempTarget is fully written, it is renamed to its intended name 
> with:
> 
> Files.move(tempTarget, target);
> 
> 
> ...but that operation will not be atomic since you didn't pass ATOMIC_MOVE to 
> the CopyOption... options. Even if you did, the atomic move will succeed only 
> when both tempTarget and target point to the same filesystem. Since you are 
> creating tempTarget in default temporary files location (/tmp on UNIXes) this 
> might not be the case (depending on where the real target is located).
> 
> So why don't you simply append a .tmp suffix to the intended target file name 
> when opening the JmodOutputStream. This would ensure the file location will 
> be on the same filesystem as the final target name as it will be created in 
> the same directory. Module path scanning should skip files with .tmp 
> extension then.

Module path only ignores files starts with “.” or is hidden.   This says making 
 the temp file starting with “.” is a possibility.

$ java -p mods -m hi
Error occurred during initialization of VM
java.lang.module.ResolutionException: Unrecognized module: mods/abc.tmp

Mandy

Re: Review Request: JDK-8173096 jmod files are not world-readable

[Peter Levart]

Hi Mandy,

On 01/20/2017 05:55 PM, Mandy Chung wrote:

Webrev:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8173096/webrev.00/index.html

A recent change in jmod tool [1] creates the JMOD file with no group
and other readable permission as that’s the default permission when
creating a temporary file with Files::createTempFile.  This fixes
the permission issue by creating the JMOD file in a temporary directory.

Mandy
[1] https://bugs.openjdk.java.net/browse/JDK-8173096


I guess the JmodTask is creating a temp file because it wants the file 
to not be observed by other processes while it is being written to, 
right? At the end when the tempTarget is fully written, it is renamed to 
its intended name with:


Files.move(tempTarget, target);


...but that operation will not be atomic since you didn't pass 
ATOMIC_MOVE to the CopyOption... options. Even if you did, the atomic 
move will succeed only when both tempTarget and target point to the same 
filesystem. Since you are creating tempTarget in default temporary files 
location (/tmp on UNIXes) this might not be the case (depending on where 
the real target is located).


So why don't you simply append a .tmp suffix to the intended target file 
name when opening the JmodOutputStream. This would ensure the file 
location will be on the same filesystem as the final target name as it 
will be created in the same directory. Module path scanning should skip 
files with .tmp extension then.


Regards, Peter

Re: RFR 8172527: Rename jdk.crypto.token to jdk.crypto.cryptoki

[Anthony Scarpino]

On 01/19/2017 11:50 AM, Mandy Chung wrote:



On Jan 19, 2017, at 11:39 AM, Anthony Scarpino  
wrote:

Hi,

I need a review to rename the jdk.crypto.token to jdk.crypto.cryptoki. This is to change 
what 8171202 had done to the original jdk.crypto.pkcs11 module.  For those not familiar 
with discussions elsewhere, the term "token" is confusing and unclear as it can 
mean many things cryptographically.  The use of cryptoki is more accurate to the original 
'pkcs11' name and still conforming to the new naming standard.

http://cr.openjdk.java.net/~ascarpino/8172527/webrev-root/
http://cr.openjdk.java.net/~ascarpino/8172527/webrev-jdk/


The webrev shows deleted/new files for src/jdk.crypto.token source files rather 
than rename.  Did you use hg rename?

Changes in other files looks good.

Mandy



Good catch.. that'll teach me for trusting the graphical tool to rename 
a directory when I used 'Rename'.


Also I found Brad's issue as it was a new changeset that just showed up 
in that file.


I updated the webrev, which looks better now:
http://cr.openjdk.java.net/~ascarpino/8172527/webrev-jdk.01/

Tony

Review Request: JDK-8173096 jmod files are not world-readable

[Mandy Chung]

Webrev:
   http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8173096/webrev.00/index.html

A recent change in jmod tool [1] creates the JMOD file with no group
and other readable permission as that’s the default permission when
creating a temporary file with Files::createTempFile.  This fixes
the permission issue by creating the JMOD file in a temporary directory.

Mandy
[1] https://bugs.openjdk.java.net/browse/JDK-8173096

Re: RFR 8172659: PluginException("TargetPlatform attribute is missing ...") - should be ModuleTarget

[Mandy Chung]

+1

Mandy

> On Jan 20, 2017, at 1:06 AM, Sundararajan Athijegannathan 
>  wrote:
> 
> Please review http://cr.openjdk.java.net/~sundar/8172659/webrev.00/ for 
> https://bugs.openjdk.java.net/browse/JDK-8172659
> 
> Thanks,
> -Sundar

Re: RFR 8172659: PluginException("TargetPlatform attribute is missing ...") - should be ModuleTarget

[Jim Laskey (Oracle)]

+1

> On Jan 20, 2017, at 5:06 AM, Sundararajan Athijegannathan 
>  wrote:
> 
> Please review http://cr.openjdk.java.net/~sundar/8172659/webrev.00/ for 
> https://bugs.openjdk.java.net/browse/JDK-8172659
> 
> Thanks,
> -Sundar

hg: jigsaw/jake/langtools: 21 new changesets

[alan . bateman]

Changeset: 5ec5a14e1627
Author:jlahoda
Date:  2017-01-09 16:44 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/5ec5a14e1627

8172411: -XDnoModules must be removed
Reviewed-by: jjg

! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symtab.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Resolve.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/main/JavaCompiler.java
! 
src/jdk.compiler/share/classes/com/sun/tools/javac/processing/JavacProcessingEnvironment.java

Changeset: ee620ff60730
Author:rfield
Date:  2017-01-09 13:50 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/ee620ff60730

8172255: JShell API: ExecutionControl/LoaderDelegate: Remove 
unused/unimplemented setClassPath
Reviewed-by: jlahoda

! src/jdk.jshell/share/classes/jdk/jshell/execution/DefaultLoaderDelegate.java
! src/jdk.jshell/share/classes/jdk/jshell/execution/DirectExecutionControl.java
! 
src/jdk.jshell/share/classes/jdk/jshell/execution/ExecutionControlForwarder.java
! src/jdk.jshell/share/classes/jdk/jshell/execution/LoaderDelegate.java
! src/jdk.jshell/share/classes/jdk/jshell/execution/RemoteCodes.java
! 
src/jdk.jshell/share/classes/jdk/jshell/execution/StreamingExecutionControl.java
! src/jdk.jshell/share/classes/jdk/jshell/spi/ExecutionControl.java
! test/jdk/jshell/KullaTesting.java

Changeset: f5dc63258a43
Author:darcy
Date:  2017-01-09 16:20 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/f5dc63258a43

8172458: Make javax.lang.model javadoc HTML 5 compliant
Reviewed-by: jjg

! src/java.compiler/share/classes/javax/annotation/processing/Processor.java
! 
src/java.compiler/share/classes/javax/lang/model/element/AnnotationValueVisitor.java
! src/java.compiler/share/classes/javax/lang/model/element/ElementVisitor.java
! src/java.compiler/share/classes/javax/lang/model/type/TypeVisitor.java
! src/java.compiler/share/classes/javax/lang/model/type/WildcardType.java
! src/java.compiler/share/classes/javax/lang/model/util/ElementKindVisitor6.java
! src/java.compiler/share/classes/javax/lang/model/util/ElementKindVisitor7.java
! src/java.compiler/share/classes/javax/lang/model/util/ElementKindVisitor8.java
! src/java.compiler/share/classes/javax/lang/model/util/ElementKindVisitor9.java
! src/java.compiler/share/classes/javax/lang/model/util/ElementScanner6.java
! src/java.compiler/share/classes/javax/lang/model/util/ElementScanner7.java
! src/java.compiler/share/classes/javax/lang/model/util/ElementScanner8.java
! src/java.compiler/share/classes/javax/lang/model/util/ElementScanner9.java
! src/java.compiler/share/classes/javax/lang/model/util/TypeKindVisitor6.java
! src/java.compiler/share/classes/javax/lang/model/util/TypeKindVisitor7.java
! src/java.compiler/share/classes/javax/lang/model/util/TypeKindVisitor8.java
! src/java.compiler/share/classes/javax/lang/model/util/TypeKindVisitor9.java

Changeset: 9ed8e9a27b00
Author:rfield
Date:  2017-01-09 18:04 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/9ed8e9a27b00

8171981: JShell: Fails compilation: new Object().getClass().getSuperclass()
Reviewed-by: mcimadamore

! 
src/jdk.jshell/share/classes/jdk/internal/jshell/debug/InternalDebugControl.java
! src/jdk.jshell/share/classes/jdk/internal/jshell/tool/JShellTool.java
! src/jdk.jshell/share/classes/jdk/jshell/Eval.java
+ src/jdk.jshell/share/classes/jdk/jshell/ExpressionToTypeInfo.java
! src/jdk.jshell/share/classes/jdk/jshell/SourceCodeAnalysisImpl.java
! src/jdk.jshell/share/classes/jdk/jshell/TaskFactory.java
! src/jdk.jshell/share/classes/jdk/jshell/TreeDissector.java
! src/jdk.jshell/share/classes/jdk/jshell/TypePrinter.java
! src/jdk.jshell/share/classes/jdk/jshell/Unit.java
+ src/jdk.jshell/share/classes/jdk/jshell/VarTypePrinter.java
! test/jdk/jshell/TypeNameTest.java
! test/jdk/jshell/VariablesTest.java

Changeset: f420d8eea54f
Author:jlahoda
Date:  2017-01-10 10:17 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/f420d8eea54f

8171385: jshell tool: unresponsive to ctrl-C in input wait on Windows
Summary: Ensuring stop is sent using the proper channel depending on the 
current state of the StopDetectingInputStream.
Reviewed-by: rfield

! 
src/jdk.jshell/share/classes/jdk/internal/jshell/tool/StopDetectingInputStream.java
! test/jdk/jshell/StopExecutionTest.java

Changeset: 6891ece2edae
Author:jlahoda
Date:  2017-01-10 13:53 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/6891ece2edae

8172414: jshell not working in exploded JDK build
Summary: Using Class.getResourceAsStream instead of a direct reference to JRT.
Reviewed-by: rfield, sundar

! src/jdk.jshell/share/classes/jdk/internal/jshell/tool/JShellTool.java

Changeset: 97a60778fc6a
Author:vromero
Date:  2017-01-10 11:31 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/97a60778fc6a

8171332: NPE in MembersPhase.finishClass
Reviewed-by: 

hg: jigsaw/jake/jdk: 52 new changesets

[alan . bateman]

Changeset: d93e93939378
Author:mli
Date:  2017-01-09 14:14 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/d93e93939378

8172347: Refactoring 
src/java.rmi/share/classes/sun/rmi/registry/RegistryImpl.java to improve 
testability of rmiregistry
Reviewed-by: rriggs, msheppar

! src/java.rmi/share/classes/sun/rmi/registry/RegistryImpl.java

Changeset: 44454471f7e7
Author:darcy
Date:  2017-01-09 21:40 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/44454471f7e7

8172475: Remove  usage from Class and ClassLoader
Reviewed-by: bpb, mchung

! src/java.base/share/classes/java/lang/Class.java
! src/java.base/share/classes/java/lang/ClassLoader.java

Changeset: 766677da17b5
Author:aefimov
Date:  2017-01-10 22:17 +0300
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/766677da17b5

8159058: SAXParseException when sending soap message
Reviewed-by: lancea

+ test/javax/xml/ws/8159058/SaajEmptyNamespaceTest.java

Changeset: 2645e2102d3c
Author:skovalev
Date:  2017-01-11 13:10 +0300
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/2645e2102d3c

8171958: Several tests from java/time/test/java/time/format requiring 
jdk.localedata for execution
Reviewed-by: naoto

! test/java/time/TEST.properties
! test/java/time/test/java/time/format/TestDateTimeFormatterBuilder.java
+ 
test/java/time/test/java/time/format/TestDateTimeFormatterBuilderWithLocale.java
! test/java/time/test/java/time/format/TestDateTimeTextProvider.java
+ test/java/time/test/java/time/format/TestDateTimeTextProviderWithLocale.java
! test/java/time/test/java/time/format/TestNarrowMonthNamesAndDayNames.java
! test/java/time/test/java/time/format/TestNonIsoFormatter.java
! test/java/time/test/java/time/format/TestTextParser.java
+ test/java/time/test/java/time/format/TestTextParserWithLocale.java
! test/java/time/test/java/time/format/TestTextPrinter.java
+ test/java/time/test/java/time/format/TestTextPrinterWithLocale.java

Changeset: 16af3df77839
Author:xiaofeya
Date:  2017-01-11 08:53 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/16af3df77839

8075884: check runtime usage tests with multi-release jar files
Reviewed-by: psandoz

+ test/tools/jar/multiRelease/RuntimeTest.java
+ test/tools/jar/multiRelease/data/runtimetest/MVJarJJSTestScript.js
+ test/tools/jar/multiRelease/data/runtimetest/base/testpackage/Helper.java
+ test/tools/jar/multiRelease/data/runtimetest/base/testpackage/Main.java
+ test/tools/jar/multiRelease/data/runtimetest/base/versionResource
+ test/tools/jar/multiRelease/data/runtimetest/manifest.txt
+ test/tools/jar/multiRelease/data/runtimetest/test/UseByImport.java
+ test/tools/jar/multiRelease/data/runtimetest/test/UseByReflection.java
+ test/tools/jar/multiRelease/data/runtimetest/v10/testpackage/Helper.java
+ test/tools/jar/multiRelease/data/runtimetest/v10/testpackage/Main.java
+ test/tools/jar/multiRelease/data/runtimetest/v10/versionResource
+ test/tools/jar/multiRelease/data/runtimetest/v9/testpackage/Helper.java
+ test/tools/jar/multiRelease/data/runtimetest/v9/testpackage/Main.java
+ test/tools/jar/multiRelease/data/runtimetest/v9/versionResource

Changeset: dbd8c17b486f
Author:mli
Date:  2017-01-11 19:36 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/dbd8c17b486f

8172314: java/rmi/registry/altSecurityManager/AltSecurityManager.java fails 
with "port in use"
Reviewed-by: rriggs

! test/java/rmi/registry/altSecurityManager/AltSecurityManager.java
! test/java/rmi/registry/altSecurityManager/TestSecurityManager.java
+ test/java/rmi/registry/altSecurityManager/registry.security.policy
+ test/java/rmi/registry/altSecurityManager/rmid.security.policy
! test/java/rmi/registry/reexport/Reexport.java
! test/java/rmi/testlibrary/JavaVM.java
- test/java/rmi/testlibrary/REGISTRY.java
! test/java/rmi/testlibrary/RMID.java
+ test/java/rmi/testlibrary/RMIRegistryRunner.java
! test/java/rmi/testlibrary/RegistryRunner.java
+ test/java/rmi/testlibrary/RegistryVM.java
! test/java/rmi/testlibrary/TestParams.java
! test/java/rmi/transport/dgcDeadLock/DGCDeadLock.java
+ test/java/rmi/transport/dgcDeadLock/registry.security.policy
! test/sun/rmi/transport/tcp/DeadCachedConnection.java

Changeset: c82c693bc2a5
Author:chegar
Date:  2017-01-12 11:41 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/c82c693bc2a5

8168149: Examine the behavior of jmod command-line options - repeating vs last 
one wins
Reviewed-by: mchung

! src/jdk.jlink/share/classes/jdk/tools/jmod/JmodTask.java
! test/tools/jmod/JmodTest.java

Changeset: 59e5101730c8
Author:redestad
Date:  2017-01-12 13:38 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/59e5101730c8

8166365: Small immutable collections should provide optimized implementations 
when possible
Reviewed-by: smarks, psandoz, attila

! src/java.base/share/classes/java/util/Collections.java
! src/java.base/share/classes/java/util/ImmutableCollections.java

hg: jigsaw/jake/nashorn: 6 new changesets

[alan . bateman]

Changeset: 6b43eb787db4
Author:sundar
Date:  2017-01-10 18:50 +0530
URL:   http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/6b43eb787db4

8172493: Nashorn FX example 3-4 using load for fx: scripts fails to run with 
latest jdk9 ea build
Reviewed-by: hannesw, jlaskey

! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/resources/fx/base.js

Changeset: db9349bc7035
Author:hannesw
Date:  2017-01-10 15:27 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/db9349bc7035

8170781: PropertyMapIterator throws NoSuchElementException on last element
Reviewed-by: sundar, attila

! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/PropertyMap.java
+ test/src/jdk/nashorn/internal/runtime/test/PropertyMapTest.java

Changeset: 4a07ebdf8b45
Author:hannesw
Date:  2017-01-10 17:52 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/4a07ebdf8b45

8166187: Regression: NPE during reparse when using persistent code cache and 
optimistic types
Reviewed-by: jlaskey, attila

! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/RecompilableScriptFunctionData.java
! test/src/jdk/nashorn/internal/runtime/test/CodeStoreAndPathTest.java

Changeset: 19aaaf2d02b7
Author:lana
Date:  2017-01-13 01:36 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/19aaaf2d02b7

Merge


Changeset: a84b49cfee63
Author:lana
Date:  2017-01-19 22:31 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/a84b49cfee63

Added tag jdk-9+153 for changeset 19aaaf2d02b7

! .hgtags

Changeset: f516869de5e6
Author:alanb
Date:  2017-01-20 11:39 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/f516869de5e6

Merge

! .hgtags
! test/src/jdk/nashorn/internal/runtime/test/CodeStoreAndPathTest.java

hg: jigsaw/jake/corba: 2 new changesets

[alan . bateman]

Changeset: d66f97a610a6
Author:lana
Date:  2017-01-19 22:31 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/corba/rev/d66f97a610a6

Added tag jdk-9+153 for changeset 68a8e8658511

! .hgtags

Changeset: 15bf1182a36c
Author:alanb
Date:  2017-01-20 11:45 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/corba/rev/15bf1182a36c

Merge

hg: jigsaw/jake: 17 new changesets

[alan . bateman]

Changeset: 4650659a4f55
Author:erikj
Date:  2017-01-10 11:23 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/4650659a4f55

8172241: Cleanup mistakes in jib publish support change
Reviewed-by: tbell

! common/autoconf/spec.gmk.in
! common/conf/jib-profiles.js
! make/Bundles.gmk
! make/Images.gmk
! make/Main.gmk

Changeset: df239221d512
Author:erikj
Date:  2017-01-10 11:28 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/df239221d512

8171932: unresolved macro in javadoc command
Reviewed-by: tbell

! make/Javadoc.gmk
! make/common/MakeBase.gmk

Changeset: 8cddd16d59ee
Author:erikj
Date:  2017-01-10 11:31 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/8cddd16d59ee

8170862: VarDeps breaks when a file with overridden CFLAGS has the same name as 
the library
Reviewed-by: tbell

! make/common/NativeCompilation.gmk

Changeset: 47f2a1a3bc8c
Author:ihse
Date:  2017-01-11 13:00 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/47f2a1a3bc8c

8171409: Create a smoother configure experience on macosx
Reviewed-by: erikj

! common/autoconf/basics.m4
! common/autoconf/generated-configure.sh
! common/autoconf/help.m4
! common/autoconf/jdk-options.m4
! common/autoconf/lib-freetype.m4

Changeset: fc80aba4ce4b
Author:ihse
Date:  2017-01-12 10:08 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/fc80aba4ce4b

8172562: Changing log level on Javadoc causes total rebuild
Reviewed-by: erikj, tbell

! make/Javadoc.gmk

Changeset: f277629427ce
Author:ihse
Date:  2017-01-12 10:30 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/f277629427ce

8172702: Remove left-over OPENJDK_TARGET_CPU_JLI_CFLAGS
Reviewed-by: erikj

! common/autoconf/buildjdk-spec.gmk.in

Changeset: 037736440a67
Author:sla
Date:  2017-01-12 14:26 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/037736440a67

8172709: Upgrade to jtreg 4.2 b05
Reviewed-by: erikj, chegar

! common/conf/jib-profiles.js

Changeset: 0b6178060b7f
Author:stuefe
Date:  2017-01-12 14:38 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/0b6178060b7f

8172712: configure should check that grep handles empty pattern correctly
Reviewed-by: ihse

! common/autoconf/basics.m4
! common/autoconf/generated-configure.sh

Changeset: 9ea35cb972ff
Author:erikj
Date:  2017-01-12 14:47 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/9ea35cb972ff

8172577: Builds for OS X after build 149 does not include Java Mission 
Control.app
Reviewed-by: tbell, ihse

! make/Bundles.gmk

Changeset: 905aa8122624
Author:amjiang
Date:  2017-01-12 17:15 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/905aa8122624

8171423: Relocate /test/lib/security/SecurityTools.java
Reviewed-by: weijun, xuelei

+ test/lib/jdk/test/lib/SecurityTools.java

Changeset: c7dd5a9e187e
Author:lana
Date:  2017-01-13 01:35 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/c7dd5a9e187e

Merge


Changeset: 2f21a36a4cc6
Author:sherman
Date:  2017-01-12 22:55 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/2f21a36a4cc6

8172432: jar cleanup/update for module and mrm jar
Reviewed-by: mchung, psandoz, chegar

! make/CompileJavaModules.gmk

Changeset: dbf8aa5aea8d
Author:sla
Date:  2017-01-16 12:32 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/dbf8aa5aea8d

8172842: Invoke lldb with --batch from failure handler
Reviewed-by: erikj, dsamersoff

! test/failure_handler/src/share/conf/mac.properties

Changeset: 5256cc3b35c8
Author:ihse
Date:  2017-01-16 13:50 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/5256cc3b35c8

8172714: Remove unused and unexpanded variables from spec.gmk.in
Reviewed-by: erikj, tbell

! common/autoconf/spec.gmk.in

Changeset: 816a6d03a7c4
Author:ihse
Date:  2017-01-16 13:53 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/816a6d03a7c4

8162750: -D__solaris__ added twice
Reviewed-by: erikj

! common/autoconf/flags.m4
! common/autoconf/generated-configure.sh

Changeset: 8d22611ffb65
Author:lana
Date:  2017-01-19 22:31 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/8d22611ffb65

Added tag jdk-9+153 for changeset 816a6d03a7c4

! .hgtags

Changeset: da598f6fda9e
Author:alanb
Date:  2017-01-20 11:37 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/da598f6fda9e

Merge

! common/autoconf/basics.m4
! common/autoconf/buildjdk-spec.gmk.in
! common/autoconf/flags.m4
! common/autoconf/generated-configure.sh
! common/autoconf/jdk-options.m4
! common/autoconf/spec.gmk.in
! common/conf/jib-profiles.js
! make/CompileJavaModules.gmk
! make/Images.gmk
! make/Javadoc.gmk
! make/Main.gmk
! make/common/MakeBase.gmk
! make/common/NativeCompilation.gmk

hg: jigsaw/jake/jaxws: 5 new changesets

[alan . bateman]

Changeset: a98174edd246
Author:aefimov
Date:  2017-01-10 22:15 +0300
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/a98174edd246

8159058: SAXParseException when sending soap message
Reviewed-by: lancea

! 
src/java.xml.ws/share/classes/com/sun/xml/internal/messaging/saaj/soap/impl/DetailImpl.java
! 
src/java.xml.ws/share/classes/com/sun/xml/internal/messaging/saaj/soap/impl/ElementImpl.java
! 
src/java.xml.ws/share/classes/com/sun/xml/internal/messaging/saaj/util/stax/SaajStaxWriter.java
! 
src/java.xml.ws/share/classes/com/sun/xml/internal/ws/api/message/saaj/SaajStaxWriter.java

Changeset: 92aa05eff5d1
Author:joehw
Date:  2017-01-11 13:08 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/92aa05eff5d1

8171243: CatalogManager.catalogResolver throws FileSystemNotFoundException with 
jar
Reviewed-by: rriggs, dfuchs, lancea, alanb

! src/java.xml.ws/share/classes/com/sun/xml/internal/ws/util/xml/XmlUtil.java
! src/jdk.xml.bind/share/classes/com/sun/tools/internal/xjc/Options.java

Changeset: 7a532a9a2271
Author:lana
Date:  2017-01-13 01:35 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/7a532a9a2271

Merge


Changeset: 34af95c7dbff
Author:lana
Date:  2017-01-19 22:31 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/34af95c7dbff

Added tag jdk-9+153 for changeset 7a532a9a2271

! .hgtags

Changeset: 911660ba3a12
Author:alanb
Date:  2017-01-20 11:38 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/911660ba3a12

Merge

hg: jigsaw/jake/jaxp: 5 new changesets

[alan . bateman]

Changeset: 8c9a2a24752b
Author:joehw
Date:  2017-01-11 13:06 -0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/8c9a2a24752b

8171243: CatalogManager.catalogResolver throws FileSystemNotFoundException with 
jar
Reviewed-by: rriggs, dfuchs, lancea, alanb

! src/java.xml/share/classes/javax/xml/catalog/AltCatalog.java
! src/java.xml/share/classes/javax/xml/catalog/BaseEntry.java
! src/java.xml/share/classes/javax/xml/catalog/Catalog.java
! src/java.xml/share/classes/javax/xml/catalog/CatalogFeatures.java
! src/java.xml/share/classes/javax/xml/catalog/CatalogImpl.java
! src/java.xml/share/classes/javax/xml/catalog/CatalogManager.java
! src/java.xml/share/classes/javax/xml/catalog/CatalogMessages.java
! src/java.xml/share/classes/javax/xml/catalog/CatalogMessages.properties
! src/java.xml/share/classes/javax/xml/catalog/CatalogReader.java
! src/java.xml/share/classes/javax/xml/catalog/CatalogResolver.java
! src/java.xml/share/classes/javax/xml/catalog/CatalogResolverImpl.java
! src/java.xml/share/classes/javax/xml/catalog/GroupEntry.java
! src/java.xml/share/classes/javax/xml/catalog/UriEntry.java
! src/java.xml/share/classes/javax/xml/catalog/Util.java
! test/javax/xml/jaxp/functional/catalog/SpecifyCatalogTest.java
+ test/javax/xml/jaxp/functional/catalog/catalogFiles/dummy.xml
! test/javax/xml/jaxp/libs/catalog/CatalogTestUtils.java
+ test/javax/xml/jaxp/libs/jaxp/library/JarUtils.java
+ test/javax/xml/jaxp/libs/jaxp/library/SimpleHttpServer.java
+ test/javax/xml/jaxp/unittest/catalog/CatalogAccessTest.java
+ test/javax/xml/jaxp/unittest/catalog/CatalogFileInputTest.java
- test/javax/xml/jaxp/unittest/catalog/CatalogInvalidPathTest.java
! test/javax/xml/jaxp/unittest/catalog/CatalogSupportBase.java
! test/javax/xml/jaxp/unittest/catalog/CatalogTest.java
+ test/javax/xml/jaxp/unittest/catalog/jar/META-INF/MANIFEST.MF
+ test/javax/xml/jaxp/unittest/catalog/jar/META-INF/catalog/ws-addr.xsd
+ test/javax/xml/jaxp/unittest/catalog/jar/META-INF/jax-ws-catalog.xml

Changeset: 76792df0bfd0
Author:lana
Date:  2017-01-13 01:35 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/76792df0bfd0

Merge

- test/javax/xml/jaxp/unittest/catalog/CatalogInvalidPathTest.java

Changeset: 1384504d2cd0
Author:clanger
Date:  2017-01-16 15:44 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/1384504d2cd0

8169631: [JAXP] XALAN: transformation of XML via namespace-unaware SAX input 
yields a different result than namespace-unaware DOM input
Reviewed-by: joehw

! 
src/java.xml/share/classes/com/sun/org/apache/xalan/internal/xsltc/trax/DOM2SAX.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/dtm/ref/sax2dtm/SAX2DTM.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/dtm/ref/sax2dtm/SAX2DTM2.java
! test/javax/xml/jaxp/unittest/transform/TransformerTest.java

Changeset: 0908877116d1
Author:lana
Date:  2017-01-19 22:31 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/0908877116d1

Added tag jdk-9+153 for changeset 1384504d2cd0

! .hgtags

Changeset: 37ceb2919ba9
Author:alanb
Date:  2017-01-20 11:38 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/37ceb2919ba9

Merge

- test/javax/xml/jaxp/unittest/catalog/CatalogInvalidPathTest.java

hg: jigsaw/jake/hotspot: 2 new changesets

[alan . bateman]

Changeset: fc7e94cb7485
Author:lana
Date:  2017-01-19 22:31 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/fc7e94cb7485

Added tag jdk-9+153 for changeset 217ba81b9a4c

! .hgtags

Changeset: 68e5712046cb
Author:alanb
Date:  2017-01-20 11:38 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/68e5712046cb

Merge

! .hgtags

hg: jigsaw/jake/jdk: More javadoc clean-up

[alan . bateman]

Changeset: 9cf519e0821d
Author:alanb
Date:  2017-01-20 11:30 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/9cf519e0821d

More javadoc clean-up

! src/java.base/share/classes/java/lang/module/Configuration.java
! src/java.base/share/classes/java/lang/module/ModuleDescriptor.java
! src/java.base/share/classes/java/lang/module/ModuleReader.java
! src/java.base/share/classes/java/lang/module/ModuleReference.java
! src/java.base/share/classes/java/lang/reflect/Layer.java

RFR 8172659: PluginException("TargetPlatform attribute is missing ...") - should be ModuleTarget

[Sundararajan Athijegannathan]

Please review http://cr.openjdk.java.net/~sundar/8172659/webrev.00/ for 
https://bugs.openjdk.java.net/browse/JDK-8172659


Thanks,
-Sundar