Re: [patch] Fix for AC_FUNC_LSTAT_FOLLOWS_SLASHED_SYMLINK

[Jim Meyering]

Tim Van Holder <address@hidden> wrote:
| When preparing the DJGPP package for 2.50, I noticed test 143
| (AC_FUNC_STAT) failed.  Cause: it found conftest.sym.exe.
|
| As it turns out, AC_FUNC_LSTAT_FOLLOWS_SLASHED_SYMLINK uses
| ln -s without considering DJGPP (where symlinks are currently only
| supported for executables, resulting in a .exe file being created).
|
| Two alternatives are provided; one knows about the possibility
| of a .exe file appearing and handles it; the other uses $as_ln_s to
| determine whether running ln -s makes sense in the first place.
| Both replace the use of conftest.* with conf$$.*.
|
| 2001-05-22  Tim Van Holder <address@hidden>
|
|     * acfunctions.m4: (AC_FUNC_LSTAT_FOLLOWS_SLASHED_SYMLINK) Use
|     ln -s in a portable manner.

Thanks Tim!
FWIW, I prefer #2, but with this tiny addition:

  AC_REQUIRE([AC_PROG_LN_S])dnl

By the way, it's a little easier to browse/compare substantive changes
(like your fix) if they're separate from the cleaning-up ones
like s/conftest/conf$$/.

Jim