From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.org/gmane.linux.lib.musl.general/10597 Path: news.gmane.org!.POSTED!not-for-mail From: Ramsay Jones Newsgroups: gmane.comp.version-control.git,gmane.linux.lib.musl.general Subject: Re: Regression: git no longer works with musl libc's regex impl Date: Thu, 6 Oct 2016 23:42:01 +0100 Message-ID: <20336ac7-a494-d725-f928-834b1b3194fe@ramsayjones.plus.com> References: <20161004150848.GA7949@brightrain.aerifal.cx> <20161004152722.ex2nox43oj5ak4yi@sigill.intra.peff.net> <20161004154045.GT19318@brightrain.aerifal.cx> NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Trace: blaine.gmane.org 1475793769 13939 195.159.176.226 (6 Oct 2016 22:42:49 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Thu, 6 Oct 2016 22:42:49 +0000 (UTC) User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 Cc: Rich Felker , Jeff King , Git , musl@lists.openwall.com To: =?UTF-8?B?w4Z2YXIgQXJuZmrDtnLDsCBCamFybWFzb24=?= , Johannes Schindelin Original-X-From: git-owner@vger.kernel.org Fri Oct 07 00:42:45 2016 Return-path: Envelope-to: gcvg-git-2@m.gmane.org Original-Received: from vger.kernel.org ([209.132.180.67]) by blaine.gmane.org with esmtp (Exim 4.84_2) (envelope-from ) id 1bsHMk-0007hJ-N6 for gcvg-git-2@m.gmane.org; Fri, 07 Oct 2016 00:42:15 +0200 Original-Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935408AbcJFWmL (ORCPT ); Thu, 6 Oct 2016 18:42:11 -0400 Original-Received: from avasout02.plus.net ([212.159.14.17]:47076 "EHLO avasout02.plus.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933672AbcJFWmI (ORCPT ); Thu, 6 Oct 2016 18:42:08 -0400 Original-Received: from [10.0.2.15] ([194.75.29.46]) by avasout02 with smtp id sNi21t0070zhorE01Ni3Eq; Thu, 06 Oct 2016 23:42:06 +0100 X-CM-Score: 0.00 X-CNFS-Analysis: v=2.2 cv=G/5eKJs5 c=1 sm=1 tr=0 a=g54qAj+LxVGqXy9pVcJ+0w==:117 a=g54qAj+LxVGqXy9pVcJ+0w==:17 a=IkcTkHD0fZMA:10 a=S3BrH6ej_XDh2nDupuMA:9 a=QEXdDO2ut3YA:10 X-AUTH: ramsayjones@:2500 In-Reply-To: Original-Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Xref: news.gmane.org gmane.comp.version-control.git:306371 gmane.linux.lib.musl.general:10597 Archived-At: On 06/10/16 20:18, Ævar Arnfjörð Bjarmason wrote: > On Tue, Oct 4, 2016 at 6:08 PM, Johannes Schindelin > wrote: >> As to making NO_REGEX conditional on REG_STARTEND: you are talking about >> apples and oranges here. NO_REGEX is a Makefile flag, while REG_STARTEND >> is a C preprocessor macro. >> >> Unless you can convince the rest of the Git developers (you would not >> convince me) to simulate autoconf by compiling an executable every time >> `make` is run, to determine whether REG_STARTEND is defined, this is a >> no-go. > > But just to clarify, does anyone have any objection to making our > configure.ac compile a C program to check for this sort of thing? > Because that seems like the easiest solution to this class of problem. Err, you do know that we already do that, right? [see commit a1e3b669 ("autoconf: don't use platform regex if it lacks REG_STARTEND", 17-08-2010)] In fact, if you run the auto tools on cygwin, you get a different setting for NO_REGEX than via config.mak.uname. Which is why I don't run configure on cygwin. :-D [The issue is exposed by t7008-grep-binary.sh, where the cygwin native regex library matches '.' in a pattern with the NUL character. ie the test_expect_failure test passes.] ATB, Ramsay Jones