From: Jonathan Nieder <jrnieder@gmail.com>
Date: Fri, 29 Jul 2011 19:16:55 GMT
Subject: Re: [PATCH 16/18] revert: Remove sequencer state when no commits are pending
Message-ID: <20110729191654.GA2368@elie.dc0b.debconf.org>
In-Reply-To: <CALkWK0mUGiCRQoW6UM+mF5tsn4DxRuaiXn0TS2CcEsR0OeusKg@mail.gmail.com>

Ramkumar Ramachandra wrote:
> Jonathan Nieder writes:

>> If "tight coupling" means "builtin/commit.c calls a function declared
>> in sequencer.h", then yes, I think it should be fine.
>
> My bad. I'll first have to figure out a way to expose the todo parsing
> functions (and dependent functions) without touching much else.

That's the wrong way to go.  One right way would be to expose a single
function, representing "here is what to do after successfully making a
commit".  Think of it as an invisible built-in post-commit hook
(see githooks(5)).