From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: should I post the patch as committed? |
Date: | 2010-04-20 17:07:02 |
Message-ID: | 201004201707.o3KH72p08623@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Magnus Hagander wrote:
> On Tue, Apr 20, 2010 at 18:30, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> > Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> >> Bruce suggested to me off-list that when I commit a patch with
> >> revisions, I ought to attach the committed patch to the email in which
> >> I say that I have committed it. ?I know that Bruce habitually does
> >> this, but I've never found it very helpful, since I prefer to look at
> >> the patch either using the pgsql-committers emails or the git
> >> respository.
> >
> > I quite agree --- people who want to read the committed patch are going
> > to go to the source, not to what somebody claims he committed.
>
> Yeah, +1. I always go to one of the other sources (most of the time,
> the gitweb or git server).
>
> It doesn't hurt to attach it if it happens to be around, but I
> wouldn't suggest spending any extra effort on it...
Well, many times I am applying a patch from an author that I have
modified, and I don't expect them to be subscribed to committers, so I
supply the patch as a courtesy so they can see my changes (and fix
them). :-)
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2010-04-20 17:07:13 | Re: should I post the patch as committed? |
Previous Message | Magnus Hagander | 2010-04-20 17:04:41 | Re: should I post the patch as committed? |