[Asterisk-code-review] tests/test utils: Repair ./configure --with-ssl=PATH. (asterisk[13])

[Alexander Traud]

Alexander Traud has posted comments on this change. ( https://gerrit.asterisk.org/9182 )

Change subject: tests/test_utils: Repair ./configure --with-ssl=PATH.
......................................................................


Patch Set 1:

Yes, that is required.

The problem is
#include "asterisk/crypto.h" /* for ast_crypto_loaded */
which does
#include "openssl/aes.h"

That requires CRYPTO_INCLUDE in -I when compiling. Otherwise the build system picks the wrong search path for the OpenSSL headers (normally just the system header). You can see that, when you enable the Noisy Developer Mode. The exact steps to reproduce this are listed in the issue report. Is is the same as ASTERISK-27908 and Gerrit 9145.

However, you just pointed me to something: It should not be ‘openssl’ but ‘crypto’.


-- 
To view, visit https://gerrit.asterisk.org/9182
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibcab8f556ee77776f203cff8b06d776a673b7bc4
Gerrit-Change-Number: 9182
Gerrit-PatchSet: 1
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Comment-Date: Thu, 14 Jun 2018 06:37:27 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180614/5d473c81/attachment.html>