Skip to content

Commit

Permalink
Fix LDFLAGS for tests
Browse files Browse the repository at this point in the history
Unlike Makefile.am, which does not overwrite LDFLAGS but defines binary-specific
LDFLAGS variables, tests/Makefile.am redefines LDFLAGS and overwrites an
already existing LDFLAGS variable.

To successfully pass this variable to the test binaries, it needs to be appended
to the newly defined LDFLAGS variable.
  • Loading branch information
Sven Hartge authored and Fabian Mauchle committed Sep 24, 2019
1 parent e7649a1 commit f4eee85
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,6 @@ LOG_DRIVER = env AM_TAP_AWK='$(AWK)' $(SHELL) \
check_PROGRAMS = t_fticks t_rewrite t_resizeattr t_rewrite_config
AM_CFLAGS = -g -Wall -Werror @SSL_CFLAGS@ @TARGET_CFLAGS@
LDADD = $(top_builddir)/librsp.a @SSL_LIBS@
LDFLAGS = @SSL_LDFLAGS@ @TARGET_LDFLAGS@
LDFLAGS = @SSL_LDFLAGS@ @TARGET_LDFLAGS@ @LDFLAGS@

TESTS = $(check_PROGRAMS)

0 comments on commit f4eee85

Please sign in to comment.