Michael Witten <mfwitten@xxxxxxx> writes: > On 30 Nov 2007, at 9:37:01 PM, Junio C Hamano wrote: > >> I'll queue your patch, but I think it should be enhanced to support >> post-update for consistency. > > I'll send another patch that includes support for post-update. > >> I'll queue your patch, > > Will the old patch already be in place? Although I would encourage total replacement patch for things not in 'next' yet, in this case, I think we both would prefer a separate patch that builds on top of the old patch that teaches the server that it should call post-update as well. I think it is potentially a sensible thing to do to eventually deprecate and then remove post-update hook (as your first patch that is queued already suggests). I was just saying that 'eventually' is not now. When that time comes, if you have a separate patch to teach post-update, that single patch can be reverted independently from the old patch, which should be less error prone than yanking the support out by hand. - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
