[Svnmerge] [PATCH] make avail andintegrated accepts URLs asarguments

Mattias Brändström thebrasse at brasse.org
Sat May 20 06:11:01 PDT 2006


Giovanni Bajo wrote:
> Yes, your last version is fine. You're still missing a comment near "False" for
> "help" though: it's needed because a bare False really doesn't mean anything.
> You also want to update the usage line of integrated and avail in their help
> screen: they say [PATH] but should now say [PATH or URL].
> 
> Send me an updated version and I will commit it.
> 

I have been thinking some more about your suggestion to make the 
"accepts url" flag a bit more descriptive and have another version of 
the patch for you all to take a look at. It s quite similar, but now the 
  information about the command accepting an URL as argument or not is 
contained in a string array in the command table. Right now this flag 
array can contain the values "WC" and/or "URL" (help's array is empty). 
This flag array, while being more descriptive, also might be used in the 
future for other command characteristics.

The patch and commit message are attached to this message.

:.:: mattias
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: svnmerge.patch
Url: /pipermail/svnmerge/attachments/20060520/845f799d/attachment.txt 
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: svnmerge-log-message
Url: /pipermail/svnmerge/attachments/20060520/845f799d/attachment-0001.txt 


More information about the Svnmerge mailing list