[modules-dev] Review request for JRepo changes

[Dave Bristor]

Hi folks,

I have a webrev which addresses several changes:

http://webrev.invokedynamic.info/bristor/jrepo.01/

6590185 (jrepo) Installing a module via repository management tool should 
check its dependencies
http://bugs.sun.com/view_bug.do?bug_id=6590185

6593310 (jrepo) "list" command should sort repository contents
http://bugs.sun.com/view_bug.do?bug_id=6593310

6605083 (jrepo) add "validate" subcommand
http://bugs.sun.com/view_bug.do?bug_id=6605083

6723138 Change LocalRepositoryTest to not check read-only-ness on Windows
http://bugs.sun.com/view_bug.do?bug_id=6723138

6727572 Remove percmission checks from RepositoryConfig
http://bugs.sun.com/view_bug.do?bug_id=6727572

There are tests for the first three only.  The test for sorting repository 
contents is in JRepoTest.  The "validate" command and install-time checking 
have new tests.

While making changes to jrepo, I changed the way it provides help.  There's a 
new command, so "jrepo help" shows all commands.  Each command takes "-h" to 
provide help on its own options.  If command is recognized but is arguments 
are not, help is again provided on just that command (e.g. "jrepo list 
-fred").  If the command is not recognized, help is given for all commands 
(e.g. "jrepo womble").

I removed the ability to specify platform binding from the jrepo dependencies 
command, and did not add it to validate either:  Both of these ultimately rely 
on the module initialization code, which needs a ModuleDefinition, which a 
Repository will only provide for the current platform.  We can revisit this 
later, if needed.

If review can be completed by COB Thu/7, I'll do a putback this week, 
presuming there are no difficulties integrating with other incoming changes. 
Otherwise it'll be after my vacation.

Thanks!
	Dave