[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: libtool.m4-mingw-lib-searchdirs-01.patch
From: |
Guido Draheim |
Subject: |
Re: libtool.m4-mingw-lib-searchdirs-01.patch |
Date: |
Wed, 01 Aug 2001 11:24:46 +0200 |
Tim Van Holder wrote:
>
> On 31 Jul 2001 22:57:49 +0200, Guido Draheim wrote:
> >
> > I noticed later that it is a better idea to just use $PATH_SEPARATOR
> > in the last sed-part and make sure that this very variable does exist.
> > Currently it is implicitly done through initialisation order, but
> > one could even attach an AC_REQUIRE at the top of this very DEFUN.
>
> Note that libtool-cvs now requires autoconf 2.50 or higher, which
> already
> sets $PATH_SEPARATOR as part of its initialisation (it also sets
> $ac_path_separator, which would be preferred as variable inside autoconf
> macros).
> I'll try to send in a patch to drop _LT_AC_LIBTOOL_SYS_PATH_SEPARATOR
> and
> use the autoconf var instead of $PATH_SEPARATOR this evening (unless
> there is a good reason to keep using PATH_SEPARATOR instead).
okay, it seems a better way. that _LT_*_PATH_SEPARATOR isn't used near
AC_LIBTOOL_SYS_DYNAMIC_LINKER ever after that 1.202 change - just the
variable which could easily be replaced by anything with same intentions.
>
> > I talked about it later on, and I did think that it would be done as
> > last communicated - see the last mail from Gary (!!!!!) that does also
> > contain `sed -e /-print-search-dirs/s,/;/,/$PATH_SEPARATOR/,` as the
>
> This would need the sed command to be double-quoted; otherwise a
> semicolon would be seen as a command separator by the shell.
>
sure. Gary's done it right. (libtool.m4#1.202)