[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