[PATCH 2 of 4 RFC V2] localrepo: add "wc" symbol to specify workingctx by command (BC)

Augie Fackler raf at durin42.com
Mon Mar 16 12:29:22 CDT 2015


On Tue, Mar 17, 2015 at 12:16:49AM +0900, Yuya Nishihara wrote:
> On Mon, 16 Mar 2015 10:50:08 -0400, Jordi GutiƩrrez Hermoso wrote:
> > On Mon, 2015-03-16 at 23:04 +0900, Yuya Nishihara wrote:
> > > # HG changeset patch
> > > # User Yuya Nishihara <yuya at tcha.org>
> > > # Date 1408247327 -32400
> > > #      Sun Aug 17 12:48:47 2014 +0900
> > > # Node ID 3e97838d4eaa3f4f5131f1f1997ead72acc81aad
> > > # Parent  193f5a309da2beea925fd22a6e039bb2dcee7a81
> > > localrepo: add "wc" symbol to specify workingctx by command (BC)
> >
> > I really like where this is going... but introducing more syntax...?
> >
> > > Pros and cons:
> > >
> > >   "wc" symbol:
> > >    - is easy to type
> > >    - can be used as a first-class symbol to specify wctx by string
> > >      (would avoid problems such as caff3675cba5)
> > >   "wc()" function:
> > >    - has no behavior change
> >
> > Big contra: it is short enough to conflict with existing bookmarks,
> > tags, or revset aliases the user has defined. Smaller contra: "wc"
> > means "toilet" or "bathroom" in several European languages.
>
> And in Japanese, yes, it means toilet. I just picked "wc" from subrepo.py.
> Other possible words: wdir, wdir(), workingdir(), ...
>
> > Since "." is used for p1, how about making "./" refer to your proposed
> > "wc"? This does not conflict with existing syntax, is short, and I
> > believe cannot be the name of a revset alias. It also looks like
> > "current directory" in Unix-speak.
>
> "./" sounds too magical for me. It doesn't look like a single symbol.

Agreed. Also, in filesystems . and ./ mean the same thing, so that'd
be confusing too.

>
> Regards,
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at selenic.com
> http://selenic.com/mailman/listinfo/mercurial-devel


More information about the Mercurial-devel mailing list