Re: guile.m4 (cvs revision 1.3)

[Alexandre Duret-Lutz]

Hi,

>>> "Thien-Thi" == Thien-Thi Nguyen <address@hidden> writes:
[...]
 Thien-Thi> AC_DEFUN([GUILE_PROGS],
 Thien-Thi> [AC_PATH_PROG(GUILE,guile)
 Thien-Thi> if test "$GUILE" = "" ; then
 Thien-Thi> AC_MSG_ERROR([guile required but not found])
 Thien-Thi> fi
 Thien-Thi> AC_SUBST(GUILE)
 Thien-Thi> AC_PATH_PROG(GUILE_CONFIG,guile-config)
 Thien-Thi> if test "$GUILE_CONFIG" = "" ; then
 Thien-Thi> AC_MSG_ERROR([guile-config required but not found])
 Thien-Thi> fi

This allows the installer to select guile-config at
configure-time (by setting GUILE_CONFIG before running
configure), so ...

 Thien-Thi> AC_DEFUN([GUILE_FLAGS],[
 Thien-Thi> ## The GUILE_FLAGS macro.
 Thien-Thi> ## First, let's just see if we can find Guile at all.
 Thien-Thi> AC_MSG_CHECKING(for Guile)
 Thien-Thi> guile-config link > /dev/null || {

... I think this should use $GUILE_CONFIG instead of
guile-config.  This probably means moving the GUILE_CONFIG
check in a separate macro required by both GUILE_PROGS and
GUILE_FLAGS.

 Thien-Thi> echo "configure: cannot find guile-config; is Guile installed?" 1>&2
 Thien-Thi> exit 1

AC_MSG_ERROR?

 Thien-Thi> }
 Thien-Thi> GUILE_CFLAGS="`guile-config compile`"
 Thien-Thi> GUILE_LDFLAGS="`guile-config link`"

Ditto for $GUILE_CONFIG.  The double quotes are superfluous.

[...]
-- 
Alexandre Duret-Lutz