With USE=-ldap, the line if test ${with_ldap-'no'} != "no"; then causes ./configure to emit checking for getspnam... yes ./configure: line 22314: test: !=: unary operator expected checking for log file location... /var/log/sudo.log It is not helped by the check for --with-ldap, which sets with-ldap to "" on the "no" case: if test "${with_ldap+set}" = set; then withval=$with_ldap; case $with_ldap in no) with_ldap="";;
Created attachment 187570 [details, diff] ./configure patch Seems to do the right thing.
The current version seems to work only because test ${with_ldap-'no'} != "no" happens to fail when with_ldap="" (since it expands to "test" "!=" "no"). It's also the only relevant place that with_ldap is used, so the only functional difference is the error message. Downgrading to minor because it's not yet a problem, but a bit fragile.
Please don't send patches to configure but rather to configure.ac. Also, this should be directly reported upstream.
