Allow remote repository to be located via non-http URI

[Alan Bateman]

On 07/02/2012 11:30, Alan Bateman wrote:
> On 07/02/2012 11:05, Chris Hegarty wrote:
>
>> The error handling in SimpleLibrary$RepoList.add could be more robust 
>> to handle this by removing the repoDir ( and its contents ) and 
>> possibly decrementing the count?
> I agree the error handling should be more robust and it should 
> rollback the changes but this is an existing problem. I'd prefer to 
> separate it from this patch. I also think there is a lot more than 
> could be done with the repository support, it's very much a prototype 
> implementation at present.
>
Just to follow up this discussion. The issue with not cleaning up after 
a failed attempt to add a remote repository is annoying so I've put a 
patch to fix it here:

http://cr.openjdk.java.net/~alanb/jigsaw-reporollback/webrev/

I'll add tests to cover this and other scenarios where expanding the 
tests for this area (as per the discussion with Mandy).

-Alan.