Re: pgsql: Update copyright for 2017

From: David Fetter <david(at)fetter(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, Andres Freund <andres(at)anarazel(dot)de>, Bruce Momjian <bruce(at)momjian(dot)us>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, pgsql-committers <pgsql-committers(at)postgresql(dot)org>
Subject: Re: pgsql: Update copyright for 2017
Date: 2017-01-04 20:02:30
Message-ID: 20170104200230.GC15431@fetter.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Wed, Jan 04, 2017 at 11:54:07AM -0800, David Fetter wrote:
> On Wed, Jan 04, 2017 at 02:20:52PM -0500, Tom Lane wrote:
> > David Fetter <david(at)fetter(dot)org> writes:
> > >>> Actually, my takeaway from this was "don't ever use git reset on
> > >>> the repo".
> >
> > > That's actually not tenable. If we ever find something in our
> > > repo that we don't have full rights to, especially if it's
> > > something that would put roadblocks in front of people who'd like
> > > to make a proprietary fork, we have to be able to expunge it, not
> > > merely paper it over.
> >
> > What, and re-do every commit after the one that added such material?
> > And somehow find every tarball that was shipped with the material,
> > and make them go away? Please don't bring straw-man arguments.
>
> It might sound like a straw man if we hadn't already done this once.
>
> > >> Except for like Andres says, always check *everything* before
> > >> pushing. I know I always push with -n and then do a git show on that
> > >> resulting set of commits just to make sure it's the one I want. It
> > >> doesn't take a lot of extra time after each commit, and it easily
> > >> finds things like this.
> >
> > > Do we see a point in the future where all pushes to that repo
> > > require a reviewer separate from the author? The cost in hassle
> > > and aggravation is, to put it mildly, non-trivial, but it makes
> > > these kinds of mistakes a lot harder to make.
> >
> > No amount of review will prevent human error at the point of the
> > final push.
>
> I suggest that it would.

ETOOLITTLECOFFEE

I do *not* suggest it would.

Best,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2017-01-04 21:01:49 pgsql: Assorted code improvements for table partitioning.
Previous Message Robert Haas 2017-01-04 20:02:08 pgsql: Remove unnecessary arguments from partitioning functions.

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Eisentraut 2017-01-04 20:09:35 Re: generating fmgr prototypes automatically
Previous Message David Fetter 2017-01-04 19:54:07 Re: pgsql: Update copyright for 2017