Review Request: 8007524: build-infra: Incremental build of tools.jar broken (original) (raw)

Tim Bell tim.bell at oracle.com
Tue Feb 5 15:16:14 UTC 2013


Erik:

The logic for setting up correct dependencies from class files to jar files in the macro SetupArchive is broken. This fixes two things. The wrong variable was used to filter files found and a ',' was missing in the macro call to CacheFind. Verifed that dependency list is now populated instead of empty as before, and that changing a class in javac caused tools.jar to be updated.

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

Looks good. That was a very subtle fix.

Tim



More information about the build-dev mailing list