Re: [bug-recutils] [platform-testers] recutils pre-release 1.6.90 in alpha.gnu.org

[Jose E. Marchesi]

    Am 20.03.2014 um 22:09 schrieb Jose E. Marchesi <address@hidden>:
    >    Am 19.03.2014 um 20:47 schrieb Jose E. Marchesi <address@hidden>:
    >> The pre-release recutils-1.6.90.tar.gz is now available at
    >> ftp://alpha.gnu.org/gnu/recutils.  The NEWS file in the tarball
    >> contains a list of the changes since 1.6.
    >> 
    >> The planned date for releasing 1.7 is Sunday 23 March 2014.
    >> 
    >> Please report any problem found with the pre-release to
    >> address@hidden
    > 
    >    I just tested on Solaris 10 Sparc with Sun Studio 12 and there seems
    >    to be something wrong at the end of configure:
    > 
    >    checking pkg-config is at least version 0.9.0... yes
    >    checking for GLIB... yes
    >    checking for mdb_init in -lmdb... no
    >    
/home/dam/mgar/pkg/recutils/trunk/work/solaris10-sparc/build-isa-sparcv8plus/recutils-1.6.90/configure:
 test: argument expected
    > 
    > Hm, it is a shame that it does not report the faulting line in
    > configure.  Could you please run the configure script with -x to see
    > which test is in fault?
    
    Sure, here are the last few lines:
    
    + test xyes = xyes
    UUID_TYPE_TRUE=
    UUID_TYPE_FALSE=#
    BASH_HEADERS=/usr/include/bash
    + test  = set
    + test -f /usr/include/bash/config.h
    bash_headers_available=no
    + test  = set
    bash_builtins_enabled=yes
    + test xno = xyes
    BASH_BUILTINS_TRUE=#
    BASH_BUILTINS_FALSE=
    + test  = set
    algol68_enabled=no
    + test xno = xyes
    + test -n
    
/home/dam/mgar/pkg/recutils/trunk/work/solaris10-sparc/build-isa-sparcv8plus/recutils-1.6.90/configure:
    test: argument expected

Ah, I think I just found the culprit:

diff --git a/configure.ac b/configure.ac
index c388579..05b8366 100644
--- a/configure.ac
+++ b/configure.ac
@@ -191,7 +191,7 @@ if test "x$algol68_enabled" = "xyes"; then
 fi
 
 AC_SUBST([CA68])
-AM_CONDITIONAL([BUILD_A68],[test -n $CA68 && test "x$algol68_enabled" = 
"xyes"])
+AM_CONDITIONAL([BUILD_A68],[test -n "$CA68" && test "x$algol68_enabled" = 
"xyes"])
 
 dnl Platform-based compilation options
 compile_w32_system=no