From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Erik Rijkers <er(at)xs4all(dot)nl>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: adding the commit to a patch's thread |
Date: | 2017-09-03 17:33:04 |
Message-ID: | CABUevExSYas-2dAPJ2T12CM4j75RY8QU79hfxdc0SgCyxsw0ZQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sun, Sep 3, 2017 at 12:00 AM, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
> > On 01 Sep 2017, at 15:40, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> >
> > On Fri, Sep 1, 2017 at 4:26 AM, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
> wrote:
> >> Erik Rijkers wrote:
> >>> Would it be possible to change the commitfest a bit and make it
> possible to
> >>> add the commit (or commit-message, or hash) to the thread in the
> >>> commitfest-app.
> >>
> >> +1 to add one or more commit hashes to CF entry metadata.
> >>
> >> (Back-filling for old entries welcome)
> >
> > Couldn't the CF app scrape the commit messages for references to
> > threads, and if the commit message points to a thread with exactly 1
> > patch record, associate the commit to that patch?
>
> Since there is a Gitlink field in the patch metadata, we could start simple
> with setting that to link to the commit on pg gitweb? While adding the
> thread
> of the commit from -committers would be great as well, this gets us off the
> ground quickly.
>
The original idea behind the gitlink field was to link to a git repo/branch
representing the patch, for people who preferred to publish full branches
for those that want it.
This has been done for a grand total of 43 patches throughout (out of a
total of 1231).
Not sure if that's enough to say "let's not repurpose it"?
--
Magnus Hagander
Me: https://www.hagander.net/ <http://www.hagander.net/>
Work: https://www.redpill-linpro.com/ <http://www.redpill-linpro.com/>
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2017-09-03 18:25:16 | Re: JIT & function naming |
Previous Message | Tom Lane | 2017-09-03 14:36:36 | Re: Fix warnings and typo in dshash |