Skip to content

Commit

Permalink
Fix CI-- LDFLAGS
Browse files Browse the repository at this point in the history
  • Loading branch information
Michael Herstine committed Sep 17, 2023
1 parent 020d766 commit 5a7c1a3
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 22 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -304,9 +304,9 @@ jobs:
eval "$(/usr/libexec/path_helper)"
PATH="/usr/local/opt/bison/bin:/usr/local/opt/texinfo/bin:$PATH" ./configure --with-boost=$(cd boost_${{ matrix.boost-ver }}/installdir; pwd) --with-openssl=/usr/local/opt/openssl CPPFLAGS="-D_LIBCPP_ENABLE_CXX17_REMOVED_AUTO_PTR -I/usr/local/opt/[email protected]/include" LDFLAGS="-L/usr/local/opt/[email protected]/lib"
- name: Setup tmate session
uses: mxschmitt/action-tmate@v3
if: matrix.os == 'ubuntu-22.04' && matrix.boost == 'oldest'
# - name: Setup tmate session
# uses: mxschmitt/action-tmate@v3
# if: matrix.os == 'ubuntu-22.04' && matrix.boost == 'oldest'

- name: Make scribbu (Ubuntu)
if: matrix.os == 'ubuntu-22.04'
Expand Down
31 changes: 14 additions & 17 deletions scribbu/Makefile.am
Original file line number Diff line number Diff line change
@@ -1,22 +1,6 @@
# Tell automake that these two must be built first; we say how below
BUILT_SOURCES = tbt-parser.hh tbt-lexer.hh scheme.x

AM_YFLAGS = -d
AM_CPPFLAGS = -I$(srcdir)/.. $(BOOST_CPPFLAGS) $(OPENSSL_INCLUDES) $(GUILE_CFLAGS)
AM_CXXFLAGS = -std=c++17 -Wno-register $(GUILE_CFLAGS)
AM_LDFLAGS = $(BOOST_LDFLAGS) $(LIBUNISTRING)

lib_LTLIBRARIES = libscribbu.la
libscribbu_la_LIBADD = $(GUILE_LIBS) \
$(OPENSSL_LIBS) \
$(BOOST_IOSTREAMS_LIB) \
$(BOOST_FILESYSTEM_LIB) \
$(BOOST_LOG_LIB) \
$(BOOST_PROGRAM_OPTIONS_LIB) \
$(BOOST_REGEX_LIB) \
$(BOOST_SYSTEM_LIB)


libscribbu_la_SOURCES = tbt-parser.yy \
tbt-lexer.ll \
scribbu.cc \
Expand Down Expand Up @@ -68,7 +52,20 @@ pkginclude_HEADERS = scribbu.hh \
scheme-serde.hh \
winamp-genres.hh

libscribbu_la_LDFLAGS = -version-info 4:1:0
AM_YFLAGS = -d
AM_CPPFLAGS = -I$(srcdir)/.. $(BOOST_CPPFLAGS) $(OPENSSL_INCLUDES) $(GUILE_CFLAGS)
AM_CXXFLAGS = -std=c++17 -Wno-register $(GUILE_CFLAGS)
AM_LDFLAGS = $(BOOST_LDFLAGS) $(LIBUNISTRING)

libscribbu_la_LIBADD = $(GUILE_LIBS) \
$(OPENSSL_LIBS) \
$(BOOST_IOSTREAMS_LIB) \
$(BOOST_FILESYSTEM_LIB) \
$(BOOST_LOG_LIB) \
$(BOOST_REGEX_LIB) \
$(BOOST_SYSTEM_LIB)

libscribbu_la_LDFLAGS = -version-info 4:1:0 $(BOOST_LDFLAGS) $(LIBUNISTRING)

# automake will figure out how to make tbt-parser.hh, but we need to tell it how
# to make tbt-lexer.h:
Expand Down
4 changes: 2 additions & 2 deletions src/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ bin_PROGRAMS = scribbu
scribbu_SOURCES = scribbu.cc command-utilities.hh command-utilities.cc \
rename.cc report.cc split.cc dump.cc popm.cc xtag.cc text.cc \
genre.cc m3u.cc encodings.cc
# AM_CPPFLAGS = -I$(srcdir)/.. $(BOOST_CPPFLAGS) $(GUILE_CFLAGS) -DDATADIR=\"$(datadir)\" -DSCRIBBU_ALWAYS_DUMP_CORE
AM_CPPFLAGS = -I$(srcdir)/.. $(BOOST_CPPFLAGS) $(GUILE_CFLAGS) -DDATADIR=\"$(datadir)\"
AM_CPPFLAGS = -I$(srcdir)/.. $(BOOST_CPPFLAGS) $(GUILE_CFLAGS) -DDATADIR=\"$(datadir)\" -DSCRIBBU_ALWAYS_DUMP_CORE
# AM_CPPFLAGS = -I$(srcdir)/.. $(BOOST_CPPFLAGS) $(GUILE_CFLAGS) -DDATADIR=\"$(datadir)\"
AM_CXXFLAGS = -std=c++17 $(GUILE_CFLAGS)
AM_LDFLAGS = $(BOOST_LDFLAGS) $(LIBUNISTRING)
scribbu_LDADD = ../scribbu/libscribbu.la \
Expand Down

0 comments on commit 5a7c1a3

Please sign in to comment.