[PATCH 0 of 8] Move bookmarks extension to core

Pierre-Yves David pierre-yves.david at logilab.fr
Wed Jan 19 12:28:54 CST 2011


On Wed, Jan 19, 2011 at 11:37:46AM -0600, Augie Fackler wrote:
> On Wed, Jan 19, 2011 at 6:47 AM, David Soria Parra <dsp at php.net> wrote:
> > On 2011-01-18, Henrik Stuart <hg at hstuart.dk> wrote:
> >> We should definitely allow non-ascii characters; I'd propose we do what
> >> we do for branch names, plus a bit more, i.e., disallow "tip", ".", and
> >> "null", and disallowing [\r\n\t\v\0] seems prudent as well, however,
> >> both tag and branch just disallows (tip|\.|null) so that should be the
> >> least that bookmarks disallows.
> > make sense, I'll change this in the next iteration. I'll wait for more review before
> > I reroll the series.
> 
> I'd also like to disallow : if nobody objects. It's illegal in git
> refspecs (I believe) and it'd be useful for certain advanced features
> (say, 'hg push -B foo:bar where you want foo to be renamed to bar on
> the server). Does that seem reasonable?

My vote probably doesn't weight much, but I'll be +1 for any proposition that
help git/hg interoperability. That ease the jobs of bridge as hg-git and allow
people to pick tools they prefer in each world.

--
Pierre-Yves David
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20110119/f8d96711/attachment.pgp>


More information about the Mercurial-devel mailing list