bug-apl
[Top][All Lists]
Advanced

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

Bug in Autoconf file


From: Gylfi
Subject: Bug in Autoconf file
Date: Fri, 26 Feb 2021 14:43:45 +0000

Hello,

I've encountered a minor bug in the configure script. I was trying to enable rational number support but the configure script kept reporting that rational numbers were not desired. It looks to me like setting RATIONAL_NUMBERS_WANTED=yes does in fact enable this feature, it's just not being reported. I believe the problem is in this snippet: (line 665 in configure.ac)

# APL: support for rational numbers ?
AC_MSG_CHECKING([whether support for rational numbers is desired ])
AC_ARG_VAR(RATIONAL_NUMBERS_WANTED,
   [ enable support for rational numbers (see README-2-configure) default: no ])
if test "x$RATIONAL_NUMBERS_WANTED" = "xyes"; then
AC_DEFINE_UNQUOTED([RATIONAL_NUMBERS_WANTED], [yes],
                   [ define to have support for rational numbers (EXPERIMENTAL!)])
else RATIONAL_NUMBERS_WANTED="no"
fi
AC_MSG_RESULT([$VISIBLE_MARKERS_WANTED])

At the bottom, the argument to AC_MSG_RESULT is $VISIBLE_MARKERS_WANTED where it seems to me it should be $RATIONAL_NUMBERS_WANTED (VISIBLE_MARKERS_WANTED is the previous option, maybe this block was duplicated and the AC_MSG_RESULT line was accidentally left unchanged). Indeed, after making these changes and generating a new configure script, the output changes depending on RATIONAL_NUMBERS_WANTED, as expected.

Best regards,
Gylfi

reply via email to

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