Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Commit

Permalink
build: always link with -lz
Browse files Browse the repository at this point in the history
Fix building with a shared zlib: ensure link with '-lz'.

Configuring for a shared zlib:

    LDFLAGS="-L/opt/local/lib -R/opt/local/lib" \
        ./configure --shared-zlib \
        --shared-zlib-includes=/opt/local/include \
        --shared-zlib-libpath=/opt/local/lib

and building would fail on SmartOS because the link would not include '-lz'.
This doesn't fail on Mac (at least in my setup) because you get lucky with the
openssl libs reported from the Homebrew pkg-config:

    $ pkg-config --libs openssl
    -lssl -lcrypto -lz

On SmartOS, the pkgsrc libs for openssl are:

    $ pkg-config --libs openssl
    -Wl,-R/opt/local/lib -L/opt/local/lib -lssl -lcrypto -lsocket -lnsl -ldl

The patch has no adverse effect on the Mac build (by control case).
  • Loading branch information
trentm authored and bnoordhuis committed Jun 19, 2012
1 parent eef57dd commit dd1c3d7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion wscript
Expand Up @@ -296,7 +296,7 @@ def configure(conf):
conf.env["USE_GDBJIT"] = o.use_gdbjit
conf.env['USE_NPM'] = not o.without_npm

if not conf.env["USE_SHARED_ZLIB"] and not sys.platform.startswith("win32"):
if not sys.platform.startswith("win32"):
conf.env.append_value("LINKFLAGS", "-lz")

conf.check(lib='dl', uselib_store='DL')
Expand Down

0 comments on commit dd1c3d7

Please sign in to comment.