John Levon wrote: > > On Wed, May 22, 2002 at 10:15:32PM -0300, Garst R. Reese wrote: > > > Attached is the minimalist patch with .la added. If you want to try one > > with AC_CHECK, feel free. I suppose the difference is that that check > > would not add the -L/usr/local/lib which appears to be redundant. > > Actually it's downright dangerous (well, it can be with -L/usr/lib on > some systems). > > > pspell_libs="/usr/lib /usr/lib/pspell /usr/local/lib /usr/local/lib/pspell" > > - AC_FIND_FILE(libpspell.a,$pspell_libs,pspell_use_lib) > > + AC_FIND_FILE(libpspell.la libpspell.so >libpspell.a,$pspell_libs,pspell_use_lib) > > pspell_lib_found="yes" > > Unfortunately AC_CHECK_LIBS makes it painful to look through several > directories for the library. > > I'm quite prepared to drop this search and require the user to do it, if > you agree then I can make a patch with AC_CHECK_LIB, otherwise I'll punt > on this. I really don't know enough to agree or disagree. When I first encountered the problem I just edited the top and src Makefiles to delete the -LNO that was appearing, leaving only the -lpspell, and everything worked. Then I did a grep for AC_FIND_FILE and saw the gnome stuff using the .a, .so etc stuff, so just did the same and the -LNO turned into -L/usr/local/lib, and it still worked, so I submitted the patch.
Personally, I would just check to see if any pspell include was there. There are so many ways to screw up that I think overchecking is futile. If the installation is screwed it will show up eventually. > Unfortunately our --with-extra-lib cannot take multiple dirs either. > > Did I tell anyone how much I hate maintaining autoconf code ? > a few times :)
