Hello Mandy,
Looks good. Just noticed another thing in CheckModules.gmk. You can
remove the explicit FIXPATH as the variable JAVA already contains it. In
general, only spec.gmk should need to worry about FIXPATH.
If there are no other changes, I don't need to see another round before
you pus
Looks great!
Thank you for polishing this fix!
/Magnus
28 aug 2014 kl. 19:21 skrev Mandy Chung :
> On 8/27/14 11:38 PM, Erik Joelsson wrote:
>> Hello Mandy,
>>
>> That certainly looks better. A couple of more thoughts, and sorry for not
>> pointing this out earlier, but the new structure is s
On 8/27/14 11:38 PM, Erik Joelsson wrote:
Hello Mandy,
That certainly looks better. A couple of more thoughts, and sorry for
not pointing this out earlier, but the new structure is still new to
me too.
* The rmic targets also generate classes, so for modules.xml to be
correct, I suspect you
On 8/28/14 1:32 AM, Magnus Ihse Bursie wrote:
On 2014-08-27 18:00, Mandy Chung wrote:
Erik, Magnus,
This is much easier than I have thought. I really like this new build.
Glad to hear! :)
I have separated out Gendata-jdk.dev.gmk and removed the modules-xml
target completely.
Webrev at:
http
On 2014-08-27 18:00, Mandy Chung wrote:
Erik, Magnus,
This is much easier than I have thought. I really like this new build.
Glad to hear! :)
I have separated out Gendata-jdk.dev.gmk and removed the modules-xml
target completely.
Webrev at:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/805
Hello Mandy,
That certainly looks better. A couple of more thoughts, and sorry for
not pointing this out earlier, but the new structure is still new to me too.
* The rmic targets also generate classes, so for modules.xml to be
correct, I suspect you need to depend on that too. Simply add "rmi
Erik, Magnus,
This is much easier than I have thought. I really like this new build.
I have separated out Gendata-jdk.dev.gmk and removed the modules-xml
target completely.
Webrev at:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8055856/webrev.01/
Mandy
On 2014-08-27 16:39, Mandy Chung wrote:
On 8/27/2014 3:19 AM, Magnus Ihse Bursie wrote:
On 2014-08-27 10:26, Erik Joelsson wrote:
Hello Mandy,
Looking at this, I just realized that
$(JDK_OUTPUTDIR)/modules/jdk.dev/com/sun/tools/jdeps/resources/jdeps-modules.xml
is a generated resource for
On 8/27/2014 3:19 AM, Magnus Ihse Bursie wrote:
On 2014-08-27 10:26, Erik Joelsson wrote:
Hello Mandy,
Looking at this, I just realized that
$(JDK_OUTPUTDIR)/modules/jdk.dev/com/sun/tools/jdeps/resources/jdeps-modules.xml
is a generated resource for a module and that you correctly added it
On 2014-08-27 10:26, Erik Joelsson wrote:
Hello Mandy,
Looking at this, I just realized that
$(JDK_OUTPUTDIR)/modules/jdk.dev/com/sun/tools/jdeps/resources/jdeps-modules.xml
is a generated resource for a module and that you correctly added it
to the gendata target. Then to make it fit with th
Hello Mandy,
Looking at this, I just realized that
$(JDK_OUTPUTDIR)/modules/jdk.dev/com/sun/tools/jdeps/resources/jdeps-modules.xml
is a generated resource for a module and that you correctly added it to
the gendata target. Then to make it fit with the new makefile model, the
running of TOOL_
JDK-8055856: checkdeps build target doesn't work for cross-compilation builds
JDK-8056113: [build] tools.jar missing modules.xml
Webrev at:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8055856/
This patch fixes a few things about modules.xml
1. jdeps is invoked at build time to verify the
12 matches
Mail list logo
