Bug 345 – ./configure produces warning message with --without-ldap

Bug 345 - ./configure produces warning message with --without-ldap


Summary:	./configure produces warning message with --without-ldap

Status:	RESOLVED FIXED

Product:	Sudo
Classification:	Unclassified
Component:	Configure
Version:	1.7.0
Hardware:	PC Linux

Importance:	low normal
Assigned To:	Todd C. Miller

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2009-04-10 16:34 MDT by T Chan
Modified:	2009-04-18 19:19 MDT (History)
CC List:	0 users

See Also:

Attachments
configure.in patch (647 bytes, patch) 2009-04-10 16:34 MDT, T Chan	Details \| Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description T Chan 2009-04-10 16:34:34 MDT

Created attachment 251 [details]
configure.in patch

Lack of quotes around a shell variable cause it to expand to nothing when --without-ldap is set (because with_ldap=no is changed to with_ldap="" earlier in the script):
    checking for getspnam... yes
    ./configure: line 22314: test: !=: unary operator expected
    checking for log file location... /var/log/sudo.log

Otherwise it behaves as expected, but seems a bit fragile.

Comment 1 Todd C. Miller 2009-04-10 17:00:09 MDT

Thanks, this will be fixed in 1.7.1

Comment 2 Todd C. Miller 2009-04-18 19:19:00 MDT

Fixed in sudo 1.7.1, released today