On 18/03/2013 19:59, David Holmes wrote:
Very simple fix/cleanup.
webrev: http://cr.openjdk.java.net/~dholmes/8009426/webrev/
nashorn.jar was being added to the set of JARS targets regardless of
what was being built, but nashorn is not a dependency for profiles
target. JARS is not used the way it used to be now we have profiles
and all jar files should simply be listed in the appropriate entry in
profiles-includes.txt. There was a couple of other misplaced items too:
makefiles/CreateJars.gmk
Removed nashorn and tzdb setting of JARS
makefiles/Profiles.gmk
Removed redundant listing of cldrdata.jar
makefiles/profile-includes.txt
Add nashorn to full JRE lists
This looks okay to me for now but as javax.script is in compact3 then we
should be looking to include that in the compact3 builds.
The other clean-ups looks okay to me too. I remember suggesting to
Sherman to add tzdb.jar to JARS but that was prior to the merge with
profiles (pre-dated profiles-includes.txt).
-Alan.