Bugs item #2914578, was opened at 2009-12-14 23:30
Message generated for change (Comment added) made by mbucc
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=482468&aid=2914578&group_id=56967

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Build
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Mark Bucciarelli (mbucc)
Assigned to: Sjoerd Mullender (sjoerd)
Summary: Test for -lcrypto with -lssl not working

Initial Comment:
On OpenBSD you need -lcrypto with -lssl.  It looks like 
MonetDB5-server-5.16.0/aclocal.m4 is trying to check
for this, but the configure output looks wrong:

../MonetDB5-server-5.16.0/configure line 34000

if test "x$ac_cv_lib_crypto_ERR_get_error" = x""yes; then
  OPENSSL_LIBS="$OPENSSL_LIBS -lcrypto"
fi

I think the right side of the test be

         = "xyes"; then

When I compile the test program by hand with the -lssl and -lcrypto args it 
compiles just fine.

----------------------------------------------------------------------

>Comment By: Mark Bucciarelli (mbucc)
Date: 2010-03-04 18:30

Message:
Hi,

I manually applied your change to Feb2010.

...
checking for ERR_get_error in -lcrypto... yes
checking for OpenSSL >= 0x0090800f... yes
checking for dlopen in -ldl... no
...

Please apply to tree!

:)

----------------------------------------------------------------------

Comment By: Sjoerd Mullender (sjoerd)
Date: 2010-01-21 11:20

Message:
It'll probably apply (I didn't try).  But you can also easily do it by hand
in case it doesn't.  All I did was add an extra argument to the call to
AC_CHECK_LIB(ssl, ...).  The extra argument is [-lcrypto] (i.e. add a comma
and the string [-lcrypto] including brackets to just before the closing
parenthesis).

----------------------------------------------------------------------

Comment By: Mark Bucciarelli (mbucc)
Date: 2010-01-21 07:39

Message:
I won't be able to try until early Feb.

Will this apply against latest stable branch?

----------------------------------------------------------------------

Comment By: Sjoerd Mullender (sjoerd)
Date: 2010-01-21 04:51

Message:
Could you try the attached diff to buildtools/conf/MonetDB.m4 to see if
that works?
You will need to rebuild and install the buildtools and then try the
MonetDB5 configure again.

----------------------------------------------------------------------

Comment By: Mark Bucciarelli (mbucc)
Date: 2009-12-15 11:50

Message:
Just to be clear; I manually edited the configure file that was generated
by autoconf.

I changed:  -lssl to -lssl -lcrypto in two places and the configure script
changed behavior.

Before the change, it said I didn't have OpenSSL installed.  After the
change, it said my
version is too old (which is the correct answer).

The original configure did run the ERR_get_error check, but I think the
logic
in that section is somehow broken.  I'm not a user of autotools, so I
couldn't
sort out what the issue was.

Here's the relevant output from a configure run from before my manual
edits:

...
checking for SSL_read in -lssl... no
checking for ERR_get_error in -lcrypto... yes
...
configure: error: MonetDB 5 requires OpenSSL (OpenSSL library not found)
$


----------------------------------------------------------------------

Comment By: Fabian (mr-meltdown)
Date: 2009-12-15 08:28

Message:
weird this is autoconf generated code, and it seems the -lssl check above
is identical, but apparently did work for you.

----------------------------------------------------------------------

Comment By: Mark Bucciarelli (mbucc)
Date: 2009-12-14 23:41

Message:
Not sure about my original analysis.  But when I changed lin 33868
of ../MonetDB5-server-5.16.0/configure from

        LIBS="-lssl $LIBS"

to

     LIBS="-lssl -lcrypto $LIBS"

Now configure can find openssl libs (it says the version I have
is too old, when it used to say openssl not found).

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=482468&aid=2914578&group_id=56967

------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Monetdb-bugs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-bugs

Reply via email to