[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