This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: GIT: Monotonically increasing trunk and release branch ids
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Jason Merrill <jason at redhat dot com>
- Cc: "Richard Earnshaw (lists)" <Richard dot Earnshaw at arm dot com>, "Joseph S. Myers" <joseph at codesourcery dot com>, Jonathan Wakely <jwakely at redhat dot com>, gcc Mailing List <gcc at gcc dot gnu dot org>
- Date: Sat, 11 Jan 2020 15:29:39 +0100
- Subject: Re: GIT: Monotonically increasing trunk and release branch ids
- References: <20200110163852.GV10088@tucnak> <CADzB+2=U=WBqu7uc37UrW_YOt45eOKEKhz=Fdg9vwUN1NGY8kQ@mail.gmail.com> <20200110214747.GD10088@tucnak> <20200111112801.GF10088@tucnak> <CADzB+2k9gOw8HPaN63n+Hk6Hc0eQmbS6-NKSpo=sr7mZcCMkKQ@mail.gmail.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Sat, Jan 11, 2020 at 09:15:11AM -0500, Jason Merrill wrote:
> > On Fri, Jan 10, 2020 at 10:47:47PM +0100, Jakub Jelinek wrote:
> > > Ah, you suggested g: rather than just g.
> > > We could then support
> > > rNNNNN (1-6 decimal digits) - the svn revs, either for old repo, or
> > transformed
> > > g:XXXXX (X is any [0-9a-zA-Z_-], something else?, 1 or more chars) -
> > gitweb with XXXXX
> >
>
> I was thinking that anything at all after g: should go to gitweb.
For the redirect perhaps. For bugzilla comments, it needs to know where it
does stop and shouldn't match unrelated text, plus as I wrote in the
bugzilla tweaks PR, not sure how smart are e.g. g:master or g:master^^ or
g:releases/gcc-9@{yesterday} references (and, there we at least shouldn't
match whitespace).
> > > rNN-MMMMM (1-2 decimal digits, 1-6 decimal digits) - git undescr to hash
> > > rNN-MMMMM-gXXXXX (perhaps limit X to hashes here, i.e. 7-40 hex digits)
> > - gitweb with XXXXX
> >
>
> Why complicate this with the added 'g'?
The advantage of the g in there is that git then can directly understand it:
git rev-parse r9-7160-ga5bc16f3199
a5bc16f3199c9aa43aec5af2c2839a1cd4bfce1e
while if the g is removed, it doesn't.
Jakub