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

Giovanni Bajo rasky at develer.com
Fri May 12 08:28:56 PDT 2006


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").
-- 
Giovanni Bajo




More information about the Svnmerge mailing list