Thumbs up from me. Not crazy about Version.java.template, but ok for now, and I don't have a better idea. :^(
-kto On Mar 21, 2011, at 6:57 PM, David Holmes wrote: > Updated webrev at: > > http://cr.openjdk.java.net/~dholmes/7025066/webrev.2/ > > I standardised the use of the jar compression options on all jar creation > commands in Release.gmk and aded a similar construct to sun/nio/cs/Makefile > for charsets.jar. These changes only affect jar files that would normally be > created without compression - I have not tried to define universal jar > creation options in the JDK build system. > > Really hoping to put this to bed in the next day or two. Looking specifically > for a thumbs-up from Kelly, but would be appreciated if Andrew could also > indicate acceptance. > > Thanks, > David > > David Holmes said the following on 03/17/11 12:07: >> webrev: http://cr.openjdk.java.net/~dholmes/7025066/webrev/ >> This places the SE-Embedded specific targets and definitions into >> Release-embedded.gmk and Defs-embedded.gmk respectively. These files are >> unconditionally included but then guarded with ifdef JAVASE_EMBEDDED >> internally. This avoids the need for JAVASE_EMBEDDED to appear in the main >> build files. >> Additional changes: >> - some control over whether certain jars are compressed or not >> - make ALSA version check conditional of having REQUIRED_ALSA_VERSION defined >> - control use of USE_MMAP flag for zip >> The final chunk here will be AWT related updates. >> Then an update to the builds readme in the top-level repo. >> Thanks, >> David