Re: [PATCH 0/5] Update compat/regex
From: Ævar Arnfjörð Bjarmason <hidden>
Date: 2016-06-15 22:49:20
2010/8/17 Jonathan Nieder [off-list ref]:
Junio C Hamano wrote:quoted
Hmm, is there [1/5] in the series?Yes, but it probably missed the list because of vger's message length limits. For convenience, here is the series. I also think the first two patches should be combined, but this does not do so because others might disagree. Patch 5 (autoconf) is the v2 version.
Pedantic: If there's going to be squashing that should probably include the regerror change too, otherwise it can't be bisected on MSVC either. Personally I think nothing should be squashed, because the history of having 1 commit be the unaltered gawk library and the rest being the changes we need on top of it will help future developers when we need to update the regex library again. And since most people hacking git probably do so on GNU or the BSD's which don't need NO_REGEX=YesPlease the practical downside of doing so is *very* little.