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