hg: jigsaw/jake/langtools: improve handling of synthetic compilation units for package.html

2016-02-08 Thread jonathan . gibbons
Changeset: 6b6ea9e2fe99 Author:jjg Date: 2016-02-08 18:01 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/6b6ea9e2fe99 improve handling of synthetic compilation units for package.html ! src/jdk.compiler/share/classes/com/sun/tools/javac/api/JavacTrees.java

hg: jigsaw/jake/langtools: Start using module element

2016-02-08 Thread jonathan . gibbons
Changeset: ab2073af5b29 Author:ksrini Date: 2016-02-08 16:45 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/ab2073af5b29 Start using module element ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/AbstractModuleIndexWriter.java ! src/jdk.

Re: RFR 8149391: Fix module dependencies in java/util

2016-02-08 Thread Alexandre (Shura) Iline
> On Feb 8, 2016, at 3:58 PM, Mandy Chung wrote: > > >> On Feb 8, 2016, at 3:36 PM, Alexandre (Shura) Iline >> wrote: >> >>> >>> On Feb 8, 2016, at 1:23 PM, Mandy Chung wrote: >>> >>> On Feb 8, 2016, at 11:43 AM, Alexandre (Shura) Iline wrote: Hi. Could

hg: jigsaw/jake/jdk: Fix test's @modules

2016-02-08 Thread mandy . chung
Changeset: 6fcd1a4df3b9 Author:shurailine Date: 2016-02-08 16:15 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/6fcd1a4df3b9 Fix test's @modules ! test/java/util/Currency/CurrencyTest.java ! test/java/util/logging/modules/GetResourceBundleTest.java

Re: RFR 8149391: Fix module dependencies in java/util

2016-02-08 Thread Mandy Chung
> On Feb 8, 2016, at 3:36 PM, Alexandre (Shura) Iline > wrote: > >> >> On Feb 8, 2016, at 1:23 PM, Mandy Chung wrote: >> >> >>> On Feb 8, 2016, at 11:43 AM, Alexandre (Shura) Iline >>> wrote: >>> >>> Hi. >>> >>> Could you please take a look on additional module dependencies to be added

hg: jigsaw/jake/langtools: 8148648: javac, neither a service definition or its implementation can be inner classes

2016-02-08 Thread jonathan . gibbons
Changeset: 46c903152cfe Author:vromero Date: 2016-02-08 15:55 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/46c903152cfe 8148648: javac, neither a service definition or its implementation can be inner classes ! src/jdk.compiler/share/classes/com/sun/tools/javac/c

hg: jigsaw/jake/langtools: update javadoc to support -addmods -limitmods

2016-02-08 Thread jonathan . gibbons
Changeset: afbf27e227d7 Author:jjg Date: 2016-02-08 15:36 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/afbf27e227d7 update javadoc to support -addmods -limitmods ! src/jdk.javadoc/share/classes/com/sun/tools/javadoc/JavadocTool.java ! src/jdk.javadoc/share/classe

hg: jigsaw/jake: update javadoc to support -addmods -limitmods

2016-02-08 Thread jonathan . gibbons
Changeset: 9b60f19f818d Author:jjg Date: 2016-02-08 15:36 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/rev/9b60f19f818d update javadoc to support -addmods -limitmods ! make/Javadoc.gmk

Re: RFR 8149391: Fix module dependencies in java/util

2016-02-08 Thread Alexandre (Shura) Iline
> On Feb 8, 2016, at 1:23 PM, Mandy Chung wrote: > > >> On Feb 8, 2016, at 11:43 AM, Alexandre (Shura) Iline >> wrote: >> >> Hi. >> >> Could you please take a look on additional module dependencies to be added >> for tests in java/util. There is more work to do for :jdk_core test group, >

hg: jigsaw/jake/hotspot: Fix up JvmtiExport function signature

2016-02-08 Thread mandy . chung
Changeset: b59ef60e72fc Author:sspitsyn Date: 2016-02-08 14:53 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/b59ef60e72fc Fix up JvmtiExport function signature ! src/share/vm/prims/jvmtiExport.hpp

hg: jigsaw/jake/jdk: Fixing GetResourceBundleTest after recent change to enabled modules in javac - the compilation now needs to use -addmods similarly to runtime.

2016-02-08 Thread jan . lahoda
Changeset: ff715ebf0299 Author:jlahoda Date: 2016-02-08 23:14 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/ff715ebf0299 Fixing GetResourceBundleTest after recent change to enabled modules in javac - the compilation now needs to use -addmods similarly to runtime. ! tes

Re: RFR 8149391: Fix module dependencies in java/util

2016-02-08 Thread Mandy Chung
> On Feb 8, 2016, at 11:43 AM, Alexandre (Shura) Iline > wrote: > > Hi. > > Could you please take a look on additional module dependencies to be added > for tests in java/util. There is more work to do for :jdk_core test group, > which is to come separately. > > JDK9 changes: http://cr.open

hg: jigsaw/jake/langtools: 8130868: error msg can be more accurate while compiling file in default package with -modulesourcepath option

2016-02-08 Thread jonathan . gibbons
Changeset: 257fc848711e Author:vromero Date: 2016-02-08 13:15 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/257fc848711e 8130868: error msg can be more accurate while compiling file in default package with -modulesourcepath option ! src/jdk.compiler/share/classes

hg: jigsaw/jake/langtools: fix and enable indy string concat tests

2016-02-08 Thread jonathan . gibbons
Changeset: 80ac7bd55719 Author:jjg Date: 2016-02-08 12:48 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/80ac7bd55719 fix and enable indy string concat tests ! test/ProblemList.jake.txt ! test/tools/javac/TestIndyStringConcat.java

hg: jigsaw/jake/jdk: Define Package objects lazily as it's not always needed

2016-02-08 Thread mandy . chung
Changeset: 7a54c4b00471 Author:mchung Date: 2016-02-08 12:44 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/7a54c4b00471 Define Package objects lazily as it's not always needed ! src/java.base/share/classes/java/lang/Class.java ! src/java.base/share/classes/java/lang/Cla

Re: hg: jigsaw/jake/langtools: support module files directly on file manager module paths

2016-02-08 Thread Jonathan Gibbons
The support for files on the classpath with a .zip extension is unchanged. It has never been considered for modules. -- Jon On 02/08/2016 11:46 AM, Robert Scholte wrote: Hi Jon, thanks, I'll give it a try with the next jigsaw-ea. I also noticed the following change below. IIRC a .zip file w

Re: hg: jigsaw/jake/langtools: support module files directly on file manager module paths

2016-02-08 Thread Robert Scholte
Hi Jon, thanks, I'll give it a try with the next jigsaw-ea. I also noticed the following change below. IIRC a .zip file was considered a valid file extension for classpath entries (even though we still don't support it with Maven). Is the zip file dropped in case of modules? thanks, Robert

RFR 8149391: Fix module dependencies in java/util

2016-02-08 Thread Alexandre (Shura) Iline
Hi. Could you please take a look on additional module dependencies to be added for tests in java/util. There is more work to do for :jdk_core test group, which is to come separately. JDK9 changes: http://cr.openjdk.java.net/~shurailine/8149391/webrev.jdk9.00/ Jake changes: http://cr.openjdk.jav

Re: [Jigsaw] Getting "Bad service configuration file" error with annotation processor

2016-02-08 Thread Alex Buckley
The NoClassDefFoundError for javax.tools.DiagnosticListener makes me suspect that the project's code is in the unnamed module (a.k.a. on the classpath) and that no other module has read the java.compiler module which exports the javax.tools package. Hence, java.compiler is not in the module gra

hg: jigsaw/jake/langtools: fix tests on langtools ProblemList.jake.txt

2016-02-08 Thread jonathan . gibbons
Changeset: 05ea8a0ca1c6 Author:vromero Date: 2016-02-08 11:14 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/05ea8a0ca1c6 fix tests on langtools ProblemList.jake.txt ! test/ProblemList.jake.txt ! test/tools/javac/T6406771.java ! test/tools/javac/T8003967/DetectMuta

hg: jigsaw/jake/jdk: 8144623: adjusting tests to recent change to enabled modules in javac (try 2).

2016-02-08 Thread jan . lahoda
Changeset: 9ecaf2673e3c Author:jlahoda Date: 2016-02-07 21:41 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/9ecaf2673e3c 8144623: adjusting tests to recent change to enabled modules in javac (try 2). ! test/java/lang/Class/getResource/ResourcesTest.java ! test/java/lang

hg: jigsaw/jake/langtools: 8144623: adding -addmods/-limitmods options; adjusting the set of enabled modules to match runtime behavior (try 2).

2016-02-08 Thread jan . lahoda
Changeset: bfa233e22145 Author:jlahoda Date: 2016-02-07 21:40 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/bfa233e22145 8144623: adding -addmods/-limitmods options; adjusting the set of enabled modules to match runtime behavior (try 2). ! src/jdk.compiler/share/

hg: jigsaw/jake/langtools: Avoid using indy concat when compiling code that needs to run inside ant itself.

2016-02-08 Thread jan . lahoda
Changeset: 7aacdf3bcf93 Author:jlahoda Date: 2016-02-08 17:53 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/7aacdf3bcf93 Avoid using indy concat when compiling code that needs to run inside ant itself. ! make/build.xml

Re: [Jigsaw] Getting "Bad service configuration file" error with annotation processor

2016-02-08 Thread Jonathan Gibbons
Gunnar, cross-posting to jigsaw-dev. Can you describe more about the setup of this example? Is the processor in a module with a module declaration with a provides clause for the processor, or is it a "pre-Jigsaw"/"JDK8-style" processor with a service configuration file in a jar file's META-I