Re: [PATCH] PATH: quote $(PATH_SEPARATOR) as well

2014-12-19 Thread Stefano Lattarini 
On 11/22/2014 08:20 AM, KO Myung-Hun wrote:
 Ping ?
 
 KO Myung-Hun wrote:
 On OS/2, $(PATH_SEPARATOR) is ';'. Without quote, it is recognized as
 a mark of end of sentence.

 * Makefile.am: quote $(PATH_SEPARATOR) as well.
 * t/Makefile.inc: Likewise.
 ---
   Makefile.am| 2 +-
   t/Makefile.inc | 4 ++--
   2 files changed, 3 insertions(+), 3 deletions(-)

 diff --git a/Makefile.am b/Makefile.am
 index 7e1c748..ba4179c 100644
 --- a/Makefile.am
 +++ b/Makefile.am
 @@ -72,7 +72,7 @@ generated_file_finalize = $(AM_V_at) \
   # For some tests or targets, we need to have the just-build automake and
   # aclocal scripts avaiable on PATH.
   extend_PATH = \
 -  { PATH='$(abs_builddir)/t/wrap'$(PATH_SEPARATOR)$$PATH  export PATH; }
 +  { PATH='$(abs_builddir)/t/wrap$(PATH_SEPARATOR)'$$PATH  export PATH; }
   
   # The master location for INSTALL is lib/INSTALL.
   # This is where make fetch will install new versions.
 diff --git a/t/Makefile.inc b/t/Makefile.inc
 index 5f10d90..62613ac 100644
 --- a/t/Makefile.inc
 +++ b/t/Makefile.inc
 @@ -64,9 +64,9 @@ AM_TESTS_ENVIRONMENT += stderr_fileno_=9; export 
 stderr_fileno_;
   # in case it is given with a relative name containing no slashes.
   AM_TESTS_ENVIRONMENT += \
 if test $(srcdir) != .; then \
 -PATH='$(abs_srcdir)/%D%/ax'$(PATH_SEPARATOR)$$PATH; \
 +PATH='$(abs_srcdir)/%D%/ax$(PATH_SEPARATOR)'$$PATH; \
 fi; \
 -  PATH='$(abs_builddir)/%D%/ax'$(PATH_SEPARATOR)$$PATH; \
 +  PATH='$(abs_builddir)/%D%/ax$(PATH_SEPARATOR)'$$PATH; \
 export PATH;
   
   # Hand-written tests.
 
Patch applied (with apologies for the delay).

Re: [PATCH] PATH: quote $(PATH_SEPARATOR) as well

2014-11-21 Thread KO Myung-Hun 
Ping ?

KO Myung-Hun wrote:
 On OS/2, $(PATH_SEPARATOR) is ';'. Without quote, it is recognized as
 a mark of end of sentence.
 
 * Makefile.am: quote $(PATH_SEPARATOR) as well.
 * t/Makefile.inc: Likewise.
 ---
  Makefile.am| 2 +-
  t/Makefile.inc | 4 ++--
  2 files changed, 3 insertions(+), 3 deletions(-)
 
 diff --git a/Makefile.am b/Makefile.am
 index 7e1c748..ba4179c 100644
 --- a/Makefile.am
 +++ b/Makefile.am
 @@ -72,7 +72,7 @@ generated_file_finalize = $(AM_V_at) \
  # For some tests or targets, we need to have the just-build automake and
  # aclocal scripts avaiable on PATH.
  extend_PATH = \
 -  { PATH='$(abs_builddir)/t/wrap'$(PATH_SEPARATOR)$$PATH  export PATH; }
 +  { PATH='$(abs_builddir)/t/wrap$(PATH_SEPARATOR)'$$PATH  export PATH; }
  
  # The master location for INSTALL is lib/INSTALL.
  # This is where make fetch will install new versions.
 diff --git a/t/Makefile.inc b/t/Makefile.inc
 index 5f10d90..62613ac 100644
 --- a/t/Makefile.inc
 +++ b/t/Makefile.inc
 @@ -64,9 +64,9 @@ AM_TESTS_ENVIRONMENT += stderr_fileno_=9; export 
 stderr_fileno_;
  # in case it is given with a relative name containing no slashes.
  AM_TESTS_ENVIRONMENT += \
if test $(srcdir) != .; then \
 -PATH='$(abs_srcdir)/%D%/ax'$(PATH_SEPARATOR)$$PATH; \
 +PATH='$(abs_srcdir)/%D%/ax$(PATH_SEPARATOR)'$$PATH; \
fi; \
 -  PATH='$(abs_builddir)/%D%/ax'$(PATH_SEPARATOR)$$PATH; \
 +  PATH='$(abs_builddir)/%D%/ax$(PATH_SEPARATOR)'$$PATH; \
export PATH;
  
  # Hand-written tests.

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.7.2
Under OS/2 Warp 4 for Korean with FixPak #15
In VirtualBox v4.1.32 on Intel Core i7-3615QM 2.30GHz with 8GB RAM

Korean OS/2 User Community : http://www.ecomstation.co.kr