I agree the dependencies in BuildJdk.gmk are messed up but I didn't think this bug was the correct place to clean it up so I just followed the existing pattern, which is a strict linear dependency chain.

/Erik

On 2013-04-30 17:58, Mike Duigou wrote:
It's very nice to see this resolved. Hopefully one more nail in the old build's 
coffin.

The jdk target should depend upon genclasses. It seems really strange to have 
this as a dependency for securityjars and not jdk.

Mike

On Apr 30 2013, at 07:45 , Erik Joelsson wrote:

With this patch the security tests will again be runnable on the exploded jdk 
image. The main changes are:

* The security classes are compiled separately to a different output directory.
* The security jars are created in the jdk target (instead of images) and put 
in the jdk/lib/... directories.

Also did:
* Removed now redundant entries in rt.jar exclude list
* Changed source location for signing unsigned jars
* Made the SetupJavaCompilation macro more friendly with multiple setups 
sharing output directories.

http://cr.openjdk.java.net/~erikj/8009280/webrev.jdk.01/
http://cr.openjdk.java.net/~erikj/8009280/webrev.root.01/

/Erik

Reply via email to