Skip to content

Commit

Permalink
Fix: use LDFLAGS instead of LIBS for python bindings
Browse files Browse the repository at this point in the history
Turns out that contrary to what is documented some versions of distutils
don't honor the LIBS variable, use LDFLAGS instead.

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
  • Loading branch information
mjeanson authored and jgalar committed Jun 13, 2017
1 parent 98d8d2b commit a0439ff
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions bindings/python/Makefile.am
Expand Up @@ -25,8 +25,7 @@ BINDINGS_DEPS=setup.py \
BUILD_FLAGS=CC="$(CC)" \
CFLAGS="$(GLIB_CFLAGS) $(AM_CFLAGS) $(CFLAGS)" \
CPPFLAGS="$(DEFS) $(DEFAULT_INCLUDES) $(AM_CPPFLAGS) $(CPPFLAGS)" \
LDFLAGS="$(AM_LDFLAGS) $(LDFLAGS)" \
LIBS="$(GLIB_LIBS) $(LIBS)"
LDFLAGS="$(AM_LDFLAGS) $(LDFLAGS) $(GLIB_LIBS) $(LIBS)"

build-python-bindings.stamp: $(BINDINGS_DEPS)
$(BUILD_FLAGS) $(PYTHON) $(builddir)/setup.py build_ext
Expand Down

0 comments on commit a0439ff

Please sign in to comment.