Skip to content

Commit

Permalink
tests-windows: Add OpenSSL directory to autotest path
Browse files Browse the repository at this point in the history
Running OpenSSL unit tests without it already being included in library path
revealed a problem: the AUTOTEST_PATH does not include it.

This patch adds a new variable `SSL_DIR` which will be added in AUTOTEST_PATH.

Signed-off-by: Alin Gabriel Serdean <[email protected]>
Acked-by: Ben Pfaff <[email protected]>
  • Loading branch information
aserdean committed Mar 8, 2018
1 parent 593e93e commit d130958
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions m4/ax_check_openssl.m4
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
# SSL_INCLUDES to the include directives required
# SSL_LIBS to the -l directives required
# SSL_LDFLAGS to the -L or -R flags required
# SSL_DIR to add it to various paths
#
# and calls ACTION-IF-FOUND or ACTION-IF-NOT-FOUND appropriately
#
Expand Down Expand Up @@ -81,8 +82,10 @@ AC_DEFUN([AX_CHECK_OPENSSL], [
SSL_LDFLAGS="-L$ssldir/lib"
if test "$WIN32" = "yes"; then
SSL_LIBS="-lssleay32 -llibeay32"
SSL_DIR=/$(echo ${ssldir} | ${SED} -e 's/://')
else
SSL_LIBS="-lssl -lcrypto"
SSL_DIR="$ssldir"
fi
found=true
AC_MSG_RESULT([yes])
Expand Down Expand Up @@ -126,4 +129,5 @@ AC_DEFUN([AX_CHECK_OPENSSL], [
AC_SUBST([SSL_INCLUDES])
AC_SUBST([SSL_LIBS])
AC_SUBST([SSL_LDFLAGS])
AC_SUBST([SSL_DIR])
])
2 changes: 1 addition & 1 deletion tests/automake.mk
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ SYSTEM_USERSPACE_TESTSUITE = $(srcdir)/tests/system-userspace-testsuite
SYSTEM_OFFLOADS_TESTSUITE = $(srcdir)/tests/system-offloads-testsuite
DISTCLEANFILES += tests/atconfig tests/atlocal

AUTOTEST_PATH = utilities:vswitchd:ovsdb:vtep:tests:$(PTHREAD_WIN32_DIR_DLL):ovn/controller-vtep:ovn/northd:ovn/utilities:ovn/controller
AUTOTEST_PATH = utilities:vswitchd:ovsdb:vtep:tests:$(PTHREAD_WIN32_DIR_DLL):$(SSL_DIR):ovn/controller-vtep:ovn/northd:ovn/utilities:ovn/controller

check-local:
set $(SHELL) '$(TESTSUITE)' -C tests AUTOTEST_PATH=$(AUTOTEST_PATH) $(TESTSUITEFLAGS); \
Expand Down

0 comments on commit d130958

Please sign in to comment.