Maintaining patchset with GIT (was Re: Hot standby, RestoreBkpBlocks and cleanup locks)

From: Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>
To: Simon Riggs <simon(at)2ndQuadrant(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Maintaining patchset with GIT (was Re: Hot standby, RestoreBkpBlocks and cleanup locks)
Date: 2009-01-09 17:38:09
Message-ID: 49678B81.4040609@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Simon Riggs wrote:
> Do you want me to start using the GIT repo to make it easier to
extract parts?

It would be useful. Even more so for your own sanity, I think :-). I
find maintaining multiple interdependent patches much easier with GIT,
though it's still easy to get confused.

Feel free to continue with CVS, of course.

> You'd need to show me the setup you use first.

Well, the first thing to do is to clone the repository, see wiki for
that. And get an account at git.postgresql.org so that you can publish
your stuff as a git repository. (I should get one myself..)

For reviewing hot standby, I created one "hotstandbyv6a" branch, and
applied and committed all the patches in right order. But if you want to
keep the patches separate, you should create a separate branch for each.

If patch B depends on patch A, create branch A first, and then branch
branch B from that. That's what I did for the relation forks and FSM
work. Just remember to always commit to the right branch. Whenever you
commit changes to branch A, also merge those changes to branch B with
"git checkout B; git merge A". To sync with PostgreSQL CVS HEAD: "git
merge origin/master"

To generate diffs, you can do for example "git diff A..B" to create a
diff between A and B, or "git diff origin/master..A" to create a diff
between PostgreSQL CVS HEAD and A. "git log" is also very helpful.

There's a learning curve, but don't hesitate to ask.

--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2009-01-09 17:40:03 Re: pgindent does a pretty awful job with function-pointer typedefs
Previous Message Heikki Linnakangas 2009-01-09 17:34:01 Re: Hot standby, RestoreBkpBlocks and cleanup locks