Re: RFR: JDK-8194141: Remove JDK9Wrappers

[Erik Joelsson]

Looks good to me.

/Erik


On 2018-01-03 23:42, Jonathan Gibbons wrote:

Updated webrev, reverting unnecessary change to generated-configure.sh

http://cr.openjdk.java.net/~jjg/8194141/webrev.01/

-- Jon

On 01/03/2018 01:44 PM, Jonathan Gibbons wrote:
Please review this code to remove a "temporary" class from javac, to 
provide reflective access to JDK 9 API, so that javac itself could be 
built with JDK 8.  This temporary class is no longer required, now 
that the boot JDK for JDK 10 onwards is JDK 9.


This is primarily about replacing the use of a javac internal class 
with the underlying direct API calls. However, a minor change to the 
autoconf spec is also required, to grant compile-time access to some 
internal JDK API. Some comments from the jdk.compiler module-info are 
also updated.


The fix is proposed for JDK 11.

JBS: https://bugs.openjdk.java.net/browse/JDK-8194141
Webrev: http://cr.openjdk.java.net/~jjg/8194141/webrev.00

-- Jon

Re: [12] RFR JDK-8202941: GenModuleInfoSource build tool does not detect missing semicolons

[Erik Joelsson]

Build change looks good.

/Erik


On 2018-07-27 13:12, mandy chung wrote:

GenModuleInfoSource build tool augments module-info.java to include
platform-specific qualified exports/opens and uses/provides services.
This fixes the build tool to catch syntax errors as closely as javac
does. The build tool remains to allow duplicated provides until
JDK-8208463 is resolved.

http://cr.openjdk.java.net/~mchung/jdk12/webrevs/8202941/webrev.00/

Mandy

Re: RFR: 8214796: Create a jlink plugin for stripping debug info symbols from native libraries

[Erik Joelsson]

Hello Severin,

There is a macro for automatically finding all source dirs for a module. 
So in Gensrc-jdk.jlink.gmk, I think it would be better expressed using 
that macro, like this:


JLINK_RESOURCE_DIRS := $(wildcard $(addsuffix 
/jdk/tools/jlink/resources, $(call FindModuleSrcSdirs, jdk.jlink)))


The above could/should even be inlined.

Otherwise build changes look ok.

/Erik

On 2019-02-07 09:09, Severin Gehwolf wrote:

Hi,

Could I please get reviews for this enhancement? It adds a debug
symbols stripping plug-in to jlink for Linux and Unix platforms. It's
the first platform specific jlink plugin and the approach taken for
keeping it contained is to use a plugin specific ResourceBundle.
Discussion for this happened in [1].

The test uses a native library which should never get debug symbols
stripped during the test library build. As such, tiny modifications
were needed to make/common/TestFilesCompilation.gmk. Hence, build-dev
being on the list for this RFR. The test currently only runs on Linux
and requires objcopy to be available. Otherwise the test is being
skipped.

Example usage of this plugin is described in the bug.

webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8214796/04/webrev/
Bug: https://bugs.openjdk.java.net/browse/JDK-8214796

Testing: test/jdk/tools/jlink test/jdk/jdk/modules tests on Linux
x86_64 (with good and broken objcopy) and the newly added test. It's
currently running through jdk/submit too.

Thoughts?

Thanks,
Severin

[1] http://mail.openjdk.java.net/pipermail/jigsaw-dev/2019-January/014109.html

Re: RFR: 8214796: Create a jlink plugin for stripping debug info symbols from native libraries

[Erik Joelsson]

On 2019-02-07 11:09, Severin Gehwolf wrote:

Hi Erik,

On Thu, 2019-02-07 at 09:39 -0800, Erik Joelsson wrote:

Hello Severin,

There is a macro for automatically finding all source dirs for a module.
So in Gensrc-jdk.jlink.gmk, I think it would be better expressed using
that macro, like this:

JLINK_RESOURCE_DIRS := $(wildcard $(addsuffix
/jdk/tools/jlink/resources, $(call FindModuleSrcSdirs, jdk.jlink)))

The above could/should even be inlined.

I've considered this. It seems, though, that FindModuleSrcDirs comes
from make/common/Modules.gmk which isn't included in
make/gensrc/Gensrc-jdk.jlink.gmk. Given that it has already caused
problems with multiple includes of Modules.gmk (JDK-8213736) I was
reluctant to include it here too. Without the new include the above
won't work.

The approach I've taken here seems to be the lesser evil. Thoughts?


I appreciate your concern, but JDK-8213736 was a problem in Main.gmk, 
which is part of where Modules.gmk gets bootstrapped. In any normal 
makefile (as in where stuff is just being built), the bootstrapping is 
done and including Modules.gmk is completely fine. Any 
-.gmk file certainly qualifies here.


/Erik


Thanks,
Severin


Otherwise build changes look ok.

/Erik

On 2019-02-07 09:09, Severin Gehwolf wrote:

Hi,

Could I please get reviews for this enhancement? It adds a debug
symbols stripping plug-in to jlink for Linux and Unix platforms. It's
the first platform specific jlink plugin and the approach taken for
keeping it contained is to use a plugin specific ResourceBundle.
Discussion for this happened in [1].

The test uses a native library which should never get debug symbols
stripped during the test library build. As such, tiny modifications
were needed to make/common/TestFilesCompilation.gmk. Hence, build-dev
being on the list for this RFR. The test currently only runs on Linux
and requires objcopy to be available. Otherwise the test is being
skipped.

Example usage of this plugin is described in the bug.

webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8214796/04/webrev/
Bug: https://bugs.openjdk.java.net/browse/JDK-8214796

Testing: test/jdk/tools/jlink test/jdk/jdk/modules tests on Linux
x86_64 (with good and broken objcopy) and the newly added test. It's
currently running through jdk/submit too.

Thoughts?

Thanks,
Severin

[1] http://mail.openjdk.java.net/pipermail/jigsaw-dev/2019-January/014109.html

Re: RFR: 8214796: Create a jlink plugin for stripping debug info symbols from native libraries

[Erik Joelsson]

Hello Severin,

I think you also need a $(wildcard ) around it, but I may be wrong. Did 
you try this version?


Also, please do not indent so much. We have style guidelines [1], which 
recommend 4 spaces for line break indentation.


/Erik

[1] http://openjdk.java.net/groups/build/doc/code-conventions.html

On 2019-02-11 10:03, Severin Gehwolf wrote:

Hi Erik,

On Thu, 2019-02-07 at 17:01 -0800, Erik Joelsson wrote:

On 2019-02-07 11:09, Severin Gehwolf wrote:

Hi Erik,

On Thu, 2019-02-07 at 09:39 -0800, Erik Joelsson wrote:

Hello Severin,

There is a macro for automatically finding all source dirs for a module.
So in Gensrc-jdk.jlink.gmk, I think it would be better expressed using
that macro, like this:

JLINK_RESOURCE_DIRS := $(wildcard $(addsuffix
/jdk/tools/jlink/resources, $(call FindModuleSrcSdirs, jdk.jlink)))

The above could/should even be inlined.

I've considered this. It seems, though, that FindModuleSrcDirs comes
from make/common/Modules.gmk which isn't included in
make/gensrc/Gensrc-jdk.jlink.gmk. Given that it has already caused
problems with multiple includes of Modules.gmk (JDK-8213736) I was
reluctant to include it here too. Without the new include the above
won't work.

The approach I've taken here seems to be the lesser evil. Thoughts?

I appreciate your concern, but JDK-8213736 was a problem in Main.gmk,
which is part of where Modules.gmk gets bootstrapped. In any normal
makefile (as in where stuff is just being built), the bootstrapping is
done and including Modules.gmk is completely fine. Any
-.gmk file certainly qualifies here.

OK. Updated:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8214796/05/

Thanks,
Severin


/Erik


Thanks,
Severin


Otherwise build changes look ok.

/Erik

On 2019-02-07 09:09, Severin Gehwolf wrote:

Hi,

Could I please get reviews for this enhancement? It adds a
debug
symbols stripping plug-in to jlink for Linux and Unix
platforms. It's
the first platform specific jlink plugin and the approach taken
for
keeping it contained is to use a plugin specific
ResourceBundle.
Discussion for this happened in [1].

The test uses a native library which should never get debug
symbols
stripped during the test library build. As such, tiny
modifications
were needed to make/common/TestFilesCompilation.gmk. Hence,
build-dev
being on the list for this RFR. The test currently only runs on
Linux
and requires objcopy to be available. Otherwise the test is
being
skipped.

Example usage of this plugin is described in the bug.

webrev:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8214796/04/webrev/
Bug: https://bugs.openjdk.java.net/browse/JDK-8214796

Testing: test/jdk/tools/jlink test/jdk/jdk/modules tests on
Linux
x86_64 (with good and broken objcopy) and the newly added test.
It's
currently running through jdk/submit too.

Thoughts?

Thanks,
Severin

[1]
http://mail.openjdk.java.net/pipermail/jigsaw-dev/2019-January/014109.html

Re: RFR: 8214796: Create a jlink plugin for stripping debug info symbols from native libraries

[Erik Joelsson]

On 2019-02-12 01:35, Severin Gehwolf wrote:

Hi Erik,

On Mon, 2019-02-11 at 11:12 -0800, Erik Joelsson wrote:

Hello Severin,

I think you also need a $(wildcard ) around it, but I may be wrong. Did
you try this version?

Yes, this version works for me without $(wildcard). Why is it needed?


I had to go back and check again to verify, but now I'm sure. The list 
of directories returned by FindModuleSrcDirs is all src dirs for the 
module. Not all of them are going to contain the specific directory 
jdk/tools/jlink/resources. This means SetupCompileProperties will get 
called with a few non existing directories. While this will work fine, 
the find implementation on some platforms will complain (Macos in 
particular), so to avoid introducing confusing warning messages in the 
build, it's better to filter down the list of directories to those that 
actually exist.



Also, please do not indent so much. We have style guidelines [1], which
recommend 4 spaces for line break indentation.

OK, sorry. Fixed locally.


Thanks!

/Erik


Thanks,
Severin


/Erik

[1] http://openjdk.java.net/groups/build/doc/code-conventions.html

On 2019-02-11 10:03, Severin Gehwolf wrote:

Hi Erik,

On Thu, 2019-02-07 at 17:01 -0800, Erik Joelsson wrote:

On 2019-02-07 11:09, Severin Gehwolf wrote:

Hi Erik,

On Thu, 2019-02-07 at 09:39 -0800, Erik Joelsson wrote:

Hello Severin,

There is a macro for automatically finding all source dirs for a module.
So in Gensrc-jdk.jlink.gmk, I think it would be better expressed using
that macro, like this:

JLINK_RESOURCE_DIRS := $(wildcard $(addsuffix
/jdk/tools/jlink/resources, $(call FindModuleSrcSdirs, jdk.jlink)))

The above could/should even be inlined.

I've considered this. It seems, though, that FindModuleSrcDirs comes
from make/common/Modules.gmk which isn't included in
make/gensrc/Gensrc-jdk.jlink.gmk. Given that it has already caused
problems with multiple includes of Modules.gmk (JDK-8213736) I was
reluctant to include it here too. Without the new include the above
won't work.

The approach I've taken here seems to be the lesser evil. Thoughts?

I appreciate your concern, but JDK-8213736 was a problem in Main.gmk,
which is part of where Modules.gmk gets bootstrapped. In any normal
makefile (as in where stuff is just being built), the bootstrapping is
done and including Modules.gmk is completely fine. Any
-.gmk file certainly qualifies here.

OK. Updated:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8214796/05/

Thanks,
Severin


/Erik


Thanks,
Severin


Otherwise build changes look ok.

/Erik

On 2019-02-07 09:09, Severin Gehwolf wrote:

Hi,

Could I please get reviews for this enhancement? It adds a
debug
symbols stripping plug-in to jlink for Linux and Unix
platforms. It's
the first platform specific jlink plugin and the approach taken
for
keeping it contained is to use a plugin specific
ResourceBundle.
Discussion for this happened in [1].

The test uses a native library which should never get debug
symbols
stripped during the test library build. As such, tiny
modifications
were needed to make/common/TestFilesCompilation.gmk. Hence,
build-dev
being on the list for this RFR. The test currently only runs on
Linux
and requires objcopy to be available. Otherwise the test is
being
skipped.

Example usage of this plugin is described in the bug.

webrev:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8214796/04/webrev/
Bug: https://bugs.openjdk.java.net/browse/JDK-8214796

Testing: test/jdk/tools/jlink test/jdk/jdk/modules tests on
Linux
x86_64 (with good and broken objcopy) and the newly added test.
It's
currently running through jdk/submit too.

Thoughts?

Thanks,
Severin

[1]
http://mail.openjdk.java.net/pipermail/jigsaw-dev/2019-January/014109.html

Re: RFR: 8214796: Create a jlink plugin for stripping debug info symbols from native libraries

[Erik Joelsson]

Hello,

On 2019-02-12 09:05, Severin Gehwolf wrote:

Hi Erik,

On Tue, 2019-02-12 at 08:44 -0800, Erik Joelsson wrote:

On 2019-02-12 01:35, Severin Gehwolf wrote:

Hi Erik,

On Mon, 2019-02-11 at 11:12 -0800, Erik Joelsson wrote:

Hello Severin,

I think you also need a $(wildcard ) around it, but I may be wrong. Did
you try this version?

Yes, this version works for me without $(wildcard). Why is it needed?

I had to go back and check again to verify, but now I'm sure. The list
of directories returned by FindModuleSrcDirs is all src dirs for the
module. Not all of them are going to contain the specific directory
jdk/tools/jlink/resources. This means SetupCompileProperties will get
called with a few non existing directories. While this will work fine,
the find implementation on some platforms will complain (Macos in
particular), so to avoid introducing confusing warning messages in the
build, it's better to filter down the list of directories to those that
actually exist.

OK, thanks for the explanation. I suppose $(wildcard ...) does that,
then? I've added it back locally but I have no way of testing whether
this makes any difference, except jdk/submit perhaps?


Yes, that is what wildcard does, it filters out any non existing dirs.

No need for you to verify anything but that it works as far as I am 
concerned. I'm happy with the below.


/Erik


diff --git a/make/gensrc/Gensrc-jdk.jlink.gmk b/make/gensrc/Gensrc-jdk.jlink.gmk
--- a/make/gensrc/Gensrc-jdk.jlink.gmk
+++ b/make/gensrc/Gensrc-jdk.jlink.gmk
@@ -29,8 +29,9 @@
  
  
  
-JLINK_RESOURCES_DIRS := $(addsuffix /jdk/tools/jlink/resources, \

-$(call FindModuleSrcDirs, jdk.jlink))
+# Use wildcard so as to avoid getting non-existing directories back
+JLINK_RESOURCES_DIRS := $(wildcard $(addsuffix /jdk/tools/jlink/resources, \
+$(call FindModuleSrcDirs, jdk.jlink)))
  
  $(eval $(call SetupCompileProperties, JLINK_PROPERTIES, \

  SRC_DIRS := $(JLINK_RESOURCES_DIRS), \

Thanks,
Severin


Also, please do not indent so much. We have style guidelines [1], which
recommend 4 spaces for line break indentation.

OK, sorry. Fixed locally.

Thanks!

/Erik


Thanks,
Severin


/Erik

[1] http://openjdk.java.net/groups/build/doc/code-conventions.html

On 2019-02-11 10:03, Severin Gehwolf wrote:

Hi Erik,

On Thu, 2019-02-07 at 17:01 -0800, Erik Joelsson wrote:

On 2019-02-07 11:09, Severin Gehwolf wrote:

Hi Erik,

On Thu, 2019-02-07 at 09:39 -0800, Erik Joelsson wrote:

Hello Severin,

There is a macro for automatically finding all source dirs for a module.
So in Gensrc-jdk.jlink.gmk, I think it would be better expressed using
that macro, like this:

JLINK_RESOURCE_DIRS := $(wildcard $(addsuffix
/jdk/tools/jlink/resources, $(call FindModuleSrcSdirs, jdk.jlink)))

The above could/should even be inlined.

I've considered this. It seems, though, that FindModuleSrcDirs comes
from make/common/Modules.gmk which isn't included in
make/gensrc/Gensrc-jdk.jlink.gmk. Given that it has already caused
problems with multiple includes of Modules.gmk (JDK-8213736) I was
reluctant to include it here too. Without the new include the above
won't work.

The approach I've taken here seems to be the lesser evil. Thoughts?

I appreciate your concern, but JDK-8213736 was a problem in Main.gmk,
which is part of where Modules.gmk gets bootstrapped. In any normal
makefile (as in where stuff is just being built), the bootstrapping is
done and including Modules.gmk is completely fine. Any
-.gmk file certainly qualifies here.

OK. Updated:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8214796/05/

Thanks,
Severin


/Erik


Thanks,
Severin


Otherwise build changes look ok.

/Erik

On 2019-02-07 09:09, Severin Gehwolf wrote:

Hi,

Could I please get reviews for this enhancement? It adds a
debug
symbols stripping plug-in to jlink for Linux and Unix
platforms. It's
the first platform specific jlink plugin and the approach taken
for
keeping it contained is to use a plugin specific
ResourceBundle.
Discussion for this happened in [1].

The test uses a native library which should never get debug
symbols
stripped during the test library build. As such, tiny
modifications
were needed to make/common/TestFilesCompilation.gmk. Hence,
build-dev
being on the list for this RFR. The test currently only runs on
Linux
and requires objcopy to be available. Otherwise the test is
being
skipped.

Example usage of this plugin is described in the bug.

webrev:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8214796/04/webrev/
Bug: https://bugs.openjdk.java.net/browse/JDK-8214796

Testing: test/jdk/tools/jlink test/jdk/jdk/modules tests on
Linux
x86_64 (with good and broken objcopy) and the newly added test.
It's
currently running through jdk/submit too.

Thoughts?

Thanks,
Severin

[1]
http://mail.openjdk.java.net/pipermail/jigsaw-dev/2019-January/014109.html

Re: RFR: 8165804: Revisit the way of loading BreakIterator rules/dictionaries

[Erik Joelsson]

Build change looks good.

/Erik


On 2016-10-20 10:33, Masayoshi Okutsu wrote:

Hi,

Please review the changes for JDK-8165804 which is a follow-up of 
JDK-8076757. Some notes on the changes.


- Removed INCLUDES := $(TEXT_PKG_LD) from 
make/gendata/GendataBreakIterator.gmk in order to avoid compiling 
non-BreakIterator*.java for the build tool with boot JDK.


- Added sun.util.resources.BreakIteratorResourceBundle which handles 
loading of rule data and dictionaries. BreadIteratorResources* are its 
subclasses in the java.base and jdk.localedata modules.


- In BreakIteratorResources*, regular ResourceBundle loading of 
BreakIteratorInfo* is avoided because a BreakIteratorInfo can't have 
its parent chain. For example, BreakIteratorInfo_th doesn't have the 
value for key "CharacterData" and 
BreakIteratorResources_th.handleGetObject() must return null for 
"CharacterData" rather than loading rule data given by the parent 
BreakIteratorInfo.


- Moved RuleBasedBreakIterator, DictionaryBasedBreakIterator, and 
BreakDictionary to sun.text. BreakIteratorProviderTest.java was 
changed to deal with this refactoring.


Issue:
https://bugs.openjdk.java.net/browse/JDK-8165804

Webrev:
http://cr.openjdk.java.net/~okutsu/9/8165804/webrev.01

Thanks,
Masayoshi

hg: jigsaw/jake: When importing modules, look in modules_src_jake for jake specific module-info.java files

[erik . joelsson]

Changeset: 07c57a17121e
Author:erikj
Date:  2016-11-08 14:40 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/07c57a17121e

When importing modules, look in modules_src_jake for jake specific 
module-info.java files

! common/autoconf/generated-configure.sh
! common/autoconf/source-dirs.m4
! make/Javadoc.gmk

hg: jigsaw/jake: Parse module-info.java.extra in java build tool instead of makefile

[erik . joelsson]

Changeset: 9795bd5076ec
Author:erikj
Date:  2016-11-08 17:01 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/9795bd5076ec

Parse module-info.java.extra in java build tool instead of makefile

! make/GensrcModuleInfo.gmk

hg: jigsaw/jake/jdk: Parse module-info.java.extra in java build tool instead of makefile

[erik . joelsson]

Changeset: 65473330550b
Author:erikj
Date:  2016-11-08 17:01 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/65473330550b

Parse module-info.java.extra in java build tool instead of makefile

! make/src/classes/build/tools/module/GenModuleInfoSource.java

Re: Review Request: JDK-8169816 Move src.zip and jrt-fs.jar under the lib directory

[Erik Joelsson]

Build changes look good.

/Erik

On 2016-11-22 22:07, Mandy Chung wrote:

This patch moves src.zip and jrt-fs.jar from the top-level into
the `lib` directory in the run-time image as we proposed [1].

Webrev:
   http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8169816/webrev.00/

thanks
Mandy
[1] http://mail.openjdk.java.net/pipermail/jigsaw-dev/2016-November/010128.html

hg: jigsaw/jake: Regenerated configure after merge

[erik . joelsson]

Changeset: 13e19d185301
Author:erikj
Date:  2016-11-24 11:37 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/13e19d185301

Regenerated configure after merge

! common/autoconf/generated-configure.sh

hg: jigsaw/jake: Adjusted workaround for javafx import with new module-info format

[erik . joelsson]

Changeset: 891ed56bc4da
Author:erikj
Date:  2016-11-29 17:52 +0100
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/891ed56bc4da

Adjusted workaround for javafx import with new module-info format

! make/common/Modules.gmk

Re: Review Request JDK-8169925: Organize licenses by module in source, JMOD file, and run-time image

[Erik Joelsson]

Hello Mandy,

In autoconf you are also adding a "man" dir, is this intended with this 
change?


In make/InterimImage.gmk, -J-Djlink.debug=true looks like left over 
debug code.


make/CreateJmods.gmk: 83-84, please indent 4 spaces for continuation.

make/Main.gmk: You should not add the dependency there. Instead, add 
"$(JMOD_TARGETS): java.base-copy" around line 661.


Copy-java.base.gmk: 246, please finish these kinds of lists with a # 
indented to the same level as the list of values.


/Erik


On 2016-12-07 22:28, Mandy Chung wrote:

This proposes to organize license files by module in source, JMOD,
and run-time image.

A summary of the proposal:
1. Organize third party notices by module in the source as follows:
 src/$MODULE/{share,$OS}/legal/*
  
The `legal` directory contains one file for each third party

library in the module, for example,
 src/java.base/share/legal/asm.md
   unicode.md
   zlib.md

The proposed template for this file is described in [1] and JEP 201
will be updated to reflect this proposed source layout.

2. Introduce a new LEGAL_NOTICES section in JMOD format. A new jmod
option `—-legal-notices` is added to package legal notices in
a JMOD file.

3. At jlink time, jlink will copy all legal notices from JMOD files
to the `legal` directory in the run-time image.  A plugin is
added to de-duplicate the legal notices if the filename and the
content matches that may reduce the image footprint.

4. THIRD_PARTY_README in the top-level directory of each repo is removed.
Manual edit to this file, multiple copies is no longer needed.

Webrev at:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8169925/webrev.00/

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

Re: Review Request JDK-8169925: Organize licenses by module in source, JMOD file, and run-time image

[Erik Joelsson]

Hello Mandy,

In Copy-java.base.gmk. Modules.gmk is no longer needed. JMOD_* varaibles 
are no longer used.


In Main.gmk, it would be good with a comment explaining why the jmods 
need java.base-copy. Something like:


# All modules include the main license files from java.base.

Otherwise I'm ok with the build changes now.

/Erik

On 2016-12-10 00:19, Mandy Chung wrote:

Erik, Magnus,

I have made the change per your suggestion:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8169925/webrev.01/

Jeannette, Jeff, and I went through all .md files and I think asm.md is the 
only one got the characters messed up.  These files are contributed by 
Jeannette.

Also, per Naoto, cldr.md are now in both java.base and jdk.localedata.

thanks
Mandy

Re: Review Request JDK-8169925: Organize licenses by module in source, JMOD file, and run-time image

[Erik Joelsson]

Looks good, thanks!

/Erik


On 2016-12-10 08:54, Mandy Chung wrote:

On Dec 9, 2016, at 11:51 PM, Erik Joelsson  wrote:

Hello Mandy,

In Copy-java.base.gmk. Modules.gmk is no longer needed. JMOD_* varaibles are no 
longer used.


Good catch.


In Main.gmk, it would be good with a comment explaining why the jmods need 
java.base-copy. Something like:

# All modules include the main license files from java.base.

Otherwise I'm ok with the build changes now.

Updated per your comment:

$ hg diff make/copy/Copy-java.base.gmk
diff --git a/make/copy/Copy-java.base.gmk b/make/copy/Copy-java.base.gmk
--- a/make/copy/Copy-java.base.gmk
+++ b/make/copy/Copy-java.base.gmk
@@ -233,3 +233,17 @@
  endif
  
  

+
+# JDK license and assembly exception files to be packaged in JMOD
+
+JDK_LICENSE ?= $(JDK_TOPDIR)/LICENSE
+JDK_NOTICE  ?= $(JDK_TOPDIR)/ASSEMBLY_EXCEPTION
+
+$(eval $(call SetupCopyFiles, COPY_JDK_NOTICES, \
+FILES := $(JDK_LICENSE) $(JDK_NOTICE), \
+DEST := $(LEGAL_DST_DIR), \
+FLATTEN := true, \
+))
+
+TARGETS += $(COPY_JDK_NOTICES)
+


$ hg diff make/Main.gmk
diff --git a/make/Main.gmk b/make/Main.gmk
--- a/make/Main.gmk
+++ b/make/Main.gmk
@@ -659,6 +659,9 @@
  exploded-image-optimize
endif
  
+  # All modules include the main license files from java.base.

+  $(JMOD_TARGETS): java.base-copy
+
zip-security: java.base-java java.security.jgss-java 
java.security.jgss-libs \
$(filter jdk.crypto%, $(JAVA_TARGETS))
  

Re: Review Request: JDK-8171201 & JDK-8171202: Drop java.compact$N aggregator modules

[Erik Joelsson]

Build changes look good to me.

/Erik


On 2016-12-14 07:49, Mandy Chung wrote:

JDK-8171201: Drop java.compact$N aggregator modules
JDK-8171202: Rename jdk.crypto.pkcs11 and jdk.pack200 to end with Java letters

http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8171201%2b8171202/webrev.00/

Compact Profiles were introduced in SE 8.  The java.compact$N
aggregator module in SE 9 module graph gives unnecessary
importance going forward.  We want people who care about
small systems to think about modules, not profiles.

In addition, the compact profile builds use these aggregator
modules but those builds still have to pull in the necessary
service providers manually, so they aren't a complete solution.

We propose to drop java.compact1, java.compact2, java.compact3
aggregator modules.

This patch also includes the change for JDK-8171202 to rename
jdk.crypto.pkcs11 to jdk.crypto.token and jdk.pack200 to jdk.pack
to prepare the stronger constraint that module names must end in
Java letters [1]

Mandy
[1] 
http://mail.openjdk.java.net/pipermail/jpms-spec-observers/2016-December/000682.html

Re: RFR 8171316: Add IMPLEMENTOR property to the release file

[Erik Joelsson]

Looks good to me.

/Erik


On 2016-12-15 16:32, Sundararajan Athijegannathan wrote:

Please review. Bug: https://bugs.openjdk.java.net/browse/JDK-8171316

top level webrev: 
http://cr.openjdk.java.net/~sundar/8171316/top/webrev.00/

jdk webrev: http://cr.openjdk.java.net/~sundar/8171316/jdk/webrev.00/

Thanks,
-Sundar

Re: 8174823: Module system implementation refresh (3/2017)

[Erik Joelsson]

(adding build-dev)

Build changes look good to me.

/Erik


On 2017-03-21 22:42, Alan Bateman wrote:
We have been accumulating changes in the jake forest for the last few 
weeks to align with the proposals in JSR 376 and also to pick up API 
changes and bug fixes. It's time to bring these changes into jdk9/dev. 
The issue to bring these changes into jdk9/dev in bulk has been 
approved via the FC extension process (still in use during JDK 9 RDP2).


A summary of the main changes is listed in JDK-8174823 [1], and the 
webrevs with the changes to bring to jdk9/dev are here:

   http://cr.openjdk.java.net/~alanb/8174823/1/

The changes are currently based on jdk-9+161 and will be rebased 
before pushing.


-Alan

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

Re: Review Request: JDK-8173303: Add module-subgraph images to main platform documentation

[Erik Joelsson]

Looks ok to me. Not sure about adding a make variable as a parameter, 
but I assume Magnus will fix that as part of JDK-8176785?


/Erik


On 2017-03-25 22:33, Mandy Chung wrote:

I edited the module descriptions per your feedback.  I also revised
GenGraphs tool to take a properties file to customize the dot graphs
for javadoc use.

Updated webrev:
   http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8173303/webrev.02/

Magnus, Erik,

I modified Javadoc.gmk and Main.gmk to add a new target to generate
.dot files for javadoc use.  GenGraphs tool depends on the exploded
image build.  Javadoc.gmk temporarily takes ENABLE_MODULE_GRAPH make
variable for us to enable @moduleGraph taglet until JDK-8176785 is
resolved.

thanks
Mandy


On Mar 25, 2017, at 2:36 AM, Alan Bateman  wrote:

On 24/03/2017 21:50, Mandy Chung wrote:


Alan,

I took another round of edits on the module descriptions:
   http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8173303/webrev.01/

Once we have the unified docs, it will make it easier to review
the module summary page for all modules where we will revise
these module descriptions again.


I went through the updated module descriptions.

Two more that seem to be missing "the" are jdk.net and jdk.sctp, I think they 
will read okay once that is added.


Fixed. I missed that.


jdk.httpserver currently has "Defines the JDK-specific API for HTTP server", it 
might be better to re-shuffle this to "Defines the API for the JDK-specific HTTP server”.


This reads better.


I think the only one that needs re-examination is jdk.charsets. The java.base module contains the 
standard charsets and all other charsets needed to start the runtime on any of the supported 
configurations. It thus varies by platform with jdk.charsets providing the charsets that aren't in 
java.base. Finding the right description is difficult, maybe we should start with "Charset 
provider for the charsets that are not in java.base (mostly double byte and IBM charsets". I 
could imagine linking this to the "Supported encodings" docs page in time.


Let’s start with this version.  I expect we will refine the module
description further next couple weeks.


Everything else looks good.


Thanks
Mandy

Re: Review Request JDK-8175819: OS name and arch in JMOD files should match the values as in the bundle name

[Erik Joelsson]

Hello,

I don't quite understand this. In the proposition below it says osx, and 
x64, but in platform.m4 you generate macos and amd64. Does this then get 
translated again and why are we introducing yet another name for the 
operating system on Apple computers?


/Erik


On 2017-04-03 20:41, Mandy Chung wrote:

Webrev:
   http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8175819/webrev.00/

This revisits the OS name and arch in packaging JDK modules
to extend the module descriptor with ModuleTarget class file
attribute.  We considered matching with the system properties.
Linux x64 JDK can run on a system whose `os.arch` system
property value can be `amd64` or `i586` or `x86_x64`.
Similiarly, windows x86/x64 JDK can run on a system whose
`os.name` system property starts with “Windows” as the
os.name property is set to "Windows XXX" for example
"Windows Server 2012 R2”.  It might be worth considering
multiple OS arch values in ModuleTarget in the future.

JDK bundle names are revised in JDK 9.  This is a good
alternative to be consistent with $OS-$ARCH value in
the bundle names.  This patch proposes to package JDK modules
with OS name and arch to match the values as in JDK bundle names.
jlink will generate the `release` file and set OS_NAME and
OS_ARCH to those values.  This also proposes to drop
OS_VERSION to align with the ModuleTarget class file attribute.

This shows the old and new value of OS_NAME/OS_ARCH properties
in the `release` file:

 JDK 8   JDK 9
 -   -
OS_NAME Linux   linux
 SunOS   solaris
 Darwin  osx
 Windows windows
  
OS_ARCH i386,x86x86

 i586,amd64,x86_64   x64
 sparcv9 sparcv9
 arm arm32
 aarch64 arm64

Mandy
  

Re: Review Request JDK-8175819: OS name and arch in JMOD files should match the values as in the bundle name

[Erik Joelsson]

Oh, I missed the further discussion before I posted this and the webrev 
was apparently updated in place to reflect that discussion. Please 
ignore my comment below.


/Erik


On 2017-04-04 11:57, Erik Joelsson wrote:

Hello,

I don't quite understand this. In the proposition below it says osx, 
and x64, but in platform.m4 you generate macos and amd64. Does this 
then get translated again and why are we introducing yet another name 
for the operating system on Apple computers?


/Erik


On 2017-04-03 20:41, Mandy Chung wrote:

Webrev:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8175819/webrev.00/

This revisits the OS name and arch in packaging JDK modules
to extend the module descriptor with ModuleTarget class file
attribute.  We considered matching with the system properties.
Linux x64 JDK can run on a system whose `os.arch` system
property value can be `amd64` or `i586` or `x86_x64`.
Similiarly, windows x86/x64 JDK can run on a system whose
`os.name` system property starts with “Windows” as the
os.name property is set to "Windows XXX" for example
"Windows Server 2012 R2”.  It might be worth considering
multiple OS arch values in ModuleTarget in the future.

JDK bundle names are revised in JDK 9.  This is a good
alternative to be consistent with $OS-$ARCH value in
the bundle names.  This patch proposes to package JDK modules
with OS name and arch to match the values as in JDK bundle names.
jlink will generate the `release` file and set OS_NAME and
OS_ARCH to those values.  This also proposes to drop
OS_VERSION to align with the ModuleTarget class file attribute.

This shows the old and new value of OS_NAME/OS_ARCH properties
in the `release` file:

 JDK 8   JDK 9
 -   -
OS_NAME Linux   linux
 SunOS   solaris
 Darwin  osx
 Windows windows
  OS_ARCH i386,x86x86
 i586,amd64,x86_64   x64
 sparcv9 sparcv9
 arm arm32
 aarch64 arm64

Mandy

Re: Review Request JDK-8175819: OS name and arch in JMOD files should match the values as in the bundle name

[Erik Joelsson]

On 2017-04-11 09:40, Magnus Ihse Bursie wrote:



On 2017-04-07 22:04, Mandy Chung wrote:


On Apr 6, 2017, at 1:09 AM, Magnus Ihse Bursie 
> wrote:



Having though this over real hard, I'd realized I need to make a 
plea for sanity and consistency. I thought I should lay low in this 
discussion, but I can't. Choosing "amd64" as the name for the 64-bit 
x86 platform is really, really unfortunate and a step backwards in 
our effort to standardize the name of this platform.




I think it may be useful to see the value of `os.arch` system 
property on all platforms

at a glance.

32-bit64-bit

linux i386amd64
arm aarch64
solaris amd64
sparcv9
windows x86 amd64
macos   x86_64

It's late in JDK 9.  The best is to revisit OS and architecture names 
in JDK 10 on the
consistency and simplicity issue.  In the context of JDK-8175819, 
these are names to
pass tojmod —-os-name —-os-arch options. The values are used at 
resolution time
to avoid linking modules for different target platforms into the same 
image.


As Mark suggests, we use the value of `os.arch` system property for 
JMOD file in
JDK 9 and we can change the value in JDK 10.  No change to system 
property and

bundle names, as we said previously.


I agree. It's late in JDK 9.

I'm OK with the proposal of using os.arch for JDK 9, as long as when 
we're revisiting the issue in JDK 10, we do not use the fact that it 
was used in JDK 9 as an argument for not changing it...



+1
/Erik

/Magnus



We should revisit OS and architecture names in JDK 10.  I have updated
JDK-8178016 to reflect that.  Are you okay with that?

Below shows the old and new values in the `release` file and the new 
values

are used in JMOD files.

JDK 8   JDK 9
  -   -
OS_NAME Linux   linux
  SunOS   solaris
  Darwin  macos
  Windows windows

OS_ARCH i386,x86x86
  i586,amd64,x86_64   amd64
  sparcv9 sparcv9
  arm arm
  aarch64 aarch64

Updated webrev:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8175819/webrev.01/index.html 
 



Mandy

Re: Review Request JDK-8182032: Make java.compiler upgradeable

[Erik Joelsson]

Hello Mandy,

Looks good to me.

/Erik


On 2017-06-13 07:35, Mandy Chung wrote:

Webrev at:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8182032/webrev.00/

java.compiler is a standalone technology that allows to be running on older 
JDK, in particular for IDE to support new language features.

This patch takes out the make logic to find the modules that directly and 
indirectly require any upgradeable modules and include them as upgradeable.  
Instead it lists all upgradeable modules.  The list of upgradeable modules is 
small whereas the exclude list is not (which is also error-prone to find them). 
 I have added a new test to verify what modules are hashed in java.base.

Mandy

hg: jigsaw/stage: Moved oracle.accessbridge to its own module. Reorginized CopyFiles.gmk

[erik . joelsson]

Changeset: a3a652668c22
Author:erikj
Date:  2014-08-08 17:32 +0200
URL:   http://hg.openjdk.java.net/jigsaw/stage/rev/a3a652668c22

Moved oracle.accessbridge to its own module. Reorginized CopyFiles.gmk

! make/CompileJavaModules.gmk
! make/Main.gmk

hg: jigsaw/stage/jdk: Moved oracle.accessbridge to its own module. Reorginized CopyFiles.gmk

[erik . joelsson]

Changeset: cc9c120c9321
Author:erikj
Date:  2014-08-08 17:33 +0200
URL:   http://hg.openjdk.java.net/jigsaw/stage/jdk/rev/cc9c120c9321

Moved oracle.accessbridge to its own module. Reorginized CopyFiles.gmk

- make/CopyFiles.gmk
! make/CreateJars.gmk
+ make/copy/Copy-java.base.gmk
+ make/copy/Copy-java.desktop.gmk
+ make/copy/Copy-java.logging.gmk
+ make/copy/Copy-java.management.gmk
+ make/copy/Copy-jdk.hprof.agent.gmk
+ make/copy/Copy-jdk.jdwp.agent.gmk
+ make/copy/CopyCommon.gmk
! make/lib/PlatformLibraries.gmk

Re: RFR [9] Modular Source Code

[Erik Joelsson]

I should probably write something about the rather extensive changes to 
the build logic in this patch.


As the source gets organized around modules, it made sense to also 
organize the build more around modules. In this patch, the makefiles 
have in large parts been split into module specific files and the top 
level targets are oriented around modules. This means the top level 
targets are much more fine granular than currently in JDK 9.


Another difference is that the top level targets are now able to run 
concurrently, to give more opportunity for utilizing multiple cpus. In 
JDK 9, the build moves sequentially through each repository, in a given 
order, now make is free to run more things in parallel. This makes the 
build faster, at least on beefy hardware. The drawback is that the build 
log gets a bit more confusing. When something fails, it won't interrupt 
other building threads at once, so the actual failure may be further up 
the log (even further than before). I have found that searching for 
"Error 2" is a good way to find the real failure. Another consequence is 
that the build time summary at the end only displays total time as there 
is no good definition for how much time was spent in each repository 
anymore.


A summary on the new targets:

make [default]
Does pretty much the same as before. It compiles everything but does not 
build all jars or create images.


make all
Builds everything, including jars, images and docs. Also runs a 
verification tool on the java classes which will point out any broken 
module boundaries.


make images
Same as before

make hotspot
Builds the hotspot repository, like before.

make docs
Builds all the documentation, including javadoc.

make docs-javadoc
Builds just the javadoc. This target has very few prerequisites so 
provides a fast way to just build javadoc.


make gensrc
Runs all source code generation steps.

make java
Compiles all java classes.
Other similar targets are libs, launchers, gendata and copy

make java.desktop
Compiles everything in the java.desktop module (and its dependencies), 
both java and native code. Works for any module name.


make java.desktop-java
Compiles the java classes in java.desktop (and its dependencies). Works 
for any module name (and -gensrc, -libs, -launchers, -gendata, -copy)


In addition to this, the suffix -only can be added to any target to 
disable the prerequisites for it. Using this is not recommended but it 
may save time when doing certain incremental builds and you are in a 
terrible hurry.


For incremental builds, sjavac can be used and works reasonably well 
(configure --enable-sjavac). Work is in progress on making it work even 
better. The old workaround JDK_FILTER=package/to/compile is still working.


The clean target is still oriented around repositories, mostly because 
the build output is still in large parts repository oriented. This is 
something we hope to improve later.


/Erik


On 2014-08-12 16:10, Chris Hegarty wrote:

This is a review request for the Initial changes for JEP 201: Modular Source 
Code [1].

There are a number of individuals responsible for these changes. Some, possibly 
not all, are explicitly listed in the To section of this mail, and they will 
help address any comments arising from this review request.

For the purposes of review, the actual source file moves have been omitted from the 
webrev below, with the exception of any source file that has a change to it’s actual 
content. The new location of the source files can be determined from JEP 201 [1] and JEP 
200 "The Modular JDK" [2], or by browsing the staging forest [3].

Webrevs:
  http://cr.openjdk.java.net/~chegar/8054834/00/

Due to the significant impact of these changes, a JDK 9 promotion has been 
tentatively reserved for their integration. All comments are welcome, although 
given the nature of the changes then we might have to create separate issues in 
JIRA to address some of them later in jdk9/dev..

-Chris.

[1] https://bugs.openjdk.java.net/browse/JDK-8051619
[2] https://bugs.openjdk.java.net/browse/JDK-8051618
[3] http://hg.openjdk.java.net/jigsaw/stage

Re: RFR [9] Modular Source Code

[Erik Joelsson]

On 2014-08-13 14:17, Chris Hegarty wrote:

Thanks for the explanation Erik.

I have taken a pass over the changes, and they look ok to me ( I am
happy to be listed as a reviewer ). I also did several build and test
runs on Solaris, Linux, Max OSX, and Windows. All look good.

I am seeing, in some cases, about a 20% reduction in image build times
on an 8 core i7, running Linux x86.


Nice to hear!

One question; Are there any new requirements on build systems as a
result of these changes?

Yes and no. The build should still be compatible with gnu make 3.81. 
However, certain builds of that make version (particularly the one we 
have used for jdk8 internally) are known to crash in Cygwin and it is 
instead recommended to use a newer 4.0 version of make in Cygwin. Also 
to get good concurrent build performance on Windows, gnu make 4.0 is 
required for Cygwin. I have done test builds with msys and it seems to 
be working too.


One more thing that I forgot to mention. If you are using gnu make 4.0, 
there is a new feature called output-sync that can be enabled that will 
make it somewhat easier to read the build output. This can be enabled 
either with the configure parameter --with-output-sync=recurse, or at 
the make command line "make OUTPUT_SYNC=recurse". (it will be enabled by 
default in JPRT). More information on this can be found here: 
http://www.gnu.org/software/make/manual/make.html#Parallel-Output


/Erik

Re: RFR [9] Modular Source Code

[Erik Joelsson]

Hello Mike,

Thanks for the comments. See inline.

On 2014-08-13 23:29, Mike Duigou wrote:

There's a lot to review here. This is not a complete review but hopefully contributes to 
our review "coverage". I am focusing on the top project in this set of comments.

- --with-output-sync seems like it should be on by default if available. 
Downside? This could also be split out from the jigsaw changes if there is any 
interest in reducing the patch size.
There are downsides yes. The way this works is that make will buffer 
output from all commands and print them when done. Depending on level 
that's when the command line is done, the recipe is done, or the 
complete submake. I played around with having it default for a while. 
For a normal build this isn't too confusing once you get used to it, but 
it really made running tests annoying as the output from jtreg would all 
be buffered until it was done. Also, a hotspot developer that was to 
build hotspot from the root repo would see nothing until the whole build 
was complete. I simply believe we need some more experience with this 
before we can decide on a better default behavior. Possibly we need to 
do the hotspot makefile rewrite first so that the hotspot build can be 
split into smaller logical chunks from the top level.


I wanted the output-sync feature to be in this from the start to make 
JPRT logs easier to look at, as the feature will be default turned on 
there. Especially debug logs become very hard to read without it.


- what is TESTMAKE_OUTPUTDIR for? (ugh, more outputdir dirs...)
While doing this work, I had the need for adding features to 
SetupArchive (mainly to support multiple source dirs for jars). It 
quickly turned quite nasty and to make development easier, I added 
specific test cases for this macro in a separate makefile structure. 
This is the output directory for those tests.

- spec.gmk.in: Can we have a separate assignment for JAVA_TOOL_FLAGS_SMALL? It 
is nice to be able to see every AC_SUBST somewhere solo.

Certainly, that makes sense. Must have just missed it.

- jdk-options.m4: should with_cacerts_file being empty not merit an error? what 
does the empty default do?
The default is to use the bundled one. I didn't like having that default 
being defined in configure. I think it better belongs in the makefile 
handling the file.

- javadoc.gmk: retire JDK_IMPSRC, JDK_GENSRC, JDK_SHARE_CLASSES and 
JDK_SHARE_SRC
We should certainly clean up javadoc.gmk properly, but I thought that 
was out of scope for this patch. You do have a point in that those 
variables are now unused so no longer need to be declared.

- javadoc.gmk: JAVADOC_CMD should perhaps use (currently non-existant) 
JAVA_TOOL_FLAGS_BIG or at least JAVA_FLAGS_BIG?

I agree, but that should be a separate change.

- MakeHelpers: CleanComponent should call strip on the $1 argument to $(RM) so 
that it is deleting what it promises to be deleting. Or it could check to make 
sure $(words $1) is 1
Strip clears leading and trailing whitespace. The reason I added a strip 
to the echo is that in some cases the macro was called with a space 
after comma and in some not. The output looked weird when the echo line 
sometimes had 2 spaces before the component name instead of one. It 
should not matter to the rm command line however.

- modules.xml "Changes to this file will require review by Committers to Project 
Jigsaw." Will this be true after integration into jdk9/dev repo?

- modules.list seems to be redundant with modules.xml but there don't seem to 
be any measures to ensure that they remain in sync. Even a comment in 
modules.xml would help. This kind of problem has been a source of errors in the 
past.
They are redundant yes. modules.list is used by make to extract 
dependency information and modules.xml is used by the verification tool. 
In jigsaw development both files were dynamically created during the 
build process and here we simply committed static versions of them. 
Ideally we should only need one, but it's a temporary solution anyway. 
We would need to create a tool to extract dependency information from 
the xml to make.

- What is TestMake.gmk and associated for?
These are my new tests for the common makefile logic. It's far from a 
complete coverage, but it has already helped me greatly.

jdk project:

- I am slightly unsettled by the number of makefiles and putting them all in to 
the same directory. Will they eventually be moved into their modules?
This is something I'm thinking about and would like to discuss. I think 
that ideally the makefiles for specific modules should be moved to 
module specific directories. For now I kept the existing task based 
directory structure.

More to come but first I want to build it!

Go for it!

/Erik

On Aug 12 2014, at 07:10 , Chris Hegarty  wrote:


This is a review request for the Initial changes for JEP 201: Modular Source 
Code [1].

There are a number of individuals responsible for these changes. Some, possibly 

Re: RFR [9] Modular Source Code

[Erik Joelsson]

On 2014-08-14 09:07, Erik Joelsson wrote:
- javadoc.gmk: JAVADOC_CMD should perhaps use (currently non-existant) 
JAVA_TOOL_FLAGS_BIG or at least JAVA_FLAGS_BIG?

I agree, but that should be a separate change.
Actually, the variable JAVA already contains the big java flags. The 
correct fix here is to remove the redundant explicit mx flag from 
Javadoc.gmk and just trust that JAVA is correctly sized.


/Erik

hg: jigsaw/stage: Removing now unused variables from Javadoc.gmk. Adding JAVA_TOOL_FLAGS_SMALL to spec.gmk.in.

[erik . joelsson]

Changeset: 7a5ae6a78455
Author:erikj
Date:  2014-08-14 10:35 +0200
URL:   http://hg.openjdk.java.net/jigsaw/stage/rev/7a5ae6a78455

Removing now unused variables from Javadoc.gmk. Adding JAVA_TOOL_FLAGS_SMALL to 
spec.gmk.in.

! common/autoconf/spec.gmk.in
! make/Javadoc.gmk

Re: RFR [9] Modular Source Code

[Erik Joelsson]

Hello, Magnus

Thanks for the thorough review. I tend to agree with Alan, that we 
should rather push this in unless we find critical issues but make sure 
to continue cleaning this up in jdk9/dev. I have created bugs for 
(almost) everything you have listed below. See inline for links and 
comments.


On 2014-08-15 10:52, Magnus Ihse Bursie wrote:

Here follows a partial review of the changes in the jdk repo. Even 
though it's long, I'm not done. ;-)


*** General issues ***

* The new directory jdk/make/bundle should instead reside in 
jdk/make/data/bundles.


* In GensrcSwing.gmk is a "$(if $(SHUFFLED)..." part that seems to be 
remnants of older, temporary work.


Created " General cleanup of minor issues from source restructure" 
https://bugs.openjdk.java.net/browse/JDK-8055188
* The new file GensrcProviders.gmk are included by the Gensrc files 
for jdk.attach and jdk.jdi, which only uses half of it each. Each half 
is just a few lines long. I believe a better solution is to remove the 
GensrcProviders.gmk file, move the process-provider macro to 
GensrcCommon.gmk and move the two actual rules to the respective 
module gensrc file.


* In the gensrc directory, there are now almost twice as many files. 
For many of them, the following pattern holds:

 GensrcOldStyle.gmk -- defines the actual logic for some gensrc target
 Gensrc-.gmk -- does nothing but includes GensrcOldStyle.gmk

In these cases, I think the contents of GensrcOldStyle should be 
inlined directly in the Gensrc-.gmk instead. This holds for 
all modules except the two mammuth modules java.base and java.desktop. 
(Depending on the treatment of GensrcProviders as described above.)


Created "Cleanup gensrc after source code restructure" 
https://bugs.openjdk.java.net/browse/JDK-8055189

* In CreateJars.gmk, the following new comment is found embedded:
  # This currently won't work with modular build layout, but 
there currently are no

  # types needing to be re added.
In my opinion, leaving code which looks like it's working but with a 
note saying "out of order", is bad practice. I'd recommend that the 
code is removed or commented out, if it is not needed.


While I agree on this in principle, I would rather not spend time on the 
profiles generation code since it's planned to go away soon anyway.
* In CreateJars.gmk, in BUILD_TOOLS_JAR: The following looks like a 
duplication; I believe the "classes" version should be removed.
$(JDK_OUTPUTDIR)/modules/jdk.jdi/META-INF/services/com.sun.jdi.connect.Connector 
\
$(JDK_OUTPUTDIR)/classes/META-INF/services/com.sun.jdi.connect.Connector 
\


* In CreateSecurityJars.gmk, there is a variable 
SECURITY_CLASSES_SUBDIR that is always set to 'modules'. I believe 
this is remains of an older temporary design, and that "modules" 
should be hard-coded into the paths.


* The old Setup.gmk had a very generic-sounding name, even though it 
only did setup java compilers. So, the rename to SetupJava.gmk was 
good; however, I'd suggest we follow it through all the way and rename 
it further to SetupJavaCompilers.gmk, since that is an even more 
accurate description of it's job.


* The file CopyIntoClasses.gmk is not used anymore and should be removed.

* In CoreLibraries.gmk, there used to be a line "BUILD_LIBRARIES += 
$(BUILD_LIBFDLIBM)" which is now removed. After discussion with Erik, 
I learned that this was since the libfdlibm was not delivered in 
itself, but was used solely as a helper lib for libjava, which has 
BUILD_LIBFDLIBM as a requirement. While this change is thus correct, I 
believe a comment describing this fact would be in place on libfdlibm, 
since it makes it behave differently than all other libraries.



Added to general cleanup

*** Issues with source files moving and its repercussions ***

* When the source code has moved, especially the native libraries, 
most of the specific INCLUDE and EXCLUDE statements are, or should be, 
unnecessary. Nevertheless, there are still several occasions of such 
statements. In some cases, it seems like dead code that can (and 
should) be removed. But in some cases, I believe it is an indication 
that the source code has still not yet been moved to a suitable 
location. I believe the end goal with this shuffling regarding native 
library source code is that there should be a one-to-one 
correspondance between compiled native library and source code 
directory. This is now indeed the case for 99% of the libraries, but 
there are still some exceptions.


This is a slightly vague point at the moment. I indent to check the 
INCLUDE and EXCLUDE statements more fully and will post a second 
review with results of what I find. Nevertheless, I think it is 
important to make sure we do get things correct this time.


Created "Cleanup include and exclude of native libraries after source 
code restructure" https://bugs.openjdk.java.net/browse/JDK-8055190

*** Issues regarding modules.xml ***

The new modules.xml and associated Java tools and 

Re: RFR [9] Modular Source Code

[Erik Joelsson]

On 2014-08-15 14:54, Maurizio Cimadamore wrote:
I'm looking at the langtools-related changes in build.xml; I what is 
the degree of support available in the build.xml ant file for the 
Jigsaw world? It seems to me that not all the target would function 
and it also seems that some of the properties previously encoded in a 
side property file (make/build.properties) have now been inlined in 
the build.xml file itself, which seems problematic maintenance-wise. 
Am I missing something?


As far as I know, build.xml has so far been supported by the langtools 
team themselves and all changes to the file in this patch have been made 
by them. Jon may have something to say?


/Erik


Maurizio

On 12/08/14 15:10, Chris Hegarty wrote:
This is a review request for the Initial changes for JEP 201: Modular 
Source Code [1].


There are a number of individuals responsible for these changes. 
Some, possibly not all, are explicitly listed in the To section of 
this mail, and they will help address any comments arising from this 
review request.


For the purposes of review, the actual source file moves have been 
omitted from the webrev below, with the exception of any source file 
that has a change to it’s actual content. The new location of the 
source files can be determined from JEP 201 [1] and JEP 200 "The 
Modular JDK" [2], or by browsing the staging forest [3].


Webrevs:
  http://cr.openjdk.java.net/~chegar/8054834/00/

Due to the significant impact of these changes, a JDK 9 promotion has 
been tentatively reserved for their integration. All comments are 
welcome, although given the nature of the changes then we might have 
to create separate issues in JIRA to address some of them later in 
jdk9/dev..


-Chris.

[1] https://bugs.openjdk.java.net/browse/JDK-8051619
[2] https://bugs.openjdk.java.net/browse/JDK-8051618
[3] http://hg.openjdk.java.net/jigsaw/stage

Re: RFR [9] Modular Source Code

[Erik Joelsson]

Hello Omair,

No, as I tried to explain in my initial mail on this thread, since the 
modules are all compiled in parallel, and not sequentially like the 
current build does with the repositories, it doesn't make much sense 
timing each module. The numbers would be meaningless.


/Erik

On 2014-08-15 17:55, Omair Majid wrote:

Hi,

* Alan Bateman  [2014-08-13 08:36]:

Just to add to Chris and Erik's mails, I would encourage everyone that
pushes to jdk9/dev or the other jdk9 project integration forests to clone
and build jigsaw/stage and get familiar with the proposed layout, new build
targets, and the very different output emitted during the build. The changes
are arguably as significant as the transition in JDK 8 from the "old build"
to the "new build" so the more people taking the forest for a test drive the
better. If you maintain your our own own IDE project then you'll likely have
to adjust file paths so any issues encountered would be useful to hear about
too.

Just one RFE for now:

The new build would provide a detailed breakdown of the build time of
each repo. With the 'new new build', all I see is an overall time:


- Build times ---
Start 2014-08-15 10:59:27
End   2014-08-15 11:17:53

00:18:26 TOTAL
-

Will a detailed breakdown of the build times for each module make a
return?

Thanks,
Omair

Re: RFR [9] Modular Source Code

[Erik Joelsson]

Hello,

The basic rule for the new source layout is that for each library, there 
is a directory where all sources for that library go. This was hard to 
apply to libawt and friends since as you say, some files go in libawt on 
windows and libawt_xawt on linux and solaris. For now, I put those in 
common for lack of a better place.


/Erik


On 2014-08-20 01:09, Phil Race wrote:

On a related note I am scratching my head about why some files
destined to be compiled into libawt or libawt_xawt go into a directory
called 'common'.
Eg OpenGL sources are in common but aren't common to all libs
or even to all awt libs, since they would go into libawt on windows
and libawt_xawt on unix and not at all into libawt_headless.

'common 'might (guessing here)  have started out as place to put 
interface

header files shared between libs but maybe got adopted for other uses ?

-phil.



On 08/18/2014 07:15 AM, Anthony Petrov wrote:

On 8/18/2014 5:47 PM, Magnus Ihse Bursie wrote:

libawt et al:
   The relation between libawt, libawt_headless, libjawt, libawt_xawt
and libawt_lwawt are hairy enough to make my brain curl up. I believe
there are simplifications to be made but I gave up trying to figure 
them

out.


libawt contains code that is shared between all AWT lib 
implementations. Depending on what mode/toolkit is requested, it 
loads a specific variant of the awt native library, such as:


 - libawt_headless - headless AWT implementation.
 - libawt_xawt - XToolkit implementation (uses X11 for GUI)
 - libawt_lwawt - CToolkit implementation (uses Cocoa for GUI)

Historically, we were able to choose between lwawt and xawt on Mac in 
the past, but we no longer support or even build xawt on Mac. Also, 
in the past there was MToolkit (which used Motif for GUI). Again, we 
no longer support this toolkit. So, currently we always use xawt on 
Linux/Solaris and lwawt on Mac. But since we still need to choose 
between a real toolkit and a headless toolkit, we need the common 
libawt library.


libjawt is a helper library that implements JAWT API and is loaded by 
applications that use the JAWT interface which allows them to get 
direct access to the native AWT drawing surface and use native APIs 
(e.g. OpenGL) to draw on the surface. This library isn't needed for 
regular AWT/Swing applications.


So I'm not sure if the current set of AWT libraries could be 
simplified any further.


Hope this helps.

--
best regards,
Anthony

hg: jigsaw/m2: 20 new changesets

[erik . joelsson]

Changeset: dbb964d72f25
Author:erikj
Date:  2014-08-21 11:05 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/dbb964d72f25

8055188: General cleanup of minor issues from source restructure
Reviewed-by: tbell

! make/CompileJavaModules.gmk
! make/Main.gmk
- make/common/SetupJava.gmk
+ make/common/SetupJavaCompilers.gmk

Changeset: 70e378877c6a
Author:erikj
Date:  2014-08-21 12:22 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/70e378877c6a

8055095: Improve "do nothing" incremental build performance after modularized 
source code integration
Reviewed-by: tbell

! make/CompileJavaModules.gmk
! make/Main.gmk
! make/MakeHelpers.gmk
! make/common/MakeBase.gmk
+ make/common/Modules.gmk

Changeset: d9edec389373
Author:mduigou
Date:  2014-08-21 10:22 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/d9edec389373

8055772: get_source.sh : version check assumes English localization
Reviewed-by: mduigou, tbell
Contributed-by: bitterf...@gmail.com

! get_source.sh

Changeset: f91649eb1db2
Author:mduigou
Date:  2014-08-21 10:23 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/f91649eb1db2

8055331: cleaner handling of sub-process non-zero exit result.
Reviewed-by: tbell

! common/bin/hgforest.sh

Changeset: 274543b2de27
Author:erikj
Date:  2014-08-22 11:51 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/274543b2de27

8055096: Remove explicit mx flag from javadoc command line
Reviewed-by: tbell, mduigou

! make/Javadoc.gmk

Changeset: ba1bab36d1bf
Author:ctornqvi
Date:  2014-08-18 14:30 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/ba1bab36d1bf

8032999: [TESTBUG] JT-Reg Runtime tests to be run as part of JPRT submit job
Summary: JPRT will now run most of the tests in /runtime
Reviewed-by: dcubed, mseledtsov, zgu, mikael

! make/jprt.properties

Changeset: 27c26438e4c8
Author:sspitsyn
Date:  2014-08-20 17:05 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/27c26438e4c8

Merge


Changeset: 446472cfcca8
Author:anoll
Date:  2014-08-05 09:05 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/446472cfcca8

8054013: run hotspot JTREG compiler tests only on fastdebug platforms and also 
on macosx
Summary: Modify hotspot/make/jprt.properties and make/jprt.properties on the 
top level to add/remove the relevant platforms.
Reviewed-by: kvn, roland
Contributed-by: Zoltan Majo 

! make/jprt.properties

Changeset: b86c13b9a7df
Author:mduigou
Date:  2014-08-13 19:39 +
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/b86c13b9a7df

8047952: Remove FORTIFY_SOURCE from fastdebug and slowdebug builds
Reviewed-by: dholmes

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

Changeset: 0736f13c91bd
Author:kvn
Date:  2014-08-21 15:37 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/0736f13c91bd

Merge

! common/autoconf/generated-configure.sh
! make/jprt.properties

Changeset: b953531f403d
Author:amurillo
Date:  2014-08-22 10:03 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/b953531f403d

Merge

! common/autoconf/flags.m4
! common/autoconf/generated-configure.sh
! make/jprt.properties

Changeset: a423abd014f7
Author:mchung
Date:  2014-08-26 14:35 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/a423abd014f7

8055230: Rename attach provider implementation class be platform neutral
Reviewed-by: alanb, chegar, dfuchs, dholmes, erikj, sla

! make/CompileJavaModules.gmk

Changeset: 3c994e1648ac
Author:erikj
Date:  2014-08-27 10:59 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/3c994e1648ac

8056064: Fix corba locale build problem on windows
Reviewed-by: alanb, tbell

! make/common/IdlCompilation.gmk

Changeset: 1823df83c30e
Author:erikj
Date:  2014-08-27 11:12 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/1823df83c30e

8056062: Additional minor cleanups from source restructure build changes
Reviewed-by: alanb, tbell

! Makefile
! make/Main.gmk

Changeset: 0ceccf7b30b2
Author:erikj
Date:  2014-08-27 11:19 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/0ceccf7b30b2

8055922: Work around sjavac limitation with public api tracking cross modules
Reviewed-by: ihse

! make/common/JavaCompilation.gmk

Changeset: 2a1d4d3e8c8e
Author:simonis
Date:  2014-08-29 12:02 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/2a1d4d3e8c8e

8056246: Fix AIX build after the Modular Source Code change 8054834
Reviewed-by: erikj, ihse

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

Changeset: e0a080fd4047
Author:erikj
Date:  2014-08-29 13:30 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/e0a080fd4047

8014510: Fix sjavac on all platforms in jprt
Reviewed-by: ihse

! common/autoconf/boot-jdk.m4
! common/autoconf/build-performance.m4
! common/autoconf/configure.ac
! common/autoconf/generated-configure.sh

Changeset: 3736cf40d962
Author:m

hg: jigsaw/m2/corba: 3 new changesets

[erik . joelsson]

Changeset: 1f5939bac4ae
Author:erikj
Date:  2014-08-27 10:59 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/1f5939bac4ae

8056064: Fix corba locale build problem on windows
Reviewed-by: alanb, tbell

! make/GensrcCorba.gmk

Changeset: 163a9cd806fd
Author:lana
Date:  2014-08-29 11:59 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/163a9cd806fd

Merge


Changeset: 98967ae6ae53
Author:katleman
Date:  2014-09-04 13:26 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/98967ae6ae53

Added tag jdk9-b29 for changeset 163a9cd806fd

! .hgtags

hg: jigsaw/m2/hotspot: 53 new changesets

[erik . joelsson]

Changeset: 16286b7d7c6e
Author:iklam
Date:  2014-08-12 17:29 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/16286b7d7c6e

8046070: Class Data Sharing clean up and refactoring
Summary: Cleaned up CDS to be more configurable, maintainable and extensible
Reviewed-by: dholmes, coleenp, acorn, mchung

! make/excludeSrc.make
! src/os/linux/vm/os_linux.cpp
! src/share/vm/classfile/classFileParser.cpp
! src/share/vm/classfile/classLoader.cpp
! src/share/vm/classfile/classLoader.hpp
+ src/share/vm/classfile/classLoaderExt.hpp
! src/share/vm/classfile/dictionary.cpp
! src/share/vm/classfile/dictionary.hpp
+ src/share/vm/classfile/sharedClassUtil.hpp
+ src/share/vm/classfile/sharedPathsMiscInfo.cpp
+ src/share/vm/classfile/sharedPathsMiscInfo.hpp
! src/share/vm/classfile/systemDictionary.cpp
! src/share/vm/classfile/systemDictionary.hpp
+ src/share/vm/classfile/systemDictionaryShared.hpp
! src/share/vm/classfile/vmSymbols.hpp
! src/share/vm/memory/allocation.hpp
! src/share/vm/memory/filemap.cpp
! src/share/vm/memory/filemap.hpp
! src/share/vm/memory/metadataFactory.hpp
! src/share/vm/memory/metaspace.cpp
! src/share/vm/memory/metaspace.hpp
! src/share/vm/memory/metaspaceShared.cpp
! src/share/vm/memory/metaspaceShared.hpp
! src/share/vm/memory/universe.cpp
! src/share/vm/oops/arrayKlass.cpp
! src/share/vm/oops/arrayKlass.hpp
! src/share/vm/oops/instanceKlass.cpp
! src/share/vm/oops/instanceKlass.hpp
! src/share/vm/oops/klass.cpp
! src/share/vm/oops/klass.hpp
! src/share/vm/prims/jvm.cpp
! src/share/vm/runtime/arguments.cpp
! src/share/vm/runtime/arguments.hpp
! src/share/vm/runtime/globals.hpp
! src/share/vm/runtime/javaCalls.cpp
! src/share/vm/runtime/vmStructs.cpp
! src/share/vm/utilities/exceptions.cpp
! src/share/vm/utilities/ostream.cpp
! src/share/vm/utilities/ostream.hpp
+ src/share/vm/utilities/stringUtils.cpp
+ src/share/vm/utilities/stringUtils.hpp
+ test/testlibrary/com/oracle/java/testlibrary/BuildHelper.java

Changeset: 72030160367a
Author:zgu
Date:  2014-08-14 09:02 -0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/72030160367a

8054368: nsk/jdi/VirtualMachine/exit/exit002 crash with detail tracking on 
(NMT2)
Summary: Dynamic allocate _reserved_regions instead of static object to avoid 
racing during process exit
Reviewed-by: dholmes, coleenp

! src/share/vm/services/memTracker.cpp
! src/share/vm/services/virtualMemoryTracker.cpp
! src/share/vm/services/virtualMemoryTracker.hpp

Changeset: 8a290a0d2153
Author:zgu
Date:  2014-08-14 13:13 +
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/8a290a0d2153

Merge


Changeset: 01e6096da640
Author:zgu
Date:  2014-08-14 13:15 -0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/01e6096da640

8054547: Re-enable warning for incompatible java launcher
Summary: Re-enabled warning as launcher change reached promotion build
Reviewed-by: hseigel, coleenp

! src/share/vm/runtime/arguments.cpp

Changeset: 4a937a64c20b
Author:zgu
Date:  2014-08-14 17:25 +
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/4a937a64c20b

Merge

! src/share/vm/runtime/arguments.cpp

Changeset: 80b831308478
Author:ctornqvi
Date:  2014-08-14 18:14 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/80b831308478

8032999: [TESTBUG] JT-Reg Runtime tests to be run as part of JPRT submit job
Summary: JPRT will now run most of the tests in /runtime
Reviewed-by: dcubed, mseledtsov, zgu, mikael

! make/jprt.properties
! test/TEST.groups
! test/runtime/7158988/FieldMonitor.java
! test/runtime/CompressedOops/CompressedClassPointers.java
! test/runtime/jsig/Test8017498.sh

Changeset: af24453bcd3e
Author:sspitsyn
Date:  2014-08-15 11:25 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/af24453bcd3e

Merge

! src/share/vm/runtime/arguments.cpp
! src/share/vm/runtime/globals.hpp

Changeset: ce9fb9e64334
Author:coleenp
Date:  2014-08-15 15:25 -0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/ce9fb9e64334

8055231: ZERO variant build is broken
Summary: Fix zero build.
Reviewed-by: coleenp
Contributed-by: Severin Gehwolf 

! src/cpu/zero/vm/cppInterpreter_zero.cpp
! src/cpu/zero/vm/frame_zero.inline.hpp
! src/share/vm/interpreter/bytecodeInterpreter.cpp

Changeset: 03b8d901bdea
Author:coleenp
Date:  2014-08-15 20:37 +
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/03b8d901bdea

Merge


Changeset: c346a7739aee
Author:zgu
Date:  2014-08-19 08:34 -0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/c346a7739aee

8055007: NMT2: emptyStack missing in minimal build
Summary: Refactored emptyStack to a static member of NativeCallStack, which is 
accessible in minimal build.
Reviewed-by: coleenp, dholmes

! src/share/vm/services/mallocSiteTable.hpp
! src/share/vm/services/memTracker.cpp
! src/share/vm/services/memTracker.hpp
! src/share/vm/services/nmtCommon.hpp
! src/share/vm/services/vir

hg: jigsaw/m2/jaxp: 3 new changesets

[erik . joelsson]

Changeset: 65e6291d9ba9
Author:joehw
Date:  2014-08-29 11:49 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxp/rev/65e6291d9ba9

8037819: Xerces Update: jaxp/validation/XMLSchemaFactory
Reviewed-by: lancea

! 
src/java.xml/share/classes/com/sun/org/apache/xerces/internal/impl/Constants.java
! 
src/java.xml/share/classes/com/sun/org/apache/xerces/internal/impl/dv/ValidatedInfo.java
! 
src/java.xml/share/classes/com/sun/org/apache/xerces/internal/impl/msg/XMLSchemaMessages.properties
+ 
src/java.xml/share/classes/com/sun/org/apache/xerces/internal/impl/validation/ConfigurableValidationState.java
! 
src/java.xml/share/classes/com/sun/org/apache/xerces/internal/impl/xs/XMLSchemaValidator.java
! 
src/java.xml/share/classes/com/sun/org/apache/xerces/internal/jaxp/validation/XMLSchema.java
! 
src/java.xml/share/classes/com/sun/org/apache/xerces/internal/jaxp/validation/XMLSchemaFactory.java
! 
src/java.xml/share/classes/com/sun/org/apache/xerces/internal/jaxp/validation/XMLSchemaValidatorComponentManager.java
! 
src/java.xml/share/classes/com/sun/org/apache/xerces/internal/parsers/StandardParserConfiguration.java
! 
src/java.xml/share/classes/com/sun/org/apache/xerces/internal/parsers/XML11Configuration.java

Changeset: 30adcd13a313
Author:lana
Date:  2014-08-29 11:56 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxp/rev/30adcd13a313

Merge


Changeset: d181d4002214
Author:katleman
Date:  2014-09-04 13:26 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxp/rev/d181d4002214

Added tag jdk9-b29 for changeset 30adcd13a313

! .hgtags

hg: jigsaw/m2/jaxws: Added tag jdk9-b29 for changeset 3d1a4bfb6abb

[erik . joelsson]

Changeset: e58d3ea638c3
Author:katleman
Date:  2014-09-04 13:26 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxws/rev/e58d3ea638c3

Added tag jdk9-b29 for changeset 3d1a4bfb6abb

! .hgtags

hg: jigsaw/m2/jdk: 41 new changesets

[erik . joelsson]

Changeset: 5aea1174ff94
Author:martin
Date:  2014-08-20 11:06 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/5aea1174ff94

8055675: java/util/Currency/PropertiesTest.sh fails on OS X after JDK-8055253
Summary: Replace obsolescent cp -r with cp -R.  Add more error checks.
Reviewed-by: naoto

! test/java/util/Currency/PropertiesTest.sh

Changeset: 0aa1bc5db42c
Author:bpb
Date:  2014-08-20 11:30 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/0aa1bc5db42c

8054720: Modifications of I/O methods for instrumentation purposes
Summary: Wrap some native methods in Java methods.
Reviewed-by: rriggs

! make/mapfiles/libjava/mapfile-vers
! make/mapfiles/libjava/reorder-sparc
! make/mapfiles/libjava/reorder-sparcv9
! make/mapfiles/libjava/reorder-x86
! src/java.base/share/classes/java/io/FileInputStream.java
! src/java.base/share/classes/java/io/FileOutputStream.java
! src/java.base/share/classes/java/io/RandomAccessFile.java
! src/java.base/share/classes/java/net/SocketInputStream.java
! src/java.base/share/native/libjava/FileInputStream.c
! src/java.base/share/native/libjava/RandomAccessFile.c
! src/java.base/unix/native/libjava/FileOutputStream_md.c
! src/java.base/windows/native/libjava/FileOutputStream_md.c

Changeset: b8f27b54cc51
Author:erikj
Date:  2014-08-21 11:05 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/b8f27b54cc51

8055188: General cleanup of minor issues from source restructure
Reviewed-by: tbell

! make/Bundles.gmk
! make/CompileDemos.gmk
! make/CompileInterimRmic.gmk
- make/CopyIntoClasses.gmk
! make/CreateJars.gmk
! make/CreateSecurityJars.gmk
! make/Tools.gmk
- make/bundle/JDK-Info.plist
- make/bundle/JRE-Info.plist
! make/copy/Copy-java.base.gmk
! make/copy/Copy-java.desktop.gmk
! make/copy/Copy-java.logging.gmk
! make/copy/Copy-java.management.gmk
! make/copy/Copy-jdk.hprof.agent.gmk
! make/copy/Copy-jdk.jdwp.agent.gmk
! make/copy/CopyCommon.gmk
+ make/data/bundle/JDK-Info.plist
+ make/data/bundle/JRE-Info.plist
! make/gendata/Gendata-java.base.gmk
! make/gendata/GendataCommon.gmk
! make/gensrc/GensrcCommon.gmk
! make/gensrc/GensrcSwing.gmk
! make/lib/Awt2dLibraries.gmk
! make/lib/CoreLibraries.gmk

Changeset: 123ee720e95a
Author:erikj
Date:  2014-08-21 12:23 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/123ee720e95a

8055095: Improve "do nothing" incremental build performance after modularized 
source code integration
Reviewed-by: tbell

! make/Tools.gmk
! make/gensrc/GensrcLocaleDataMetaInfo.gmk
! make/gensrc/GensrcProperties.gmk
! make/lib/Lib-java.base.gmk
! make/lib/Lib-java.desktop.gmk
! make/lib/LibCommon.gmk

Changeset: b9572a8c4e1a
Author:sla
Date:  2014-08-21 13:09 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/b9572a8c4e1a

8055677: java/lang/instrument/RedefineBigClass.sh RetransformBigClass.sh start 
failing after JDK-8055012
Summary: Write dcmd output to separate files so it does not confuse the output.
Reviewed-by: ctornqvi, mgronlun

! test/java/lang/instrument/NMTHelper.java
! test/java/lang/instrument/RedefineBigClass.sh
! test/java/lang/instrument/RetransformBigClass.sh

Changeset: 3d879b5f8137
Author:mchung
Date:  2014-08-21 22:07 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/3d879b5f8137

8055393: [Testbug] Some tests are being executed and fail under profiles
Reviewed-by: mchung, dholmes
Contributed-by: evgeniya.stepan...@oracle.com

! test/TEST.groups
! test/java/lang/ProcessBuilder/InheritIO/InheritIO.sh

Changeset: ec20e30bba4c
Author:jbachorik
Date:  2014-08-21 14:36 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/ec20e30bba4c

8040692: [TESTBUG] sun/management/jmxremote/bootstrap/JvmstatCountersTest.java 
requires -XX:+UsePerfData option to pass on embedded platforms
Reviewed-by: sla

! test/sun/management/HotspotClassLoadingMBean/GetClassInitializationTime.java
! test/sun/management/HotspotRuntimeMBean/GetSafepointSyncTime.java
! test/sun/management/HotspotThreadMBean/GetInternalThreads.java
! test/sun/management/jmxremote/bootstrap/JvmstatCountersTest.java
! test/sun/management/jmxremote/startstop/JMXStartStopTest.java

Changeset: 6931bc291000
Author:igerasim
Date:  2014-08-23 01:05 +0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/6931bc291000

8055731: sun/security/smartcardio/TestDirect.java throws 
java.lang.IndexOutOfBoundsException
Reviewed-by: valeriep

! test/sun/security/smartcardio/TestDirect.java

Changeset: 74078474d9bd
Author:mchung
Date:  2014-08-22 14:56 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/74078474d9bd

8042003: Update java/lang/Math tests to eliminate dependency on 
sun.misc.DoubleConsts and sun.misc.FloatConsts
Reviewed-by: mchung, darcy
Contributed-by: amy...@oracle.com

! test/java/lang/Double/BitwiseConversion.java
! test/java/lang/Double/ParseHexFloatingPoint.java
! test/java/lang/Double/ToHexString.java
! test/java/lang/Float/Bitwise

hg: jigsaw/m2/nashorn: 21 new changesets

[erik . joelsson]

Changeset: 3516182f0612
Author:sundar
Date:  2014-08-20 21:32 +0530
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/3516182f0612

8055687: Wrong "this" passed to JSObject.eval call
Reviewed-by: lagergren, hannesw, attila

! samples/find_nonfinals.js
+ samples/find_nonfinals2.js
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/api/scripting/ScriptObjectMirror.java
! test/src/jdk/nashorn/api/scripting/ScriptObjectMirrorTest.java

Changeset: 127a155b49c8
Author:sundar
Date:  2014-08-21 20:06 +0530
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/127a155b49c8

8055762: Nashorn misses linker for netscape.javascript.JSObject instances
Reviewed-by: lagergren, jlaskey

! make/build.xml
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/linker/Bootstrap.java
+ 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/linker/BrowserJSObjectLinker.java
+ test/script/basic/JDK-8055762.js
+ test/script/basic/JDK-8055762.js.EXPECTED

Changeset: ee00439d1d54
Author:sundar
Date:  2014-08-22 15:47 +0530
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/ee00439d1d54

8055796: JSObject and browser JSObject linkers should provide fallback to call 
underlying Java methods directly
Reviewed-by: attila, hannesw

! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/linker/Bootstrap.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/linker/BrowserJSObjectLinker.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/linker/JSObjectLinker.java
+ test/script/basic/JDK-8055796.js
+ test/script/basic/JDK-8055796.js.EXPECTED

Changeset: 0f91116bb4bd
Author:sundar
Date:  2014-08-22 22:50 +0530
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/0f91116bb4bd

8055830: JDK-8015969.js is silently failing
Reviewed-by: lagergren, jlaskey, hannesw

! test/script/basic/JDK-8015969.js

Changeset: 11e4f12a45b0
Author:yan
Date:  2014-08-25 17:38 +0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/11e4f12a45b0

8055878: Nashorn: all tests failed with AccessControlException
Reviewed-by: sundar, hannesw
Contributed-by: Sergey Lugovoy 

! make/build.xml

Changeset: ad7a46083364
Author:sundar
Date:  2014-08-25 20:26 +0530
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/ad7a46083364

8055899: Two nashorn tests fail in 8u40 nightly build with 
ClassNotFoundException
Reviewed-by: jlaskey, lagergren

! test/script/basic/JDK-8055762.js
! test/script/basic/JDK-8055796.js
! test/script/basic/JDK-8055796.js.EXPECTED
+ test/script/basic/JDK-8055796_2.js
+ test/script/basic/JDK-8055796_2.js.EXPECTED

Changeset: 494092ee7a01
Author:attila
Date:  2014-08-25 22:36 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/494092ee7a01

8055870: iteration fails if index var is not used
Reviewed-by: hannesw, lagergren

! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/CodeGenerator.java
+ test/script/basic/JDK-8055870.js
+ test/script/basic/JDK-8055870.js.EXPECTED

Changeset: d6b5c2c6c1d0
Author:yan
Date:  2014-08-26 13:26 +0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/d6b5c2c6c1d0

8055811: Tests for Nashorn ClassFilter Support
Reviewed-by: sundar, attila
Contributed-by: Sergey Lugovoy 

+ test/script/trusted/classfilter_extends.js
+ test/script/trusted/classfilter_extends.js.EXPECTED
+ test/script/trusted/classfilter_mozilla_compat.js
+ test/script/trusted/classfilter_mozilla_compat.js.EXPECTED
+ test/src/jdk/nashorn/internal/runtime/ClassFilterTest.java

Changeset: 10c95d040380
Author:attila
Date:  2014-08-26 11:31 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/10c95d040380

8055911: Don't use String.intern for IdentNode
Reviewed-by: lagergren, sundar

! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/IdentNode.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/parser/AbstractParser.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/parser/Parser.java

Changeset: 9388010e3a8e
Author:attila
Date:  2014-08-26 11:32 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/9388010e3a8e

8055913: Node.hashCode() delegates to Object.hashCode() and is hot
Reviewed-by: lagergren, sundar

! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/Node.java

Changeset: f03ee361fa76
Author:lagergren
Date:  2014-08-26 11:42 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/f03ee361fa76

8055906: Avoid throwing an exception with filled in stack trace as part of the 
normal control flow
Reviewed-by: attila, hannesw

! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/ApplySpecialization.java

Changeset: d81f7c897bdf
Author:attila
Date:  2014-08-26 11:45 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/d81f7c897bdf

8055923: colle

hg: jigsaw/m2/langtools: 16 new changesets

[erik . joelsson]

Changeset: bad77727fa11
Author:erikj
Date:  2014-08-22 12:25 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/bad77727fa11

8055767: Sjavac is leaking servers
Reviewed-by: alundblad, jfranck

! src/jdk.compiler/share/classes/com/sun/tools/sjavac/server/SjavacServer.java

Changeset: 50b448c5be54
Author:jjg
Date:  2014-08-22 16:28 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/50b448c5be54

8044859: javac duplicates option processing when using Compiler API
Reviewed-by: mcimadamore, vromero, jlahoda

! make/build.properties
! src/jdk.compiler/share/classes/com/sun/source/util/JavacTask.java
! src/jdk.compiler/share/classes/com/sun/tools/doclint/DocLint.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/api/BasicJavacTask.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/api/JavacTaskImpl.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/api/JavacTool.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/api/JavacTrees.java
+ src/jdk.compiler/share/classes/com/sun/tools/javac/main/Arguments.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/main/JavaCompiler.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/main/Main.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/main/Option.java
! test/tools/javac/6410653/T6410653.java
! test/tools/javac/6457284/T6457284.java
! test/tools/javac/T6358166.java
! test/tools/javac/T6358168.java
! test/tools/javac/T8009640/CheckRejectProfileBCPOptionsIfUsedTogetherTest.java
! test/tools/javac/api/6410643/T6410643.java
! test/tools/javac/api/T6357331.java
! test/tools/javac/api/TestJavacTaskScanner.java
! test/tools/javac/api/TestJavacTask_Lock.java
! test/tools/javac/api/TestResolveError.java
! test/tools/javac/api/taskListeners/CompileEvent.java
! test/tools/javac/api/taskListeners/EventsBalancedTest.java
! test/tools/javac/failover/CheckAttributedTree.java
+ test/tools/javac/options/modes/AtFilesTest.java
+ test/tools/javac/options/modes/DocLintTest.java
+ test/tools/javac/options/modes/FSInfoTest.java
+ test/tools/javac/options/modes/InfoOptsTest.java
+ test/tools/javac/options/modes/NoOperandsTest.java
+ test/tools/javac/options/modes/OutputDirTest.java
+ test/tools/javac/options/modes/ProfileBootClassPathTest.java
+ test/tools/javac/options/modes/ProfileTargetTest.java
+ test/tools/javac/options/modes/SourceTargetTest.java
+ test/tools/javac/options/modes/StdOutTest.java
+ test/tools/javac/options/modes/Tester.java
! test/tools/javac/profiles/ProfileOptionTest.java
! test/tools/javac/versions/Versions.java

Changeset: f2518ce1dabc
Author:mcimadamore
Date:  2014-08-26 12:45 +0100
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/f2518ce1dabc

8056055: IntelliJ source paths broken after modularization of langtools
Summary: Add 'share/classes' prefix to all src roots
Reviewed-by: jfranck

! make/intellij/langtools.iml

Changeset: 8e638f046bf0
Author:jlahoda
Date:  2014-08-27 07:44 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/8e638f046bf0

8056061: Mark implementations of public interfaces with an annotation
Summary: Adding @DefinedBy annotation to mark methods that implement public API 
methods; annotating the methods; adding a coding rules analyzer to enforce all 
such methods are annotated.
Reviewed-by: jjg, mcimadamore, jfranck
Contributed-by: jan.lah...@oracle.com, jonathan.gibb...@oracle.com

+ make/test/crules/DefinedByAnalyzer/Test.java
+ make/test/crules/DefinedByAnalyzer/Test.out
! make/tools/crules/AbstractCodingRulesAnalyzer.java
! make/tools/crules/CodingRulesAnalyzerPlugin.java
+ make/tools/crules/DefinedByAnalyzer.java
! make/tools/crules/resources/crules.properties
! src/jdk.compiler/share/classes/com/sun/tools/doclint/Checker.java
! src/jdk.compiler/share/classes/com/sun/tools/doclint/DocLint.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/api/BasicJavacTask.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/api/ClientCodeWrapper.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/api/JavacScope.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/api/JavacTaskImpl.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/api/JavacTool.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/api/JavacTrees.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/api/MultiTaskListener.java
! 
src/jdk.compiler/share/classes/com/sun/tools/javac/api/WrappingJavaFileManager.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/code/AnnoConstruct.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Attribute.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symbol.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symtab.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Type.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Annotate.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Attr.java
! src/jdk.com

RFR: JDK-8058118: Generate modules.list during the build

[Erik Joelsson]

Hello,

The checked in modules.list file defines the dependencies between 
modules for the build. The dependency information in this file is 
already captured in the modules.xml. Rather than keeping two copies of 
this information, with this change, modules.list is generated during the 
build.


Chris provided the java implementation for the actual list generation. 
I've made the makefile changes.


Bug: https://bugs.openjdk.java.net/browse/JDK-8058118
Webrevs: http://cr.openjdk.java.net/~erikj/8058118/

/Erik

Re: RFR: JDK-8058118: Generate modules.list during the build

[Erik Joelsson]

Hello,

New webrevs with Chris' latest fixes and a few small modifications to 
makefiles. I removed some of the extra -I flags in Makefile and used the 
variable for all references to modules.list in Modules.gmk.


http://cr.openjdk.java.net/~erikj/8058118/webrev.root.02/
http://cr.openjdk.java.net/~erikj/8058118/webrev.jdk.02/

/Erik

On 2014-09-12 16:03, Erik Joelsson wrote:

Hello,

The checked in modules.list file defines the dependencies between 
modules for the build. The dependency information in this file is 
already captured in the modules.xml. Rather than keeping two copies of 
this information, with this change, modules.list is generated during 
the build.


Chris provided the java implementation for the actual list generation. 
I've made the makefile changes.


Bug: https://bugs.openjdk.java.net/browse/JDK-8058118
Webrevs: http://cr.openjdk.java.net/~erikj/8058118/

/Erik

hg: jigsaw/m2: 8 new changesets

[erik . joelsson]

Changeset: c4b3aa150cdf
Author:erikj
Date:  2014-09-09 17:30 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/c4b3aa150cdf

Changed makefiles in corba and langtools to modular organization

! make/Main.gmk
! make/MakeHelpers.gmk
! make/common/SetupJavaCompilers.gmk

Changeset: b86fbafe095a
Author:erikj
Date:  2014-09-10 11:18 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/b86fbafe095a

Moved buildtools to new outputdir

! common/autoconf/spec.gmk.in
! make/Main.gmk

Changeset: eb334cb7bdb4
Author:erikj
Date:  2014-09-10 14:47 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/eb334cb7bdb4

Moved gensrc to support outputdir

! common/autoconf/spec.gmk.in
! make/CompileJavaModules.gmk
! make/Javadoc.gmk
! make/Main.gmk
! make/common/JavaCompilation.gmk

Changeset: 7aa5595d7f44
Author:erikj
Date:  2014-09-11 14:27 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/7aa5595d7f44

Moved the rest of jdk output that didn't belong there to other output dirs

! make/CompileJavaModules.gmk
! make/Javadoc.gmk

Changeset: c4fedd528473
Author:erikj
Date:  2014-09-12 12:53 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/c4fedd528473

Moved lib, bin and include out of jdk outputdir to support/modules_*/$module

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

Changeset: 946f717812d7
Author:erikj
Date:  2014-09-12 16:42 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/946f717812d7

Automatic copying to jdk outputdir for exploded image

! make/Main.gmk
! make/MakeHelpers.gmk

Changeset: 6b30317f3f12
Author:erikj
Date:  2014-09-12 17:12 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/6b30317f3f12

Adding missed file ModuleWrapper.gmk

+ make/ModuleWrapper.gmk

Changeset: 88005086afe6
Author:erikj
Date:  2014-09-15 15:33 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/88005086afe6

Fixed build on windows

! make/ModuleWrapper.gmk

hg: jigsaw/m2/corba: 5 new changesets

[erik . joelsson]

Changeset: 236d821dc623
Author:erikj
Date:  2014-09-09 17:30 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/236d821dc623

Changed makefiles in corba and langtools to modular organization

- make/CommonCorba.gmk
- make/CompileCorba.gmk
+ make/CompileInterim.gmk
- make/GensrcCorba.gmk
+ make/copy/Copy-java.corba.gmk
+ make/gensrc/Gensrc-java.corba.gmk

Changeset: caef75bfbf98
Author:erikj
Date:  2014-09-10 11:18 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/caef75bfbf98

Moved buildtools to new outputdir

! make/CompileInterim.gmk
! make/gensrc/Gensrc-java.corba.gmk

Changeset: 29c78e8a2428
Author:erikj
Date:  2014-09-10 14:47 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/29c78e8a2428

Moved gensrc to support outputdir

! make/CompileInterim.gmk
! make/gensrc/Gensrc-java.corba.gmk

Changeset: 04d944d64906
Author:erikj
Date:  2014-09-12 12:53 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/04d944d64906

Moved lib, bin and include out of jdk outputdir to support/modules_*/$module

! make/copy/Copy-java.corba.gmk

Changeset: 02c4212c9f0a
Author:erikj
Date:  2014-09-12 16:42 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/02c4212c9f0a

Automatic copying to jdk outputdir for exploded image

! make/copy/Copy-java.corba.gmk

hg: jigsaw/m2/jdk: 7 new changesets

[erik . joelsson]

Changeset: 07d0ae401d4c
Author:erikj
Date:  2014-09-10 11:18 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/07d0ae401d4c

Moved buildtools to new outputdir

! make/CompileInterimRmic.gmk
! make/Tools.gmk
! make/gensrc/GensrcIcons.gmk
! make/gensrc/GensrcMisc.gmk
! make/gensrc/GensrcSwing.gmk
! make/gensrc/GensrcX11Wrappers.gmk
! make/rmic/RmicCommon.gmk

Changeset: fb4e1d876548
Author:erikj
Date:  2014-09-10 14:47 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/fb4e1d876548

Moved gensrc to support outputdir

! make/CreateJars.gmk
! make/ProfileNames.gmk
! make/gensrc/Gensrc-jdk.jconsole.gmk
! make/gensrc/GensrcBuffer.gmk
! make/gensrc/GensrcCLDR.gmk
! make/gensrc/GensrcCharacterData.gmk
! make/gensrc/GensrcCharsetCoder.gmk
! make/gensrc/GensrcCharsetMapping.gmk
! make/gensrc/GensrcExceptions.gmk
! make/gensrc/GensrcIcons.gmk
! make/gensrc/GensrcJDWP.gmk
! make/gensrc/GensrcLocaleDataMetaInfo.gmk
! make/gensrc/GensrcMisc.gmk
! make/gensrc/GensrcProperties.gmk
! make/gensrc/GensrcProviders.gmk
! make/gensrc/GensrcSwing.gmk
! make/gensrc/GensrcX11Wrappers.gmk
! make/lib/Lib-jdk.jdwp.agent.gmk
! make/lib/PlatformLibraries.gmk

Changeset: 250f4b4797f5
Author:erikj
Date:  2014-09-11 14:27 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/250f4b4797f5

Moved the rest of jdk output that didn't belong there to other output dirs

! make/CompileDemos.gmk
! make/CopySamples.gmk
! make/CreateJars.gmk
! make/CreatePolicyJars.gmk
! make/CreateSecurityJars.gmk
! make/Images.gmk
! make/gendata/GendataBreakIterator.gmk
! make/gensrc/GensrcSwing.gmk
! make/launcher/Launcher-java.base.gmk
! make/launcher/Launcher-jdk.runtime.gmk
! make/launcher/LauncherCommon.gmk
! make/lib/Awt2dLibraries.gmk
! make/lib/CoreLibraries.gmk
! make/lib/Lib-java.instrument.gmk
! make/lib/Lib-java.management.gmk
! make/lib/Lib-java.prefs.gmk
! make/lib/Lib-java.security.jgss.gmk
! make/lib/Lib-java.smartcardio.gmk
! make/lib/Lib-jdk.attach.gmk
! make/lib/Lib-jdk.crypto.ec.gmk
! make/lib/Lib-jdk.crypto.mscapi.gmk
! make/lib/Lib-jdk.crypto.pkcs11.gmk
! make/lib/Lib-jdk.deploy.osx.gmk
! make/lib/Lib-jdk.hprof.agent.gmk
! make/lib/Lib-jdk.jdi.gmk
! make/lib/Lib-jdk.jdwp.agent.gmk
! make/lib/Lib-jdk.runtime.gmk
! make/lib/Lib-jdk.sctp.gmk
! make/lib/Lib-jdk.security.auth.gmk
! make/lib/LibCommon.gmk
! make/lib/NetworkingLibraries.gmk
! make/lib/NioLibraries.gmk
! make/lib/PlatformLibraries.gmk
! make/lib/SoundLibraries.gmk
! make/rmic/Rmic-java.management.gmk
! make/rmic/RmicCommon.gmk

Changeset: 0ea13b82cdd2
Author:erikj
Date:  2014-09-12 12:53 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/0ea13b82cdd2

Moved lib, bin and include out of jdk outputdir to support/modules_*/$module

! make/CreatePolicyJars.gmk
! make/Import.gmk
! make/copy/Copy-java.base.gmk
! make/copy/Copy-java.desktop.gmk
! make/copy/CopyCommon.gmk
! make/gendata/Gendata-java.base.gmk
! make/gendata/GendataBlacklistedCerts.gmk
! make/gendata/GendataFontConfig.gmk
! make/gendata/GendataTZDB.gmk
! make/launcher/Launcher-java.base.gmk
! make/launcher/Launcher-java.rmi.gmk
! make/launcher/Launcher-jdk.runtime.gmk
! make/launcher/LauncherCommon.gmk
! make/lib/Awt2dLibraries.gmk
! make/lib/Lib-java.instrument.gmk
! make/lib/Lib-java.management.gmk
! make/lib/Lib-java.prefs.gmk
! make/lib/Lib-jdk.attach.gmk
! make/lib/Lib-jdk.deploy.osx.gmk
! make/lib/Lib-jdk.jdwp.agent.gmk
! make/lib/Lib-jdk.runtime.gmk
! make/lib/Lib-jdk.sctp.gmk
! make/lib/Lib-jdk.security.auth.gmk
! make/lib/LibCommon.gmk

Changeset: 7007eee6d284
Author:erikj
Date:  2014-09-12 16:42 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/7007eee6d284

Automatic copying to jdk outputdir for exploded image

! make/CreateJars.gmk
! make/Import.gmk
! make/copy/Copy-java.base.gmk
! make/copy/Copy-java.desktop.gmk
! make/copy/Copy-java.logging.gmk
! make/copy/Copy-java.management.gmk
! make/copy/Copy-jdk.hprof.agent.gmk
! make/copy/Copy-jdk.jdwp.agent.gmk
! make/copy/CopyCommon.gmk
! make/gendata/Gendata-java.base.gmk
! make/gendata/Gendata-java.desktop.gmk
! make/gendata/Gendata-jdk.dev.gmk
! make/gendata/GendataCommon.gmk
= make/gendata/GendataPolicyJars.gmk < make/CreatePolicyJars.gmk
! make/launcher/Launcher-java.base.gmk
! make/launcher/Launcher-java.corba.gmk
! make/launcher/Launcher-java.desktop.gmk
! make/launcher/Launcher-java.rmi.gmk
! make/launcher/Launcher-java.scripting.gmk
! make/launcher/Launcher-java.security.jgss.gmk
! make/launcher/Launcher-jdk.compiler.gmk
! make/launcher/Launcher-jdk.dev.gmk
! make/launcher/Launcher-jdk.hotspot.agent.gmk
! make/launcher/Launcher-jdk.javadoc.gmk
! make/launcher/Launcher-jdk.jcmd.gmk
! make/launcher/Launcher-jdk.jconsole.gmk
! make/launcher/Launcher-jdk.jdi.gmk
! make/launcher/Launcher-jdk.jvmstat.gmk
! make/launcher/Launcher-jdk.rmic.gmk
! make/launcher/Launcher-jdk.runtime.gmk
! make/launcher/Launcher-jdk.scripting.nashorn.gmk
! make/launcher/Launcher-jdk.xml.bind.gmk
! m

hg: jigsaw/m2/langtools: 3 new changesets

[erik . joelsson]

Changeset: 45bea920d95d
Author:erikj
Date:  2014-09-09 17:30 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/45bea920d95d

Changed makefiles in corba and langtools to modular organization

- make/CommonLangtools.gmk
! make/CompileInterim.gmk
- make/GensrcLangtools.gmk
+ make/gensrc/Gensrc-jdk.compiler.gmk
+ make/gensrc/Gensrc-jdk.dev.gmk
+ make/gensrc/Gensrc-jdk.javadoc.gmk
+ make/gensrc/GensrcCommon.gmk

Changeset: 1f58e2ed38b0
Author:erikj
Date:  2014-09-10 11:18 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/1f58e2ed38b0

Moved buildtools to new outputdir

! make/CompileInterim.gmk
! make/gensrc/GensrcCommon.gmk

Changeset: 8adab9676927
Author:erikj
Date:  2014-09-10 14:47 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/8adab9676927

Moved gensrc to support outputdir

! make/gensrc/GensrcCommon.gmk

hg: jigsaw/m2/corba: 2 new changesets

[erik . joelsson]

Changeset: 3aa9b894892f
Author:katleman
Date:  2014-09-11 09:08 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/3aa9b894892f

Added tag jdk9-b30 for changeset 98967ae6ae53

! .hgtags

Changeset: 880bc27dff9b
Author:erikj
Date:  2014-09-15 16:47 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/880bc27dff9b

Merge

hg: jigsaw/m2: 7 new changesets

[erik . joelsson]

Changeset: 0610b52412cf
Author:ihse
Date:  2014-09-01 10:39 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/0610b52412cf

8027627: Don't hardcode bash path in LOG=trace
Reviewed-by: erikj, tbell
Contributed-by: Dmitry Samersoff 

! make/common/MakeBase.gmk

Changeset: d6cb4cea9c54
Author:prr
Date:  2014-08-22 11:00 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/d6cb4cea9c54

8039271: CMM profile files (cmm/*) should not be in ${java.home}/lib
Reviewed-by: bae, mchung, erikj

! make/CompileJavaModules.gmk

Changeset: b3e42a2a9619
Author:prr
Date:  2014-09-02 11:21 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/b3e42a2a9619

Merge

! make/CompileJavaModules.gmk

Changeset: ce36ddb6dc61
Author:erikj
Date:  2014-09-03 14:10 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/ce36ddb6dc61

8057132: Build fails if PROFILE is set in the environment
Reviewed-by: ihse, dholmes, fparain

! make/Main.gmk

Changeset: 36e9bc875325
Author:lana
Date:  2014-09-04 14:44 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/36e9bc875325

Merge


Changeset: 65f52e02bc18
Author:katleman
Date:  2014-09-11 09:08 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/65f52e02bc18

Added tag jdk9-b30 for changeset 36e9bc875325

! .hgtags

Changeset: cc6a19aa92cb
Author:erikj
Date:  2014-09-15 16:47 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/cc6a19aa92cb

Merge

! make/CompileJavaModules.gmk
! make/Main.gmk

hg: jigsaw/m2/hotspot: 46 new changesets

[erik . joelsson]

Changeset: 91f25b8a47af
Author:zgu
Date:  2014-08-20 08:41 -0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/91f25b8a47af

8054546: NMT2 leaks memory
Summary: Fixed memory leak in NMT by baselining memory in c heap instead of an 
arena.
Reviewed-by: coleenp, minqi

! src/share/vm/services/mallocTracker.hpp
! src/share/vm/services/memBaseline.cpp
! src/share/vm/services/memBaseline.hpp

Changeset: 6ae3b7da7952
Author:zgu
Date:  2014-08-20 08:50 -0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/6ae3b7da7952

Merge


Changeset: f2417008f618
Author:zgu
Date:  2014-08-20 15:58 -0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/f2417008f618

8055236: Deadlock during NMT2 shutdown on Windows
Summary: Removed MemTracker::shutdown() call to avoid deadlock
Reviewed-by: minqi, ctornqvi

! src/os/windows/vm/os_windows.cpp

Changeset: 115d958b07de
Author:zgu
Date:  2014-08-20 21:37 +
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/115d958b07de

Merge


Changeset: 943c5da523fd
Author:ctornqvi
Date:  2014-08-20 14:59 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/943c5da523fd

8055164: [TESTBUG] runtime/CompressedOops/CompressedClassPointers.java fails 
with OpenJDK build
Summary: Removed explicit check for HotSpot in output
Reviewed-by: sla, coleenp, hseigel

! test/runtime/CompressedOops/CompressedClassPointers.java

Changeset: 23270ceb0be7
Author:ctornqvi
Date:  2014-08-20 14:59 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/23270ceb0be7

8027480: Build Windows x64 fastdebug builds using /homeparams
Summary: Windows x64 fastdebug builds now uses the /homeparams compiler flag
Reviewed-by: dcubed, zgu, lfoltan

! make/windows/makefiles/vm.make
! src/share/tools/ProjectCreator/BuildConfig.java
! src/share/tools/ProjectCreator/WinGammaPlatformVC10.java
! src/share/tools/ProjectCreator/WinGammaPlatformVC7.java
! src/share/tools/ProjectCreator/WinGammaPlatformVC8.java

Changeset: b231e0ef5547
Author:ctornqvi
Date:  2014-08-21 01:43 +
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/b231e0ef5547

Merge


Changeset: af72c1a537dc
Author:igerasim
Date:  2014-08-21 16:32 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/af72c1a537dc

8055338: (process) Add instrumentation to help diagnose JDK-6573254
Reviewed-by: dcubed, ohair, iklam, dholmes, sspitsyn, sla

! src/os/windows/vm/os_windows.cpp
! src/share/vm/runtime/java.cpp

Changeset: f7df20ae08e2
Author:sla
Date:  2014-08-22 08:13 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/f7df20ae08e2

8055754: filemap.cpp does not compile with clang
Reviewed-by: kvn, iklam, coleenp

! src/share/vm/memory/filemap.cpp
! src/share/vm/memory/filemap.hpp

Changeset: b5c908c4e997
Author:ctornqvi
Date:  2014-08-22 09:30 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/b5c908c4e997

8055814: [TESTBUG] runtime/NMT/NMTWithCDS.java fails with product builds due to 
missing UnlockDiagnosticVMOptions
Summary: Passing -XX:+UnlockDiagnosticVMOptions to ProcessBuilder
Reviewed-by: zgu, gtriantafill

! test/runtime/NMT/NMTWithCDS.java

Changeset: 6c4b3632b42f
Author:dcubed
Date:  2014-08-22 11:23 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/6c4b3632b42f

Merge


Changeset: 8519a5cf7cd4
Author:gtriantafill
Date:  2014-08-22 06:46 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/8519a5cf7cd4

8055052: [TESTBUG] runtime/NMT/JcmdDetailDiff.java fails on Windows when there 
are no debug symbols available
Reviewed-by: ctornqvi, hseigel

! test/runtime/NMT/JcmdDetailDiff.java

Changeset: d764332b3890
Author:gtriantafill
Date:  2014-08-22 07:38 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/d764332b3890

8055053: [TESTBUG] runtime/NMT/VirtualAllocCommitUncommitRecommit.java fails
Reviewed-by: ctornqvi, zgu

! test/runtime/NMT/VirtualAllocCommitUncommitRecommit.java

Changeset: 2d87b6b4660c
Author:gtriantafill
Date:  2014-08-21 14:37 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/2d87b6b4660c

8055684: runtime/NMT/CommandLineEmptyArgument.java fails
Reviewed-by: ctornqvi, sla

! test/runtime/NMT/CommandLineEmptyArgument.java

Changeset: 04b4bc792043
Author:gtriantafill
Date:  2014-08-22 06:28 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/04b4bc792043

8055765: Misplaced @key stress prevents MallocSiteHashOverflow.java and 
MallocStressTest.java tests from running
Reviewed-by: ctornqvi, zgu, hseigel

! test/runtime/NMT/MallocSiteHashOverflow.java
! test/runtime/NMT/MallocStressTest.java

Changeset: 026e9d17ba9b
Author:ctornqvi
Date:  2014-08-22 14:21 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/026e9d17ba9b

8055844: [TESTBUG] test/runtime/NMT/VirtualAllocCommitUncommitRecommit.java 
fails on Solaris Sparc du

hg: jigsaw/m2/jaxp: Added tag jdk9-b30 for changeset d181d4002214

[erik . joelsson]

Changeset: c31f1eee7700
Author:katleman
Date:  2014-09-11 09:08 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxp/rev/c31f1eee7700

Added tag jdk9-b30 for changeset d181d4002214

! .hgtags

hg: jigsaw/m2/jaxws: Added tag jdk9-b30 for changeset e58d3ea638c3

[erik . joelsson]

Changeset: ddf1ef4a4a54
Author:katleman
Date:  2014-09-11 09:08 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxws/rev/ddf1ef4a4a54

Added tag jdk9-b30 for changeset e58d3ea638c3

! .hgtags

hg: jigsaw/m2/nashorn: 8 new changesets

[erik . joelsson]

Changeset: 2520d5e7bc5f
Author:yan
Date:  2014-09-01 17:34 +0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/2520d5e7bc5f

8054343: Nashorn: Some tests fails on windows with AccessControlException
Reviewed-by: lagergren, sundar
Contributed-by: Sergey Lugovoy 

! test/script/basic/compile-octane-normal.js
! test/script/basic/compile-octane-splitter.js
! test/script/basic/compile-octane.js

Changeset: 20d8ab569eb6
Author:attila
Date:  2014-09-03 14:33 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/20d8ab569eb6

8056913: Limit the size of type info cache on disk
Reviewed-by: jlaskey, lagergren

! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/OptimisticTypesPersistence.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/types/Type.java

Changeset: 0b7b3bd3cc04
Author:lagergren
Date:  2014-09-04 10:52 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/0b7b3bd3cc04

8057019: Various problems with extra arguments to applies
Reviewed-by: attila, hannesw, jlaskey

- bin/dump_octane_code.sh
- bin/fixorphantests.sh
- bin/fixwhitespace.sh
- bin/jjsdebug.sh
- bin/rm-non-tracked.sh
- bin/run_octane.sh
- bin/rundiff.sh
- bin/runopt.sh
- bin/runopt_noassert.sh
- bin/runopt_nojfr.sh
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/ScriptFunction.java
+ test/script/basic/JDK-8057019-2.js
+ test/script/basic/JDK-8057019-2.js.EXPECTED
+ test/script/basic/JDK-8057019-payload.js
+ test/script/basic/JDK-8057019.js
+ test/script/basic/JDK-8057019.js.EXPECTED
! test/script/basic/apply_to_call/apply_to_call4.js.EXPECTED

Changeset: ef3c1957d29b
Author:lagergren
Date:  2014-09-04 14:42 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/ef3c1957d29b

8057551: Let the -d flag dump _all_ generated classes to disk and work outside 
--compile-only mode
Reviewed-by: attila, jlaskey

! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/DumpBytecode.java

Changeset: 2f0161551858
Author:hannesw
Date:  2014-09-04 15:37 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/2f0161551858

8051889: Implement block scoping in symbol assignment and scope computation
Reviewed-by: attila, lagergren

! make/build.xml
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/AssignSymbols.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/CodeGenerator.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/Compiler.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/FieldObjectCreator.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/MapCreator.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/MethodEmitter.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/TypeEvaluator.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/FunctionNode.java
! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/IdentNode.java
! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/Symbol.java
! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/VarNode.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/parser/Parser.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/AccessorProperty.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/Context.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/FindProperty.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/Property.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/RecompilableScriptFunctionData.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/ScriptEnvironment.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/ScriptObject.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/ScriptRuntime.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/SetMethodCreator.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/linker/NashornCallSiteDescriptor.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/resources/Messages.properties
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/resources/Options.properties
! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/tools/Shell.java
+ test/script/basic/es6/block-function-decl.js
+ test/script/basic/es6/block-function-decl.js.EXPECTED
+ test/script/basic/es6/const-empty.js
+ test/script/basic/es6/const-empty.js.EXPECTED
+ test/script/basic/es6/const-reassign.js
+ test/script/basic/es6/const-reassign.js.EXPECTED
+ test/script/basic/es6/const-redeclare.js
+ test/script/basic/es6/const-redeclare.js.EXPECTED
+ test/script/basic/es6/const-self.js
+ test/sc

hg: jigsaw/m2/langtools: 10 new changesets

[erik . joelsson]

Changeset: 37e0edfac52e
Author:sogoel
Date:  2014-08-29 14:08 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/37e0edfac52e

8056288: Missing bug id in test/com/sun/javadoc/testOrdering/TestOrdering.java
Reviewed-by: ksrini

! test/com/sun/javadoc/testOrdering/TestOrdering.java

Changeset: 73b1d870a886
Author:mcimadamore
Date:  2014-09-01 17:41 +0100
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/73b1d870a886

8057005: IntelliJ should allow import for nested classes
Summary: Minor tweaks to langtools intellij project
Reviewed-by: jfranck

! make/intellij/codeStyleSettings.xml
! make/intellij/copyright/profiles_settings.xml
! make/intellij/workspace.xml

Changeset: c565b044f8c4
Author:kizune
Date:  2014-09-03 19:22 +0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/c565b044f8c4

8044597: Request to update tools/javap/T4501661.java to add test for package 
option
Reviewed-by: jjg

! test/tools/javap/T4501661.java

Changeset: 9a41fce5a626
Author:jjg
Date:  2014-09-03 14:06 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/9a41fce5a626

8034861: Incorrect format and indentation of InnerClasses section
Reviewed-by: jjg, ksrini, mcimadamore, lagergren

! src/jdk.compiler/share/classes/com/sun/tools/javap/AttributeWriter.java
! test/tools/javap/T4975569.java

Changeset: b33de166fafb
Author:jjg
Date:  2014-09-03 14:19 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/b33de166fafb

8038414: Constant pool's strings are not escaped properly
Reviewed-by: jjg, mcimadamore, ksrini

! src/jdk.compiler/share/classes/com/sun/tools/javap/ConstantWriter.java
+ test/tools/javap/T8038414.java

Changeset: 093d86cc0f8d
Author:jlahoda
Date:  2014-09-04 08:48 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/093d86cc0f8d

8054563: Update RunCodingRules.java for source code reorg
Summary: Tests RunCodingRules and MessageInfo fixed to work in the new source 
code structure.
Reviewed-by: jjg

! test/tools/all/RunCodingRules.java
! test/tools/javac/diags/MessageInfo.java

Changeset: d898543d13fb
Author:jlahoda
Date:  2014-09-04 08:49 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/d898543d13fb

8047675: tools/javac/defaultMethods/Assertions.java fails if run with 
-enableassertions (-ea)
Summary: Using ToolBox to start new Java processes, to avoid passing default VM 
options to the newly started process.
Reviewed-by: jjg

! test/tools/javac/defaultMethods/Assertions.java
+ test/tools/javac/defaultMethods/AssertionsTest.java
! test/tools/lib/ToolBox.java

Changeset: ef5427c13e1e
Author:lana
Date:  2014-09-04 14:45 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/ef5427c13e1e

Merge


Changeset: a4843ba8f2dc
Author:katleman
Date:  2014-09-11 09:09 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/a4843ba8f2dc

Added tag jdk9-b30 for changeset ef5427c13e1e

! .hgtags

Changeset: d81045b26dd5
Author:erikj
Date:  2014-09-15 16:47 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/d81045b26dd5

Merge

hg: jigsaw/m2/jdk: Fixed Solaris build

[erik . joelsson]

Changeset: e028066118ab
Author:erikj
Date:  2014-09-15 17:00 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/e028066118ab

Fixed Solaris build

! make/Tools.gmk

hg: jigsaw/m2/jdk: Fixed build on Macosx

[erik . joelsson]

Changeset: 2cf3fa848af7
Author:erikj
Date:  2014-09-15 18:16 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/2cf3fa848af7

Fixed build on Macosx

! make/launcher/LauncherCommon.gmk
! make/lib/Awt2dLibraries.gmk
! make/lib/Lib-jdk.deploy.osx.gmk

hg: jigsaw/m2: Fixed jdeps in interim langtools

[erik . joelsson]

Changeset: 96ea6781e81f
Author:erikj
Date:  2014-09-16 16:21 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/96ea6781e81f

Fixed jdeps in interim langtools

! make/Main.gmk

hg: jigsaw/m2/langtools: Fixed jdeps in interim langtools

[erik . joelsson]

Changeset: 6c2708eb4dca
Author:erikj
Date:  2014-09-16 16:21 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/6c2708eb4dca

Fixed jdeps in interim langtools

! make/CompileInterim.gmk
+ make/Tools.gmk

hg: jigsaw/m2: 11 new changesets

[erik . joelsson]

Changeset: dc29cb072e8a
Author:erikj
Date:  2014-09-05 12:42 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/dc29cb072e8a

8057537: Serialize reconfigure and fix make clean-foo foo
Reviewed-by: dholmes, ihse, tbell

! Makefile
! make/Main.gmk

Changeset: 16f1b2409462
Author:ihse
Date:  2014-09-09 15:12 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/16f1b2409462

8057125: The fix for JDK-8027627 was incomplete: Don't hardcode bash anywhere.
Reviewed-by: erikj

! make/common/MakeBase.gmk

Changeset: 5d7de212359d
Author:simonis
Date:  2014-09-09 17:42 +0100
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/5d7de212359d

8057538: Build the freetype library during configure on Windows
Reviewed-by: erikj, ihse

! common/autoconf/generated-configure.sh
! common/autoconf/help.m4
! common/autoconf/libraries.m4
! common/autoconf/toolchain.m4
! common/autoconf/toolchain_windows.m4

Changeset: fae1eb99785e
Author:coffeys
Date:  2014-09-10 16:13 +0100
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/fae1eb99785e

8057813: Alterations to jdk_security3 test target
Reviewed-by: mullan, wetmore, xuelei

! make/jprt.properties

Changeset: edbc7981f3be
Author:mduigou
Date:  2014-09-10 12:28 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/edbc7981f3be

8056195: Add CUSTOM_SUMMARY_AND_WARNINGS_HOOK
Reviewed-by: ihse, erikj

! common/autoconf/configure.ac
! common/autoconf/generated-configure.sh

Changeset: 9b44369c5737
Author:mchung
Date:  2014-09-10 17:32 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/9b44369c5737

8057627: Add org.w3c.dom.ranges and org.w3c.dom.traversal as exported API in 
modules.xml
Reviewed-by: alanb

! modules.xml

Changeset: 7047a1fd4aa8
Author:lana
Date:  2014-09-10 19:36 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/7047a1fd4aa8

Merge


Changeset: a1a2bb537718
Author:chegar
Date:  2014-09-11 10:29 +0100
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/a1a2bb537718

8058117: Missing jdk.deploy.osx from modules.xml
Reviewed-by: alanb, mchung

! modules.xml

Changeset: 69a84c16d9c2
Author:lana
Date:  2014-09-11 14:27 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/69a84c16d9c2

Merge


Changeset: 9918cd2de40d
Author:katleman
Date:  2014-09-17 22:55 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/9918cd2de40d

Added tag jdk9-b31 for changeset 69a84c16d9c2

! .hgtags

Changeset: 894e6e0aa6f4
Author:erikj
Date:  2014-09-18 11:35 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/894e6e0aa6f4

Merge

! common/autoconf/generated-configure.sh
! make/Main.gmk
! modules.xml

hg: jigsaw/m2/hotspot: 40 new changesets

[erik . joelsson]

Changeset: dc1a065e6989
Author:ccheung
Date:  2014-08-27 10:42 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/dc1a065e6989

8056175: Change "8048150: Allow easy configurations for large CDS archives" 
triggers conversion warning with older GCC
Summary: cast the result of the conversion to uintx
Reviewed-by: ccheung, coleenp
Contributed-by: volker.simo...@gmail.com

! src/share/vm/memory/metaspaceShared.hpp

Changeset: fe13911efa5b
Author:sla
Date:  2014-08-28 10:28 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/fe13911efa5b

8055755: Information about loaded dynamic libraries is wrong on MacOSX
Summary: The information about loaded dynamic libraries printed in hs_err_pid 
files or by running the jcmd VM.dynlib is partly incorrect. The address printed 
in front of the library file name is wrong.
Reviewed-by: sla, gziemski, gtriantafill
Contributed-by: fredrik.arvids...@oracle.com

! src/os/bsd/vm/os_bsd.cpp

Changeset: 92bc0d7679a7
Author:dsamersoff
Date:  2014-08-28 06:32 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/92bc0d7679a7

8031583: warnings from b03 for hotspot/agent/src/os/solaris/proc: JNI exception 
pending
Summary: Added missed exception checks
Reviewed-by: sla, sspitsyn

! agent/src/os/solaris/proc/saproc.cpp

Changeset: 68e9f36fd6c8
Author:dsamersoff
Date:  2014-08-28 07:46 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/68e9f36fd6c8

8029172: warnings from b117 for hotspot.agent.src.os.linux: JNI exception 
pending
Summary: Added missed exception checks
Reviewed-by: chegar, sla

! agent/src/os/linux/LinuxDebuggerLocal.c

Changeset: 4acab3118361
Author:coleenp
Date:  2014-08-28 11:29 -0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/4acab3118361

8049105: Move array component mirror to instance of java/lang/Class (hotspot 
part 2)
Summary: This removes component mirrors from arrayKlass metadata and the C2 
intrinsic for them.
Reviewed-by: kvn, twisti, mgerdin

! agent/src/share/classes/sun/jvm/hotspot/oops/ArrayKlass.java
! make/aix/makefiles/mapfile-vers-debug
! make/aix/makefiles/mapfile-vers-product
! make/bsd/makefiles/mapfile-vers-darwin-debug
! make/bsd/makefiles/mapfile-vers-darwin-product
! make/bsd/makefiles/mapfile-vers-debug
! make/bsd/makefiles/mapfile-vers-product
! make/linux/makefiles/mapfile-vers-debug
! make/linux/makefiles/mapfile-vers-product
! make/solaris/makefiles/mapfile-vers
! src/share/vm/ci/ciArrayKlass.hpp
! src/share/vm/classfile/javaClasses.cpp
! src/share/vm/classfile/javaClasses.hpp
! src/share/vm/classfile/vmSymbols.hpp
! src/share/vm/oops/arrayKlass.cpp
! src/share/vm/oops/arrayKlass.hpp
! src/share/vm/oops/klass.hpp
! src/share/vm/opto/library_call.cpp
! src/share/vm/opto/memnode.cpp
! src/share/vm/prims/jvm.cpp
! src/share/vm/prims/jvm.h
! src/share/vm/runtime/reflection.cpp
! src/share/vm/runtime/vmStructs.cpp

Changeset: 6e55d72da10c
Author:coleenp
Date:  2014-08-28 11:30 -0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/6e55d72da10c

Merge


Changeset: fca0e8790cca
Author:coleenp
Date:  2014-08-28 15:44 +
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/fca0e8790cca

Merge


Changeset: 920134274861
Author:lfoltan
Date:  2014-08-28 11:56 -0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/920134274861

8054355: ENFORCE_CC_COMPILER_REV needs to be updated to Solaris C++ 12u3 for 
JDK 9.
Summary: Update of C++ validiation check for JDK 9 on Solaris.
Reviewed-by: coleenp, dcubed, kvn

! make/solaris/makefiles/sparcWorks.make

Changeset: 34f2b1c63639
Author:lfoltan
Date:  2014-08-28 13:01 -0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/34f2b1c63639

Merge


Changeset: c8238a5aa66a
Author:sla
Date:  2014-08-29 13:34 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/c8238a5aa66a

8055838: Hotspot does not compile with clang 6.0 (OS X Yosemite)
Reviewed-by: lfoltan, kvn

! make/bsd/makefiles/gcc.make

Changeset: 2e3b1aa376cc
Author:sspitsyn
Date:  2014-08-29 11:35 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/2e3b1aa376cc

Merge


Changeset: 9cfb26e195ac
Author:iklam
Date:  2014-09-02 09:51 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/9cfb26e195ac

8056971: Minor class loading clean-up
Summary: Misplacement of #if INCLUE_CDS, typos, unnecessary C string duplication
Reviewed-by: dholmes, ccheung

! src/share/vm/classfile/classFileStream.cpp
! src/share/vm/classfile/classFileStream.hpp
! src/share/vm/classfile/classLoader.cpp
! src/share/vm/classfile/classLoader.hpp
! src/share/vm/classfile/classLoaderExt.hpp
! src/share/vm/classfile/sharedPathsMiscInfo.cpp
! src/share/vm/classfile/sharedPathsMiscInfo.hpp
! src/share/vm/classfile/systemDictionary.cpp

Changeset: 04760b64fe35
Author:ctornqvi
Date:  2014-09-03 17:30 -0400
URL:   http://hg.openjdk.java.n

hg: jigsaw/m2/corba: 4 new changesets

[erik . joelsson]

Changeset: 9baf79eda109
Author:wetmore
Date:  2014-09-11 12:17 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/9baf79eda109

8058180: .hgignore should be updated with webrev in all repos
Reviewed-by: jrose, katleman, jjg

! .hgignore

Changeset: c432b80aadd0
Author:lana
Date:  2014-09-11 14:28 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/c432b80aadd0

Merge


Changeset: b5b139354630
Author:katleman
Date:  2014-09-17 22:55 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/b5b139354630

Added tag jdk9-b31 for changeset c432b80aadd0

! .hgtags

Changeset: 0d0bcdce21f9
Author:erikj
Date:  2014-09-18 11:35 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/0d0bcdce21f9

Merge

hg: jigsaw/m2/jaxws: 3 new changesets

[erik . joelsson]

Changeset: 434628e088ce
Author:wetmore
Date:  2014-09-11 12:18 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxws/rev/434628e088ce

8058180: .hgignore should be updated with webrev in all repos
Reviewed-by: jrose, katleman, jjg

! .hgignore

Changeset: 7af228ae847f
Author:lana
Date:  2014-09-11 14:29 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxws/rev/7af228ae847f

Merge


Changeset: ffefee7f46fd
Author:katleman
Date:  2014-09-17 22:56 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxws/rev/ffefee7f46fd

Added tag jdk9-b31 for changeset 7af228ae847f

! .hgtags

hg: jigsaw/m2/jaxp: 6 new changesets

[erik . joelsson]

Changeset: 93a5ed174422
Author:joehw
Date:  2014-09-05 14:04 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxp/rev/93a5ed174422

8056202: Xerces Update: Catalog Resolver
Reviewed-by: lancea

! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/Catalog.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/CatalogEntry.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/CatalogException.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/CatalogManager.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/Resolver.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/helpers/BootstrapResolver.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/helpers/Debug.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/helpers/FileURL.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/helpers/Namespaces.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/helpers/PublicId.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/readers/CatalogReader.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/readers/DOMCatalogParser.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/readers/DOMCatalogReader.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/readers/ExtendedXMLCatalogReader.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/readers/OASISXMLCatalogReader.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/readers/SAXCatalogParser.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/readers/SAXCatalogReader.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/readers/SAXParserHandler.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/readers/TR9401CatalogReader.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/readers/TextCatalogReader.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/readers/XCatalogReader.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/tools/CatalogResolver.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/tools/ResolvingParser.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/tools/ResolvingXMLFilter.java
! 
src/java.xml/share/classes/com/sun/org/apache/xml/internal/resolver/tools/ResolvingXMLReader.java

Changeset: 705a3999eb77
Author:martin
Date:  2014-09-10 15:17 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxp/rev/705a3999eb77

8058175: [XML 1.0/1.1] - Attribute values with supplemental characters are 
being corrupted.
Reviewed-by: joehw

! 
src/java.xml/share/classes/com/sun/org/apache/xerces/internal/impl/XMLScanner.java

Changeset: e3853f655392
Author:lana
Date:  2014-09-10 19:38 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxp/rev/e3853f655392

Merge


Changeset: 9729d853cb16
Author:wetmore
Date:  2014-09-11 12:17 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxp/rev/9729d853cb16

8058180: .hgignore should be updated with webrev in all repos
Reviewed-by: jrose, katleman, jjg

! .hgignore

Changeset: 292317ebc7db
Author:lana
Date:  2014-09-11 14:28 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxp/rev/292317ebc7db

Merge


Changeset: b940ca3d2c7e
Author:katleman
Date:  2014-09-17 22:55 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jaxp/rev/b940ca3d2c7e

Added tag jdk9-b31 for changeset 292317ebc7db

! .hgtags

hg: jigsaw/m2/langtools: 16 new changesets

[erik . joelsson]

Changeset: 0ecc0804ec11
Author:sogoel
Date:  2014-09-04 16:52 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/0ecc0804ec11

8055996: Remove @ignore from tools/javac/T6725036.java
Reviewed-by: jjg

! test/tools/javac/T6725036.java

Changeset: 0877f3c322c3
Author:sogoel
Date:  2014-09-05 16:43 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/0877f3c322c3

8055075: Group 9b: golden files for tests in tools/javac dir
Reviewed-by: jjg

! test/tools/javac/ConditionalWithVoid.java
+ test/tools/javac/ConditionalWithVoid.out
! test/tools/javac/DuplicateClass.java
+ test/tools/javac/DuplicateClass.out
! test/tools/javac/EOI.java
+ test/tools/javac/EOI.out
! test/tools/javac/ExceptionalFinally2.java
+ test/tools/javac/ExceptionalFinally2.out
! test/tools/javac/ExprQualifiedType.java
+ test/tools/javac/ExprQualifiedType.out
! test/tools/javac/ExtendsScope.java
+ test/tools/javac/ExtendsScope.out
! test/tools/javac/ExtraneousEquals.java
+ test/tools/javac/ExtraneousEquals.out
! test/tools/javac/FlatnameClash2.java
+ test/tools/javac/FlatnameClash2.out
! test/tools/javac/InconsistentInheritedSignature.java
+ test/tools/javac/InconsistentInheritedSignature.out

Changeset: ea62dc3a866e
Author:sogoel
Date:  2014-09-05 16:51 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/ea62dc3a866e

8055079: Group 9c: golden files for tests in tools/javac dir
Reviewed-by: jjg

! test/tools/javac/InterfaceObjectIncompatibility.java
+ test/tools/javac/InterfaceObjectIncompatibility.out
! test/tools/javac/InterfaceObjectInheritance.java
+ test/tools/javac/InterfaceObjectInheritance.out
! test/tools/javac/InterfaceOverrideFinal.java
+ test/tools/javac/InterfaceOverrideFinal.out
! test/tools/javac/LabeledDeclaration.java
+ test/tools/javac/LabeledDeclaration.out
! test/tools/javac/NestedDuplicateLabels.java
+ test/tools/javac/NestedDuplicateLabels.out
! test/tools/javac/NewGeneric.java
+ test/tools/javac/NewGeneric.out
! test/tools/javac/NoClass.java
+ test/tools/javac/NoClass.out
! test/tools/javac/Object1.java
+ test/tools/javac/Object1.out
! test/tools/javac/Object2.java
+ test/tools/javac/Object2.out

Changeset: b545a2b1039d
Author:jlahoda
Date:  2014-09-08 10:48 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/b545a2b1039d

8056014: Type inference may be skipped for a complex receiver generic method in 
a parameter position
Summary: When checking if deferred attribution is needed for a chain of 
methods, stop on any method that returns any type variable, as the rest of 
analysis cannot use the correct type.
Reviewed-by: mcimadamore, vromero

! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/DeferredAttr.java
+ test/tools/javac/lambda/T8056014.java

Changeset: d7e615e0115c
Author:jlahoda
Date:  2014-09-08 10:50 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/d7e615e0115c

8056984: Exception in compiler: java.lang.AssertionError: isSubClass T
Summary: Must use type variable's captured bound as a method receiver, instead 
of the type variable itself, while checking if deferred attribution is needed.
Reviewed-by: mcimadamore, vromero

! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/DeferredAttr.java
+ test/tools/javac/lambda/T8056984.java

Changeset: 8df25ec8c930
Author:jfranck
Date:  2014-09-08 13:11 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/8df25ec8c930

8056021: checkin for JDK-8027262 breaks Checker Framework
Reviewed-by: jjg, mcimadamore

! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Attribute.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/code/TargetType.java
! 
src/jdk.compiler/share/classes/com/sun/tools/javac/code/TypeAnnotationPosition.java
+ src/jdk.compiler/share/classes/com/sun/tools/javac/code/TypeAnnotations.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Annotate.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Attr.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/AttrContext.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/DeferredAttr.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Env.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/MemberEnter.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/ClassReader.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/ClassWriter.java
! src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/Gen.java
! test/com/sun/javadoc/testTypeAnnotations/TestTypeAnnotations.java
! 
test/tools/javac/annotations/typeAnnotations/failures/CantAnnotatePackages.java
! test/tools/javac/annotations/typeAnnotations/failures/CantAnnotateScoping.java
! 
test/tools/javac/annotations/typeAnnotations/failures/CantAnnotateStaticClass2.java
! 
test/tools/javac/annotations/typeAnnotations/failures/CantAnnotateStaticClass3.java
! 
test/tools/javac/annotations/typeAnnotations/failures/common/arrays/De

hg: jigsaw/m2/nashorn: 16 new changesets

[erik . joelsson]

Changeset: f04f14587586
Author:lagergren
Date:  2014-09-05 16:28 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/f04f14587586

8057611: Nashorn did not dump the JOx classes to disk when running with the -d 
flag
Reviewed-by: attila, sundar, hannesw

! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/ObjectClassGenerator.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/CompiledFunction.java

Changeset: 4258ccc2eb8a
Author:lagergren
Date:  2014-09-05 16:28 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/4258ccc2eb8a

8057588: Lots of trivial (empty) classes were generated by the Nashorn compiler 
as part of restOf-method generation
Reviewed-by: attila, sundar, hannesw

! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/CompilationPhase.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/CompileUnit.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/Compiler.java
+ 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/CompileUnitHolder.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/FunctionNode.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/LiteralNode.java
! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/SplitNode.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/CompiledFunction.java

Changeset: 459874975ef6
Author:sundar
Date:  2014-09-08 15:51 +0530
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/459874975ef6

8057742: ant clean test should not fail if one or more external test suites are 
missing
Reviewed-by: lagergren, hannesw

- test/script/basic/JDK-8048079_1.js
- test/script/basic/JDK-8048079_1.js.EXPECTED
+ test/script/basic/JDK-8048079_1a.js
+ test/script/basic/JDK-8048079_1a.js.EXPECTED
+ test/script/basic/JDK-8048079_1b.js
+ test/script/basic/JDK-8048079_1b.js.EXPECTED
- test/script/basic/JDK-8048079_2.js
- test/script/basic/JDK-8048079_2.js.EXPECTED
+ test/script/basic/JDK-8048079_2a.js
+ test/script/basic/JDK-8048079_2a.js.EXPECTED
+ test/script/basic/JDK-8048079_2b.js
+ test/script/basic/JDK-8048079_2b.js.EXPECTED
! test/script/basic/splitter.js
! test/script/basic/splitter.js.EXPECTED
+ test/script/basic/splitter_prototype.js
+ test/script/basic/splitter_prototype.js.EXPECTED
+ test/script/basic/splitter_yui.js
+ test/script/basic/splitter_yui.js.EXPECTED
! test/src/jdk/nashorn/internal/test/framework/TestFinder.java

Changeset: 884a8ffb6038
Author:yan
Date:  2014-09-08 15:37 +0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/884a8ffb6038

8057678: Tests for let and const keywords in Nashorn
Reviewed-by: hannesw, lagergren
Contributed-by: Sergey Lugovoy 

+ test/script/basic/es6/const-redeclare-extra.js
+ test/script/basic/es6/const-redeclare-extra.js.EXPECTED
+ test/script/basic/es6/let-redeclare-extra.js
+ test/script/basic/es6/let-redeclare-extra.js.EXPECTED
+ test/script/basic/es6/let_const_closure.js
+ test/script/basic/es6/let_const_closure.js.EXPECTED
+ test/script/basic/es6/let_const_reuse.js
+ test/script/basic/es6/let_const_reuse.js.EXPECTED
+ test/script/basic/es6/let_different_types.js
+ test/script/basic/es6/let_different_types.js.EXPECTED
+ test/script/basic/es6/let_loops.js
+ test/script/basic/es6/let_loops.js.EXPECTED

Changeset: fc80190e129f
Author:attila
Date:  2014-09-08 18:40 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/fc80190e129f

8057148: Skip nested functions on reparse
Reviewed-by: hannesw, lagergren

! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/AssignSymbols.java
! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/Block.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/FunctionNode.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/LexicalContext.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/parser/Parser.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/parser/TokenStream.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/RecompilableScriptFunctionData.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/ScriptFunctionData.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/Timing.java
! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/tools/Shell.java
! test/script/basic/optimistic_check_type.js

Changeset: c17045fd979c
Author:attila
Date:  2014-09-09 15:33 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/c17045fd979c

8057930: remove eval ID
Reviewed-by: hannesw, sundar

! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/Lower.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/CodeInstaller.java
! 
src/jdk.scripting.nashorn/share/classes/jdk/nashorn

hg: jigsaw/m2/jdk: 47 new changesets

[erik . joelsson]

Changeset: 7af64e3e095d
Author:dl
Date:  2014-09-05 10:54 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/7af64e3e095d

8056248: Improve ForkJoin thread throttling
Reviewed-by: psandoz, martin

! src/java.base/share/classes/java/util/concurrent/ForkJoinPool.java
! src/java.base/share/classes/java/util/concurrent/ForkJoinTask.java
! src/java.base/share/classes/java/util/concurrent/ForkJoinWorkerThread.java

Changeset: 4f0cc1ad85b4
Author:alanb
Date:  2014-09-05 12:18 +0100
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/4f0cc1ad85b4

8029516: (fs) WatchKey cancel unreliable on Windows
Reviewed-by: chegar

! src/java.base/windows/classes/sun/nio/fs/WindowsNativeDispatcher.java
! src/java.base/windows/classes/sun/nio/fs/WindowsWatchService.java
! src/java.base/windows/native/libnio/fs/WindowsNativeDispatcher.c
+ test/java/nio/file/WatchService/LotsOfCancels.java

Changeset: c00881597c3e
Author:ykantser
Date:  2014-09-05 15:09 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/c00881597c3e

8057686: Clean up ProblemList.txt
Reviewed-by: jbachorik

! test/ProblemList.txt

Changeset: afacfa3a9401
Author:martin
Date:  2014-08-29 14:50 -0700
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/afacfa3a9401

8056951: pico-optimize contains(Object) methods
Summary: use idiiom >= 0 consistently to check for valid index
Reviewed-by: mduigou

! src/java.base/share/classes/java/lang/String.java
! src/java.base/share/classes/java/util/Arrays.java
! src/java.base/share/classes/java/util/LinkedList.java
! src/java.base/share/classes/java/util/PriorityQueue.java

Changeset: f7de7da2eb58
Author:aefimov
Date:  2014-09-07 23:04 +0400
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/f7de7da2eb58

8049343: (tz) Support tzdata2014g
Reviewed-by: mfang, okutsu

! make/data/tzdata/VERSION
! make/data/tzdata/africa
! make/data/tzdata/antarctica
! make/data/tzdata/asia
! make/data/tzdata/australasia
! make/data/tzdata/backward
! make/data/tzdata/etcetera
! make/data/tzdata/europe
! make/data/tzdata/factory
! make/data/tzdata/iso3166.tab
! make/data/tzdata/leapseconds
! make/data/tzdata/northamerica
! make/data/tzdata/pacificnew
! make/data/tzdata/southamerica
! make/data/tzdata/systemv
! make/data/tzdata/zone.tab
! src/java.base/share/classes/sun/util/calendar/ZoneInfoFile.java
! src/java.base/share/classes/sun/util/resources/TimeZoneNames.java
! src/jdk.localedata/share/classes/sun/util/resources/de/TimeZoneNames_de.java
! src/jdk.localedata/share/classes/sun/util/resources/es/TimeZoneNames_es.java
! src/jdk.localedata/share/classes/sun/util/resources/fr/TimeZoneNames_fr.java
! src/jdk.localedata/share/classes/sun/util/resources/it/TimeZoneNames_it.java
! src/jdk.localedata/share/classes/sun/util/resources/ja/TimeZoneNames_ja.java
! src/jdk.localedata/share/classes/sun/util/resources/ko/TimeZoneNames_ko.java
! 
src/jdk.localedata/share/classes/sun/util/resources/pt/TimeZoneNames_pt_BR.java
! src/jdk.localedata/share/classes/sun/util/resources/sv/TimeZoneNames_sv.java
! 
src/jdk.localedata/share/classes/sun/util/resources/zh/TimeZoneNames_zh_CN.java
! 
src/jdk.localedata/share/classes/sun/util/resources/zh/TimeZoneNames_zh_TW.java
! test/java/time/test/java/time/format/TestZoneTextPrinterParser.java
! test/sun/util/calendar/zi/tzdata/VERSION
! test/sun/util/calendar/zi/tzdata/africa
! test/sun/util/calendar/zi/tzdata/antarctica
! test/sun/util/calendar/zi/tzdata/asia
! test/sun/util/calendar/zi/tzdata/australasia
! test/sun/util/calendar/zi/tzdata/backward
! test/sun/util/calendar/zi/tzdata/etcetera
! test/sun/util/calendar/zi/tzdata/europe
! test/sun/util/calendar/zi/tzdata/factory
! test/sun/util/calendar/zi/tzdata/iso3166.tab
! test/sun/util/calendar/zi/tzdata/leapseconds
! test/sun/util/calendar/zi/tzdata/northamerica
! test/sun/util/calendar/zi/tzdata/pacificnew
! test/sun/util/calendar/zi/tzdata/southamerica
! test/sun/util/calendar/zi/tzdata/systemv
! test/sun/util/calendar/zi/tzdata/zone.tab
! test/sun/util/resources/TimeZone/Bug6317929.java

Changeset: d4fb1717a6c2
Author:sla
Date:  2014-09-08 14:46 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/d4fb1717a6c2

8057776: Misc cleanups of the attach code
Reviewed-by: alanb, dfuchs, jbachorik

! src/jdk.attach/linux/classes/sun/tools/attach/AttachProviderImpl.java
! src/jdk.attach/macosx/classes/sun/tools/attach/AttachProviderImpl.java
! src/jdk.attach/share/classes/com/sun/tools/attach/VirtualMachine.java
! src/jdk.attach/share/classes/sun/tools/attach/HotSpotAttachProvider.java
! src/jdk.attach/solaris/classes/sun/tools/attach/AttachProviderImpl.java
! test/sun/management/jmxremote/startstop/JMXStartStopTest.java

Changeset: 63cbc8a216f7
Author:sla
Date:  2014-09-08 20:29 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/63cbc8a216f7

8057778: Misc cleanups of the attach code (aix)
Reviewed-by: simonis

! src/jdk.attach/aix/classes/sun/tools/attach/Attach

hg: jigsaw/m2: New basic target for jimages

[erik . joelsson]

Changeset: ea1777436829
Author:erikj
Date:  2014-09-18 16:52 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/ea1777436829

New basic target for jimages

+ make/Images.gmk
! make/Main.gmk

hg: jigsaw/m2: 3 new changesets

[erik . joelsson]

Changeset: 909d68bc96d2
Author:erikj
Date:  2014-09-19 11:40 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/909d68bc96d2

Moved docs and docstemp to images/support

! make/Javadoc.gmk
! make/Main.gmk

Changeset: 212428cc2431
Author:erikj
Date:  2014-09-19 12:06 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/212428cc2431

Renaming images to dist

! common/autoconf/bootcycle-spec.gmk.in
! common/autoconf/generated-configure.sh
! common/autoconf/spec.gmk.in
! make/Images.gmk
! make/Javadoc.gmk
! make/Main.gmk

Changeset: 5e390420e4b7
Author:erikj
Date:  2014-09-19 12:19 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/5e390420e4b7

Merge

! make/Images.gmk

hg: jigsaw/m2/jdk: 3 new changesets

[erik . joelsson]

Changeset: 73f38e156879
Author:erikj
Date:  2014-09-19 12:05 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/73f38e156879

Renaming images to dist

! make/CompileDemos.gmk
! make/CopySamples.gmk
! make/CreateJars.gmk
! make/Images.gmk
! make/Profiles.gmk

Changeset: 8dbf8c2943f2
Author:erikj
Date:  2014-09-19 12:19 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/8dbf8c2943f2

Merge

! make/CreateJars.gmk
- 
src/jdk.localedata/META-INF/cldrdata-services/sun.util.locale.provider.LocaleDataMetaInfo
- 
src/jdk.localedata/META-INF/localedata-services/sun.util.locale.provider.LocaleDataMetaInfo

Changeset: 4c59d3ad3403
Author:erikj
Date:  2014-09-19 12:21 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/4c59d3ad3403

Merge

hg: jigsaw/m2: Switching bootcycle-images to use jimage (not working yet)

[erik . joelsson]

Changeset: f811c6354ec2
Author:erikj
Date:  2014-09-19 12:33 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/f811c6354ec2

Switching bootcycle-images to use jimage (not working yet)

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

hg: jigsaw/m2: Stripping binaries

[erik . joelsson]

Changeset: 545d0f49a0d3
Author:erikj
Date:  2014-09-19 14:14 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/545d0f49a0d3

Stripping binaries

! make/Images.gmk
! make/Main.gmk
+ make/StripBinaries.gmk

hg: jigsaw/m2: Added man pages, include, release files and readme/license files

[erik . joelsson]

Changeset: fc83359acd30
Author:erikj
Date:  2014-09-19 15:46 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/fc83359acd30

Added man pages, include, release files and readme/license files

! make/Images.gmk

hg: jigsaw/m2: Added src.zip to jdk jimage

[erik . joelsson]

Changeset: 494f41aa65f7
Author:erikj
Date:  2014-09-19 16:14 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/494f41aa65f7

Added src.zip to jdk jimage

! make/Images.gmk
! make/Main.gmk
+ make/ZipSource.gmk

hg: jigsaw/m2: Reverted dist back to images. Moved jimages to jre/jdk. Added demos and samples to jdk.

[erik . joelsson]

Changeset: ce31454f86d8
Author:erikj
Date:  2014-09-19 17:27 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/ce31454f86d8

Reverted dist back to images. Moved jimages to jre/jdk. Added demos and samples 
to jdk.

! common/autoconf/spec.gmk.in
! make/Images.gmk
! make/Javadoc.gmk
! make/Main.gmk

hg: jigsaw/m2/jdk: Reverted dist back to images. Moved jimages to jre/jdk. Added demos and samples to jdk.

[erik . joelsson]

Changeset: b909a2c4564b
Author:erikj
Date:  2014-09-19 17:27 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/b909a2c4564b

Reverted dist back to images. Moved jimages to jre/jdk. Added demos and samples 
to jdk.

! make/CompileDemos.gmk
! make/CopySamples.gmk
! make/CreateJars.gmk
! make/Images.gmk
! make/Profiles.gmk

hg: jigsaw/m2: Fixed man pages in images again

[erik . joelsson]

Changeset: 853db8b9e476
Author:erikj
Date:  2014-09-22 11:33 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/853db8b9e476

Fixed man pages in images again

! make/Images.gmk

hg: jigsaw/m2/jdk: Removed sa-jdi.jar from exploded image and new jdk image

[erik . joelsson]

Changeset: 1b369f4e3775
Author:erikj
Date:  2014-09-22 11:49 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/1b369f4e3775

Removed sa-jdi.jar from exploded image and new jdk image

! make/Import.gmk

hg: jigsaw/m2: Removed .diz files from images bin dir

[erik . joelsson]

Changeset: 59eff3df1313
Author:erikj
Date:  2014-09-22 12:02 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/59eff3df1313

Removed .diz files from images bin dir

! make/Images.gmk
! make/StripBinaries.gmk

hg: jigsaw/m2: Combining two com.sun.jdi.connect.Connector into one

[erik . joelsson]

Changeset: 85a5abbba3b8
Author:erikj
Date:  2014-09-22 12:34 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/85a5abbba3b8

Combining two com.sun.jdi.connect.Connector into one

! make/Main.gmk

hg: jigsaw/m2/jdk: Combining two com.sun.jdi.connect.Connector into one

[erik . joelsson]

Changeset: daea3df99bfe
Author:erikj
Date:  2014-09-22 12:34 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/daea3df99bfe

Combining two com.sun.jdi.connect.Connector into one

! make/gensrc/GensrcProviders.gmk

hg: jigsaw/m2/jdk: Use combined com.sun.jdi.connect.Connector in both places

[erik . joelsson]

Changeset: c23bf055325b
Author:erikj
Date:  2014-09-22 14:17 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/c23bf055325b

Use combined com.sun.jdi.connect.Connector in both places

! make/Import.gmk
! make/gensrc/GensrcProviders.gmk

hg: jigsaw/m2: Making the new images the default and legacy optional

[erik . joelsson]

Changeset: 0f1024cadf20
Author:erikj
Date:  2014-09-22 14:46 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/0f1024cadf20

Making the new images the default and legacy optional

! common/autoconf/spec.gmk.in
! make/Images.gmk
! make/Main.gmk

hg: jigsaw/m2/jdk: Making the new images the default and legacy optional

[erik . joelsson]

Changeset: 09d9cd373d65
Author:erikj
Date:  2014-09-22 14:46 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/09d9cd373d65

Making the new images the default and legacy optional

! make/Images.gmk

hg: jigsaw/m2/langtools: Exclude jdk repo generated source from interim langtools compilation of jdk.dev

[erik . joelsson]

Changeset: 664507494bd8
Author:erikj
Date:  2014-09-22 15:38 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/664507494bd8

Exclude jdk repo generated source from interim langtools compilation of jdk.dev

! make/CompileInterim.gmk

hg: jigsaw/m2/jdk: Fixed windows build

[erik . joelsson]

Changeset: 21d255560bca
Author:erikj
Date:  2014-09-22 16:02 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/21d255560bca

Fixed windows build

! make/Import.gmk

hg: jigsaw/m2: Fixed intermittent windows build fail

[erik . joelsson]

Changeset: f06bc8eb2756
Author:erikj
Date:  2014-09-22 16:04 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/f06bc8eb2756

Fixed intermittent windows build fail

! make/Main.gmk

hg: jigsaw/m2: 2 new changesets

[erik . joelsson]

Changeset: e0ed05bed368
Author:erikj
Date:  2014-09-22 17:32 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/e0ed05bed368

Building mac bundles

! common/autoconf/spec.gmk.in
+ make/MacBundles.gmk
! make/Main.gmk

Changeset: 3d7b88817c42
Author:erikj
Date:  2014-09-23 11:05 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/3d7b88817c42

Fixed echo line in images build

! make/Images.gmk

hg: jigsaw/m2: Compact profiles using jimages

[erik . joelsson]

Changeset: e4198a8029d1
Author:erikj
Date:  2014-09-23 12:40 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/e4198a8029d1

Compact profiles using jimages

! make/Images.gmk
! make/Main.gmk

hg: jigsaw/m2/jdk: Fixed incremental build issue

[erik . joelsson]

Changeset: 0f14c5e99be0
Author:erikj
Date:  2014-09-23 15:21 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/0f14c5e99be0

Fixed incremental build issue

! make/gensrc/GensrcProperties.gmk

hg: jigsaw/m2: Added jdk.localedata jdk.crypto.pkcs11 jdk.crypto.ec to compact images

[erik . joelsson]

Changeset: 4f388cda1cf2
Author:erikj
Date:  2014-09-23 16:12 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/4f388cda1cf2

Added jdk.localedata jdk.crypto.pkcs11 jdk.crypto.ec to compact images

! make/Images.gmk

hg: jigsaw/m2/corba: Repplied 8049389: Move orb.idl and ir.idl to JDK include directory

[erik . joelsson]

Changeset: ce39758b2d60
Author:erikj
Date:  2014-09-29 10:13 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/corba/rev/ce39758b2d60

Repplied 8049389: Move orb.idl and ir.idl to JDK include directory

! make/copy/Copy-java.corba.gmk

hg: jigsaw/m2: Reenabled bootcycle-images

[erik . joelsson]

Changeset: 7ff4bdb75613
Author:erikj
Date:  2014-10-02 10:36 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/7ff4bdb75613

Reenabled bootcycle-images

! common/autoconf/bootcycle-spec.gmk.in
! make/Main.gmk

hg: jigsaw/m2/jdk: Moved sec-bin creation to new images

[erik . joelsson]

Changeset: e7740098d052
Author:erikj
Date:  2014-10-02 16:41 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/e7740098d052

Moved sec-bin creation to new images

! make/CreateJars.gmk
! make/UnpackSecurity.gmk

hg: jigsaw/m2: Moved sec-bin creation to new images

[erik . joelsson]

Changeset: c9bbaf770ea6
Author:erikj
Date:  2014-10-02 16:41 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/c9bbaf770ea6

Moved sec-bin creation to new images

! make/Main.gmk
+ make/SecBin.gmk

hg: jigsaw/m2/jdk: Fixed sec-bin on windows

[erik . joelsson]

Changeset: cab0742c5fa9
Author:erikj
Date:  2014-10-02 17:25 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/cab0742c5fa9

Fixed sec-bin on windows

! make/lib/LibCommon.gmk

hg: jigsaw/m2: Fixed sec-bin on windows

[erik . joelsson]

Changeset: 517ba0a9d5e1
Author:erikj
Date:  2014-10-02 17:25 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/517ba0a9d5e1

Fixed sec-bin on windows

! make/SecBin.gmk

hg: jigsaw/m2: Properly fixing bootcycle-images

[erik . joelsson]

Changeset: 4b33b1539e15
Author:erikj
Date:  2014-10-02 17:46 +0200
URL:   http://hg.openjdk.java.net/jigsaw/m2/rev/4b33b1539e15

Properly fixing bootcycle-images

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