Code Review for jrepo

Prepared by: David Bristor (db13166) on Wed Aug 6 09:30:42 PDT 2008
Workspace:/java/tl/libs/bristor/ws/jrepo
Compare against: ssh://bristor@hg.openjdk.java.net/jdk7/modules-gate/jdk
Summary of changes: 786 lines changed: 672 ins; 70 del; 44 mod; 2700 unchg
Patch of changes: jrepo.patch
Author comments:
6590185 (jrepo) Installing a module via repository management tool should check its dependencies 6593310 (jrepo) "list" command should sort repository contents 6605083 (jrepo) add "validate" subcommand 6723138 Change LocalRepositoryTest to not check read-only-ness on Windows 6727572 Remove percmission checks from RepositoryConfig Reviewed-by:
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/classes/sun/module/repository/RepositoryConfig.java

11 lines changed: 0 ins; 11 del; 0 mod; 625 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/classes/sun/module/tools/JRepo.java

360 lines changed: 280 ins; 58 del; 22 mod; 1029 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/module/modinit/JRepoDependenciesTest.java

2 lines changed: 0 ins; 1 del; 1 mod; 182 unchg

------ ------ ------ ------ --- New Patch Raw test/java/module/modinit/JRepoInstallTest.java

187 lines changed: 187 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/java/module/modinit/JRepoValidateTest.java

171 lines changed: 171 ins; 0 del; 0 mod; 0 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/module/repository/LocalRepositoryTest.java

7 lines changed: 5 ins; 0 del; 2 mod; 550 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/module/tools/jrepo/JRepoTest.java

48 lines changed: 29 ins; 0 del; 19 mod; 314 unchg

This code review page was prepared using /java/jdk/bin/webrev (vers 23.14-hg).