Re: PostgreSQL Developer meeting minutes up

From: Marko Kreen <markokr(at)gmail(dot)com>
To: Markus Wanner <markus(at)bluegap(dot)ch>
Cc: Aidan Van Dyk <aidan(at)highrise(dot)ca>, Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>, Magnus Hagander <magnus(at)hagander(dot)net>, Andrew Dunstan <andrew(at)dunslane(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: PostgreSQL Developer meeting minutes up
Date: 2009-06-02 06:43:29
Message-ID: e51f66da0906012343r46aaeef1ve38a442140821412@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 6/1/09, Markus Wanner <markus(at)bluegap(dot)ch> wrote:
> a newish conversion with cvs2git is available to check here:
>
> git://www.bluegap.ch/
>
> (it's not incremental and will only stay for a few days)

+1 for the idea of replacing CVS usernames with full names.

The knowledge about CVS usernames will be increasingly obscure.

Also worth mentioning is that there is no need to assign absolutely
up-to-date email addresses, it's enough if they uniquely identify
person.

> Aidan Van Dyk wrote:
> > Yes, but the point is you want an exact replica of CVS right? You're
> > git repo should have $PostgreSQL$ and the cvs export/checkout (you do
> > use -kk right) should also have $PostgreSQL$.
>
>
> No, I'm testing against cvs checkout, as that's what everybody is used to.
>
>
> > But it's important, because on *some* files you *do* want expanded
> > "keywords" (like the $OpenBSD ... Exp $. One of the reasons pg CVS went
> > to the $PostgreSQL$ keyword (I'm guessing) was so they could explictly
> > de-couple them from other keywords that they didn't want munging on.
>
>
> I don't care half as much about the keyword expansion stuff - that's
> doomed to disappear anyway.

But this is one aspect we need to get right for the conversion.

So preferably we test it sooner not later.

I think Aidan got it right - expand $PostgreSQL$ and others that are
actually expanded on current repo, but not $OpenBSD$ and others
coming from external sources.

> What I'm much more interested in is correctness WRT historic contents,
> i.e. that git log, git blame, etc.. deliver correct results. That's
> certainly harder to check.
>
> In my experience, cvs2svn (or cvs2git) does a pretty decent job at that,
> even in case of some corruptions. Plus it offers lots of options to fine
> tune the conversion, see the attached configuration I've used.
>
>
> > So, I wouldn't consider any conversion good unless it had all these:
> >
>
> > As well as stuff like:
> > parsecvs-master:src/backend/access/index/genam.c: * $PostgreSQL$
>
>
> I disagree here and find it more convenient for the git repository to
> keep the "old" RCS versions - as in the source tarballs that got (and
> still get) shipped. Just before switching over to git one can (and
> should, IMO) remove these tags to avoid confusion.

I'd prefer we immediately test full conversion and not leave some
steps to last moment.

--
marko

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Fujii Masao 2009-06-02 06:45:14 Re: pg_standby -l might destory the archived file
Previous Message Heikki Linnakangas 2009-06-02 06:40:02 Re: pg_standby -l might destory the archived file