emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[debbugs-tracker] bug#25971: closed (POSIX path separator used in cfg.mk


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#25971: closed (POSIX path separator used in cfg.mk breaks makefiles for all OS that do not use colon as path separator)
Date: Sun, 05 Mar 2017 17:57:03 +0000

Your message dated Sun, 5 Mar 2017 09:56:25 -0800
with message-id <address@hidden>
and subject line Re: bug#25971: POSIX path separator used in cfg.mk breaks 
makefiles for all OS that do not use colon as path separator
has caused the debbugs.gnu.org bug report #25971,
regarding POSIX path separator used in cfg.mk breaks makefiles for all OS that 
do not use colon as path separator
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
25971: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=25971
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: POSIX path separator used in cfg.mk breaks makefiles for all OS that do not use colon as path separator Date: Sun, 05 Mar 2017 08:03:31 +0100 User-agent: Mozilla/5.0 (X11; U; Linux x86_64; de; rv:1.9.2.13) Gecko/20101206 SUSE/3.1.7 Thunderbird/3.1.7
I do not known if this has already been reported, but the prepending of
$(srcdir)/src to the PATH in cfg.mk makes it impossible to build grep on
all OS where the path separator is different from colon.  I would suggest
to use PATH_SEPARATOR instead of hard coding the UNIX path separator making
the implicit assumption that this will DTRT for all other OS too.  The
current implementation ruins the syntax of the PATH because of mixed path
separator characters and inhibits make to find any program at all.
Please see patch below.

Regards,
Juan M. Guerrero



diff -aprNU5 grep-2.28.orig/cfg.mk grep-2.28/cfg.mk
--- grep-2.28.orig/cfg.mk       2017-01-01 11:34:28 +0000
+++ grep-2.28/cfg.mk    2017-03-05 01:18:46 +0000
@@ -15,11 +15,11 @@
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.

 # Cause the tool(s) built by this package to be used also when running
 # commands via e.g., "make syntax-check".  Doing this a little sooner
 # would have avoided a grep infloop bug.
-export PATH := $(srcdir)/src:${PATH}
+export PATH := $(srcdir)/src$(PATH_SEPARATOR)${PATH}

 # Used in maint.mk's web-manual rule
 manual_title = GNU Grep: Print lines matching a pattern

 # Use the direct link.  This is guaranteed to work immediately, while




--- End Message ---
--- Begin Message --- Subject: Re: bug#25971: POSIX path separator used in cfg.mk breaks makefiles for all OS that do not use colon as path separator Date: Sun, 5 Mar 2017 09:56:25 -0800
pushed:

  
http://git.savannah.gnu.org/cgit/grep.git/commit/?id=70032889a81f2dd391e5053dadb9b697e94107b6
  
http://git.savannah.gnu.org/cgit/grep.git/commit/?id=dff64f9ededbea38f573e06967e85f15a1c46e4a


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]