Thread (4 messages) flat view 4 messages, 2 authors, 2016-06-15

Re: [PATCH 7/8] mergetools: use the correct tool for Beyond Compare 3 on Windows

From: Sebastian Schuberth <hidden>
Date: 2016-06-15 22:52:16

On Sat, Oct 15, 2011 at 22:14, Junio C Hamano [off-list ref] wrote:
quoted
quoted
Hmm, does this only apply to Windows, or are there other platforms on
which BC3 supplies bcomp for the exact same reason? What I am trying to
BC3 is only available for Linux and Windows, so it only applies to
Windows currently.
Who asked anything about "currently"?
Heh, no one. But obviously I cannot predict the future, that's why I
explicitly said "currently".

Here's what I was meaning to say in more long-winded words: Although
BC3 only supports Linux and Windows currently, IMHO it is unlikely
that future support for other platforms will suffer from the same
issue as Windows and will require a different executable than
"bcompare". And even if it was the case, that could be fixed then. For
now the proposed patch covers all cases and works well, and although
your suggestion would be more generic, it's not necessary, and the
discussion about whether or not to use your more generic approach is
starting to outweigh the time that was required to write this patch as
well as the time that would be required for a future patch.

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