[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