[modules-dev] Review request for 5682244 (remove ExpandedJamRepository)

Dave Bristor David.Bristor at Sun.COM
Tue Aug 28 18:25:46 PDT 2007


Hi folks,

This removes ExpandedJamRepository; LocalRepository now does the heavy lifting.

bugster: http://monaco.sfbay/detail.jsf?cr=6582244
webrev: http://analemma.sfbay.sun.com/java/jdk/ws/libs/rev/6582244/

JAMs are not fully expanded as they were before, but JamUtils.expandJam() is 
invoked to pull out any embedded jars and/or native libraries.

expandJam() makes a copy of the JAM file under the directory into which it 
expands the JAM's contents, and puts a copy of the MODULE.METADATA file under 
this directory too.  I don't believe expandJam() needs to do this, and don't 
see any other uses of expandJam beyond new file LocalJamDefinitionContent.  If 
there is reason for the functionality I removed, please let me know!

Thanks,
	Dave




More information about the modules-dev mailing list