In commit f91e4863bc1382 we fixed an issue where LZ4_LIBS could be
overwritten in some situations.  But on systems where lz4 is installed on
the system but is lacking pkg-config information, the linker will not know
about the lz4 library when completing the build.

This fixes the issue by explicitly setting LZ4_LIBS to contain -llz4
if pkg-config test was run and failed verifying the installed lz4 version
number.  This also ensures that LZ4_LIBS will not be overwritten if it
has been provided on the ./configure command line.

Signed-off-by: David Sommerseth <dav...@openvpn.net>
---
 configure.ac | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/configure.ac b/configure.ac
index 394f5b09..6f59baef 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1093,7 +1093,9 @@ if test "$enable_lz4" = "yes" && test "$enable_comp_stub" 
= "no"; then
        PKG_CHECK_MODULES([LZ4],
                          [liblz4 >= 1.7.1 liblz4 < 100],
                          [have_lz4="yes"],
-                         [] # If this fails, we will do another test next
+                         [LZ4_LIBS="-llz4"] # If this fails, we will do 
another test next.
+                                            # We also add set LZ4_LIBS 
otherwise the
+                                            # linker will not know about the 
lz4 library
        )
     fi
 
-- 
2.13.5


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to