[Svnmerge] [PATCH] allow the use of the avail command against a URL
Dustin J. Mitchell
dustin at zmanda.com
Tue Jun 17 18:04:49 PDT 2008
On Tue, Jun 17, 2008 at 5:01 AM, Shaddy Baddah
<shaddy_baddah at hotmail.com> wrote:
> I've always found it strange that I was required to target a subversion
> working copy when executing an svnmerge avail. I felt it would be intuitive
> to allow either a path to a working copy, or the URL location of branch.
I remember wondering the same thing, and I seem to recall figuring out
why .. but I don't recall the reason.
This patch passes all of the tests, although the python could be
tightened up a little bit, and I don't like the unnecessary print:
if not is_wc(branch_dir):
if cmd == "avail":
try:
info = get_svninfo(branch_dir)
except LaunchError:
info = None
if not info:
error('"%s" is neither a valid URL, nor a working
directory' % branch_dir)
else:
error('"%s" is not a subversion working directory' % branch_dir)
Finally, I'm not sure why this only applies to "avail", and not
"merge" -- in fact, why not to all commands? Clearly I'm missing
something.
Dustin
--
Storage Software Engineer
http://www.zmanda.com
More information about the Svnmerge
mailing list