libtool-patches
[Top][All Lists]
Advanced

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

Re: Use the pathsep deduced by autoconf


From: Akim Demaille
Subject: Re: Use the pathsep deduced by autoconf
Date: 07 Sep 2001 10:01:06 +0200
User-agent: Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.4 (Artificial Intelligence)

| This drops libtool's pathsep macro, and uses autoconfs
| relevant variable throughout.
| Note that we'd probably want to reformat AC_PATH_TOOL_PREFIX
| to match auoconf 2.5x's AC_PATH_* family at some point.
| 
| 2001-08-01  Tim Van Holder  <address@hidden>
| 
|       * libtool.m4 (_LT_AC_LIBTOOL_SYS_PATH_SEPARATOR): Remove.
|       (_LT_AC_PROG_ECHO_BACKSLASH): Use $ac_path_separator
|       instead of $PATH_SEPARATOR.
|       (AC_LIBTOOL_SYS_DYNAMIC_LINKER): Ditto.
|       (_LT_AC_TAGCONFIG): Ditto.
|       (AC_ENABLE_SHARED): Ditto.
|       (AC_ENABLE_STATIC): Ditto.
|       (AC_ENABLE_FAST_INSTALL): Ditto.
|       (AC_PATH_TOOL_PREFIX): Ditto.
|       (AC_PATH_MAGIC): Ditto.
|       (AC_PROG_LD): Ditto.
|       (AC_PROG_NM): Ditto.
|       * ltdl.m4 (AC_LTDL_SYSSEARCHPATH): Ditto.

I disagree with this patch.  Using Autoconf's PATH_SEPARATOR is fine,
it's documented and will be maintained.  Do not depend on private
undocumented variables such as ac_path_separator.  FYI, it will
disappear since that code will move into M4sh.

Never ever *EVER* use an Autoconf private variable.

Autoconf itself should use PATH_SEPARATOR only, thanks for the bug
report ;)



reply via email to

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