[Bug 601] configure script doesen't setup preprocessor flags properly

[bugzilla-daemon at mindrot.org]

http://bugzilla.mindrot.org/show_bug.cgi?id=601





------- Additional Comments From mouring at eviladmin.org  2003-06-25 15:20 -------
The /var/run is a bogus thing for solaris 2.5.1.. OpenSSH is doing the correct 
thing by not using it if it does not exist.  So I think we can safely ignore 
that complaint.

I don't know if the configure.ac that Darren is suggest is useful.  I thought 
configure already tested for such things.

As for LD_LIBRARY_PATH... This is really not a solaris only issue.  This is an 
issue with any OS that honors that variable (which is most UNIXes).  However, 
the use/abuse of it can cause odd-ball failures/successes. 
Since /usr/local/ssl/lib is set in LD_LIBRARY_PATH the AC_TRY_LINK_FUNC
(RAND_add, AC_DEFINE(HAVE_OPENSSL),..) is successful without having to set -I 
then we see this error.

We shouldn't distrupt LD_LIBRARY_PATH in the off chance that it may be set for 
a valid reason (dynamically linked OpenSSL w/ libraries in non-standard 
locations).

Darren, maybe this needs to be solved by added another check between the check 
to see if the library is usable and the checks to see if the headers/libraries 
match.

Something like:

AC_TRY_RUN([#include <..>
main(..) { .. )],
[dnl good, no LD_LIBRARY_PATH issues found],
[dnl crap, no header, we got an LD_LIBRARY_ISSUE.  Mirror tests in above 
AC_TRY_LINK_FUNC(RAND_add, ..)]
)

That ensures if for some reason the LD_LIBRARY_PATH is set that the -I is 
attempted.  

There may be a better way, but it is too late over in the midwest to consider 
such things.=)  Plus we are in the middle of a lighning storm. <cough>

My ONLY concern is if LD_LIBRARY_PATH is set and --with-ssl-dir is set.  If
we need another case to handle it.



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.