Re: [PATCHv2 1/8] Makefile: apply dependencies consistently to sparse/asm targets
- From
Jeff King <peff@peff.net>
- Date
- Jun 21, 2012, 05:18 UTC
- Message-ID
- <20120621051817.GA18102@sigill.intra.peff.net>
- In-Reply-To
- <7vk3z1u4lw.fsf@alter.siamese.dyndns.org>
On Wed, Jun 20, 2012 at 01:10:19PM -0700, Junio C Hamano wrote:
Show 7 quoted lines
> Jeff King <peff@peff.net> writes: > > > Here's an updated series that drops these lines and I hope will address > > the commit message issues you brought up: > [...] > > Looks good modulo a minor nit. Will queue.
You probably noticed a lot of nice review from Jonathan. I had hoped to do another round exploring some alternatives we discussed, but didn't get to it tonight, and now I'm going to be away from email for about 5 days.
I don't think there is anything _wrong_ in the series as it is, and certainly the things we talked about changing could be built on top. So if you want to hold it in pu for a week, I can revisit it next week. But I'd also be just as happy to see it move to next, and revisit the topic later by building on top.
-Peff