[MERGE] Add a 'nosmart+' transport decorator.

Martin Pool mbp at canonical.com
Thu Apr 3 02:24:30 BST 2008


On undefined, Andrew Bennetts <andrew at canonical.com> wrote:

>  Well, we already have "bzr+ssh", for instance (and I don't see an RFC
>  for "svn+ssh"...).  That's not technically a decorator in bzrlib, but a
>  self-contained transport.  But the precedent for using "+" in documented
>  and intended-for-public-consumption URLs in bzr is already there.
>
>  Mainly our decorators are used for testing at the moment.  The currently
>  defined prefixes we have are: "readonly+", "fakenfs+", "trace+",
>  "unlistable+", "brokenrename+", and "vfat+".  It might be good for us to
>  rename all our decorators to be more conformant with BCP 35, by changing
>  the "+" to a "-" (although I'd be slightly sad to lose the mnemoic of "x
>  plus y").  I think such a change is out of scope for this patch though.

I think of both + and - as having different meanings in urls as they
are commonly used: plus indicating stacking, as we use it, and dash
joining words within a name.  Our current use is pretty consistent
with the precedents.

+1 to nosmart

-- 
Martin <http://launchpad.net/~mbp/>



More information about the bazaar mailing list