[Svnmerge] [PATCH] make avail and integrated accepts URLs asarguments

[Mattias Brändström]

Giovanni Bajo wrote:
> Mattias Brändström wrote:
> 
> 
>>>I have written a small patch that makes avail and integrated accept a
>>>URL as their PATH argument. No test case yet. I just wanted to show
>>>the
>>>patch early so that I might get some comments on the small changes it
>>>introduces. I took Giovanni's advice and made the ability to accept
>>>an
>>>URL as PATH argument a flag in the command table.
> 
> 
> Thanks.
> 
> Please put a comment on each line containing True/False in the options, so
> that we know what the boolean flag means. It's not immediately obvious which
> stands for "accepts URL". Another solution is to change it to a
> whitespace-separated string, so that it is self-descriptive and allows other
> variations in the future (eg: "WC", "WC URL"", "URL").

Ok. Here is my patch and a test case for it. I did not change the flag 
to a string, I think it will be easy enough to change that if it is 
needed in the future.

:.:: mattias
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: svnmerge.patch
Url: /pipermail/svnmerge/attachments/20060515/7267ab1d/attachment.txt