Re: [PATCH v2 0/2] DEFAULT_DOC_TARGET

2013-01-03 Thread Jeff King
On Thu, Jan 03, 2013 at 02:20:31PM -0800, Junio C Hamano wrote: > Jeff King writes: > > > Not surprising for such a large refactoring, but this has conflicts with > > what's in next. Here's the patch to apply on top of the conflicted tree > > you get from merging this with "next": > > Yeah, ver

Re: [PATCH v2 0/2] DEFAULT_DOC_TARGET

2013-01-03 Thread Junio C Hamano
Jeff King writes: > Not surprising for such a large refactoring, but this has conflicts with > what's in next. Here's the patch to apply on top of the conflicted tree > you get from merging this with "next": Yeah, verifying the manual fixup is a bit tricky indeed. The output from $ git blame

Re: [PATCH v2 0/2] DEFAULT_DOC_TARGET

2013-01-03 Thread Jeff King
On Thu, Jan 03, 2013 at 04:05:41PM -0500, Jeff King wrote: > On Thu, Jan 03, 2013 at 03:32:22PM -0500, Jeff King wrote: > > > I think the usefulness is that it can be set by default for a particular > > uname, so people on Windows can just type "make install-doc" without > > having to care about

Re: [PATCH v2 0/2] DEFAULT_DOC_TARGET

2013-01-03 Thread Jeff King
On Thu, Jan 03, 2013 at 03:32:22PM -0500, Jeff King wrote: > I think the usefulness is that it can be set by default for a particular > uname, so people on Windows can just type "make install-doc" without > having to care about setting anything (though to be honest, I do not > even know what they

Re: [PATCH v2 0/2] DEFAULT_DOC_TARGET

2013-01-03 Thread Jeff King
On Thu, Jan 03, 2013 at 03:32:22PM -0500, Jeff King wrote: > On Thu, Jan 03, 2013 at 11:05:18AM -0800, Junio C Hamano wrote: > > > This allows things like: > > > > $ DEFAULT_DOC_TARGET=html make doc > > $ DEFAULT_DOC_INSTALL_TARGET=html make install-doc > > > > on a platform that does not h

Re: [PATCH v2 0/2] DEFAULT_DOC_TARGET

2013-01-03 Thread Jeff King
On Thu, Jan 03, 2013 at 11:05:18AM -0800, Junio C Hamano wrote: > This allows things like: > > $ DEFAULT_DOC_TARGET=html make doc > $ DEFAULT_DOC_INSTALL_TARGET=html make install-doc > > on a platform that does not have manpage viewer. Which is not very > useful, given that you can already

[PATCH v2 0/2] DEFAULT_DOC_TARGET

2013-01-03 Thread Junio C Hamano
This allows things like: $ DEFAULT_DOC_TARGET=html make doc $ DEFAULT_DOC_INSTALL_TARGET=html make install-doc on a platform that does not have manpage viewer. Which is not very useful, given that you can already say $ make install-html on such a platform, and these install-$format targe