RFR: Try to install already installed jmod/modular jar issue
    Mandy Chung 
    mandy.chung at oracle.com
       
    Wed May 30 10:54:17 PDT 2012
    
    
  
On 5/29/2012 1:45 PM, Chris Hegarty wrote:
> Trivial issue that crept in whereby if you try to install an already 
> installed module it will actually remove it!
>
> If a module is already installed ( or there is a view or alias with a 
> similar name ) the addition to the module directory will fail. The 
> error handling code incorrectly assumes that the requested module 
> should be removed.
>
> Trivial change to only attempt removal after the addition to the 
> module directory has succeeded, i.e. only if there has been a change 
> to the actual library or meta data.
>
> http://cr.openjdk.java.net/~chegar/jigsaw/doubleInstall_webrev.00/webrev/
Thanks for catching this issue and adding a test case is good.
Looks fine to me.  Nit: SimpleLibrary.java L1163 - missing space before '='.
Mandy
    
    
More information about the jigsaw-dev
mailing list