Thread (5 messages) flat view 5 messages, 4 authors, 2017-11-27

Re: [PATCH] Makefile: check that tcl/tk is installed

From: Jeff King <hidden>
Date: 2017-11-27 15:27:58

On Mon, Nov 27, 2017 at 09:24:47AM +0100, Christian Couder wrote:
quoted
Yeah, this side-steps the "other half" of the issue that Christian's
patch addresses, which seems like the more controversial part (I don't
have a strong opinion myself, though).
I don't think any part of my patch should be controversial. I
repeatedly wrote very long messages to show all the possible cases, so
that it is easy to see that we are not worse in any case. And all the
competing suggestions, even the above from Junio either have
significant problems or address a different problem.
Sorry, controversial may not have been the right word. It's just that
the review discussion focused around packagers who may want to build git
without having "wish" on the build machine. If everybody is happy with
the BYPASS mechanism you added to address that, then I'm perfectly fine
with it.

-Peff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help