Subject: bug#35879: closed (Re: [bug-diffutils] bug#35879: [PATCH] tests: init: use $PATH_SEPARATOR instead of hard-coded colon(:))

From: [email-masked] (GNU bug Tracking System)
Date: Fri, 24 May 2019 20:39:02 +0000

Your bug report

#35879: [PATCH] tests: init: use $PATH_SEPARATOR instead of hard-coded colon(:)

which was filed against the diffutils package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
35879: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=35879
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: [bug-diffutils] bug#35879: [PATCH] tests: init: use $PATH_SEPARATOR instead of hard-coded colon(:)
From: Paul Eggert <[email-masked]>
To: KO Myung-Hun <[email-masked]>
Cc: [email-masked]
Date: Fri, 24 May 2019 13:38:16 -0700
List-Id: <debbugs-submit.debbugs.gnu.org>

Thanks, I installed that. You might consider attaching git-format 
patches instead of sending them directly, as gmail seems to be altering 
the "From:" line.

Subject: [PATCH] tests: init: use $PATH_SEPARATOR instead of hard-coded colon(:)
From: KO Myung-Hun <[email-masked]>
To: [email-masked]
Date: Fri, 24 May 2019 18:38:53 +0900
List-Id: <debbugs-submit.debbugs.gnu.org>

On OS/2, a path separator is semi-colon(;) not colon(:).

* tests/init.cfg: Replace hard-coded colon(:) with $PATH_SEPARATOR.
---
 tests/init.cfg | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/tests/init.cfg b/tests/init.cfg
index d563156..c1f7941 100644
--- a/tests/init.cfg
+++ b/tests/init.cfg
@@ -33,7 +33,7 @@ sanitize_path_()
   # FIXME: remove double quotes around $IFS when all tests use init.sh.
   # They constitute a work-around for a bug in FreeBSD 8.1's /bin/sh.
   local saved_IFS="$IFS"
-    IFS=:
+    IFS=$PATH_SEPARATOR
     set -- $PATH
   IFS=$saved_IFS
 
@@ -44,14 +44,14 @@ sanitize_path_()
     test -z "$d" && d1=. || d1=$d
     if ls -d "$d1/." > /dev/null 2>&1; then
       new_path="$new_path$colon$d"
-      colon=':'
+      colon=$PATH_SEPARATOR
     fi
   done
 
   for d in /sbin /usr/sbin ; do
-    case ":$new_path:" in
-      *:$d:*) ;;
-      *) new_path="$new_path:$d" ;;
+    case "$PATH_SEPARATOR$new_path$PATH_SEPARATOR" in
+      *$PATH_SEPARATOR$d$PATH_SEPARATOR*) ;;
+      *) new_path="$new_path$PATH_SEPARATOR$d" ;;
     esac
   done
 
-- 
2.13.3