Re: [PATCH 1/7] build: change @foo@ to $(foo) in automake makefiles

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jan,
On Wed, Nov 21, 2018 at 2:58 PM Jan Engelhardt <jengelh@xxxxxxx> wrote:
>
> When using automake, all AC_SUBSTed variables are made available as
> make variables, so that they can also be set at make time. Therefore,
> they need not use the @harcoded@ style.
> ---
>  Makefile.am         | 70 ++++++++++++++++++++++-----------------------
>  Makefile.mesh       |  4 +--
>  Makefile.obexd      | 10 +++----
>  Makefile.plugins    |  6 ++--
>  Makefile.tools      | 62 +++++++++++++++++++--------------------
>  android/Makefile.am | 18 ++++++------
>  6 files changed, 85 insertions(+), 85 deletions(-)
>
> diff --git a/Makefile.am b/Makefile.am
> index d4f4572d5..1905a74b6 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -19,11 +19,11 @@ CLEANFILES =
>
>  EXTRA_DIST =
>
> -pkglibexecdir = @libexecdir@/bluetooth
> +pkglibexecdir = $(libexecdir)/bluetooth
>
>  libexec_PROGRAMS =
>
> -includedir = @includedir@/bluetooth
> +includedir = $(includedir)/bluetooth
>
>  include_HEADERS =
>
> @@ -31,7 +31,7 @@ AM_CFLAGS = $(WARNING_CFLAGS) $(MISC_CFLAGS)
>  AM_LDFLAGS = $(MISC_LDFLAGS)
>
>  if DATAFILES
> -dbusdir = @DBUS_CONFDIR@/dbus-1/system.d
> +dbusdir = $(DBUS_CONFDIR)/dbus-1/system.d
>  dbus_DATA = src/bluetooth.conf
>
>  confdir = $(sysconfdir)/bluetooth
> @@ -42,10 +42,10 @@ state_DATA =
>  endif
>
>  if SYSTEMD
> -systemdsystemunitdir = @SYSTEMD_SYSTEMUNITDIR@
> +systemdsystemunitdir = $(SYSTEMD_SYSTEMUNITDIR)
>  systemdsystemunit_DATA = src/bluetooth.service
>
> -dbussystembusdir = @DBUS_SYSTEMBUSDIR@
> +dbussystembusdir = $(DBUS_SYSTEMBUSDIR)
>  dbussystembus_DATA = src/org.bluez.service
>  endif
>
> @@ -204,7 +204,7 @@ src_bluetoothd_SOURCES = $(builtin_sources) \
>  src_bluetoothd_LDADD = lib/libbluetooth-internal.la \
>                         gdbus/libgdbus-internal.la \
>                         src/libshared-glib.la \
> -                       @BACKTRACE_LIBS@ @GLIB_LIBS@ @DBUS_LIBS@ -ldl -lrt \
> +                       $(BACKTRACE_LIBS) $(GLIB_LIBS) $(DBUS_LIBS) -ldl -lrt \
>                         $(builtin_ldadd)
>  src_bluetoothd_LDFLAGS = $(AM_LDFLAGS) -Wl,--export-dynamic \
>                                 -Wl,--version-script=$(srcdir)/src/bluetooth.ver
> @@ -239,7 +239,7 @@ include android/Makefile.am
>  include Makefile.mesh
>
>  if HID2HCI
> -rulesdir = @UDEV_DIR@/rules.d
> +rulesdir = $(UDEV_DIR)/rules.d
>
>  rules_DATA = tools/97-hid2hci.rules
>
> @@ -278,7 +278,7 @@ EXTRA_DIST += doc/btsnoop.txt
>
>  EXTRA_DIST += tools/magic.btsnoop
>
> -AM_CFLAGS += @DBUS_CFLAGS@ @GLIB_CFLAGS@
> +AM_CFLAGS += $(DBUS_CFLAGS) $(GLIB_CFLAGS)
>
>  AM_CPPFLAGS = -I$(builddir)/lib
>
> @@ -287,51 +287,51 @@ unit_tests += unit/test-eir
>
>  unit_test_eir_SOURCES = unit/test-eir.c src/eir.c src/uuid-helper.c
>  unit_test_eir_LDADD = src/libshared-glib.la lib/libbluetooth-internal.la \
> -                                                               @GLIB_LIBS@
> +                                                               $(GLIB_LIBS)
>
>  unit_tests += unit/test-uuid
>
>  unit_test_uuid_SOURCES = unit/test-uuid.c
>  unit_test_uuid_LDADD = src/libshared-glib.la lib/libbluetooth-internal.la \
> -                                                               @GLIB_LIBS@
> +                                                               $(GLIB_LIBS)
>
>  unit_tests += unit/test-textfile
>
>  unit_test_textfile_SOURCES = unit/test-textfile.c src/textfile.h src/textfile.c
> -unit_test_textfile_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +unit_test_textfile_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-crc
>
>  unit_test_crc_SOURCES = unit/test-crc.c monitor/crc.h monitor/crc.c
> -unit_test_crc_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +unit_test_crc_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-crypto
>
>  unit_test_crypto_SOURCES = unit/test-crypto.c
> -unit_test_crypto_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +unit_test_crypto_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-ecc
>
>  unit_test_ecc_SOURCES = unit/test-ecc.c
> -unit_test_ecc_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +unit_test_ecc_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-ringbuf unit/test-queue
>
>  unit_test_ringbuf_SOURCES = unit/test-ringbuf.c
> -unit_test_ringbuf_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +unit_test_ringbuf_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_test_queue_SOURCES = unit/test-queue.c
> -unit_test_queue_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +unit_test_queue_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-mgmt
>
>  unit_test_mgmt_SOURCES = unit/test-mgmt.c
> -unit_test_mgmt_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +unit_test_mgmt_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-uhid
>
>  unit_test_uhid_SOURCES = unit/test-uhid.c
> -unit_test_uhid_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +unit_test_uhid_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-sdp
>
> @@ -340,21 +340,21 @@ unit_test_sdp_SOURCES = unit/test-sdp.c \
>                                 src/log.h src/log.c \
>                                 src/sdpd-service.c src/sdpd-request.c
>  unit_test_sdp_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-avdtp
>
>  unit_test_avdtp_SOURCES = unit/test-avdtp.c \
>                                 src/log.h src/log.c \
>                                 android/avdtp.c android/avdtp.h
> -unit_test_avdtp_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +unit_test_avdtp_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-avctp
>
>  unit_test_avctp_SOURCES = unit/test-avctp.c \
>                                 src/log.h src/log.c \
>                                 android/avctp.c android/avctp.h
> -unit_test_avctp_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +unit_test_avctp_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-avrcp
>
> @@ -363,53 +363,53 @@ unit_test_avrcp_SOURCES = unit/test-avrcp.c \
>                                 android/avctp.c android/avctp.h \
>                                 android/avrcp-lib.c android/avrcp-lib.h
>  unit_test_avrcp_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-hfp
>
>  unit_test_hfp_SOURCES = unit/test-hfp.c
> -unit_test_hfp_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +unit_test_hfp_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-gdbus-client
>
>  unit_test_gdbus_client_SOURCES = unit/test-gdbus-client.c
>  unit_test_gdbus_client_LDADD = gdbus/libgdbus-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@ @DBUS_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS) $(DBUS_LIBS)
>
>  unit_tests += unit/test-gobex-header unit/test-gobex-packet unit/test-gobex \
>                         unit/test-gobex-transfer unit/test-gobex-apparam
>
>  unit_test_gobex_SOURCES = $(gobex_sources) unit/util.c unit/util.h \
>                                                 unit/test-gobex.c
> -unit_test_gobex_LDADD = @GLIB_LIBS@
> +unit_test_gobex_LDADD = $(GLIB_LIBS)
>
>  unit_test_gobex_packet_SOURCES = $(gobex_sources) unit/util.c unit/util.h \
>                                                 unit/test-gobex-packet.c
> -unit_test_gobex_packet_LDADD = @GLIB_LIBS@
> +unit_test_gobex_packet_LDADD = $(GLIB_LIBS)
>
>  unit_test_gobex_header_SOURCES = $(gobex_sources) unit/util.c unit/util.h \
>                                                 unit/test-gobex-header.c
> -unit_test_gobex_header_LDADD = @GLIB_LIBS@
> +unit_test_gobex_header_LDADD = $(GLIB_LIBS)
>
>  unit_test_gobex_transfer_SOURCES = $(gobex_sources) unit/util.c unit/util.h \
>                                                 unit/test-gobex-transfer.c
> -unit_test_gobex_transfer_LDADD = @GLIB_LIBS@
> +unit_test_gobex_transfer_LDADD = $(GLIB_LIBS)
>
>  unit_test_gobex_apparam_SOURCES = $(gobex_sources) unit/util.c unit/util.h \
>                                                 unit/test-gobex-apparam.c
> -unit_test_gobex_apparam_LDADD = @GLIB_LIBS@
> +unit_test_gobex_apparam_LDADD = $(GLIB_LIBS)
>
>  unit_tests += unit/test-lib
>
>  unit_test_lib_SOURCES = unit/test-lib.c
>  unit_test_lib_LDADD = src/libshared-glib.la \
> -                               lib/libbluetooth-internal.la @GLIB_LIBS@
> +                               lib/libbluetooth-internal.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-gatt
>
>  unit_test_gatt_SOURCES = unit/test-gatt.c
>  unit_test_gatt_LDADD = src/libshared-glib.la \
> -                               lib/libbluetooth-internal.la @GLIB_LIBS@
> +                               lib/libbluetooth-internal.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-hog
>
> @@ -424,7 +424,7 @@ unit_test_hog_SOURCES = unit/test-hog.c \
>                         attrib/gatt.h attrib/gatt.c \
>                         attrib/gattrib.h attrib/gattrib.c
>  unit_test_hog_LDADD = src/libshared-glib.la \
> -                               lib/libbluetooth-internal.la @GLIB_LIBS@
> +                               lib/libbluetooth-internal.la $(GLIB_LIBS)
>
>  unit_tests += unit/test-gattrib
>
> @@ -432,16 +432,16 @@ unit_test_gattrib_SOURCES = unit/test-gattrib.c attrib/gattrib.c \
>                                         $(btio_sources) src/log.h src/log.c
>  unit_test_gattrib_LDADD = lib/libbluetooth-internal.la \
>                         src/libshared-glib.la \
> -                       @GLIB_LIBS@ @DBUS_LIBS@ -ldl -lrt
> +                       $(GLIB_LIBS) $(DBUS_LIBS) -ldl -lrt
>
>  if MIDI
>  unit_tests += unit/test-midi
> -unit_test_midi_CFLAGS = $(AM_CFLAGS) @ALSA_CFLAGS@ -DMIDI_TEST
> +unit_test_midi_CFLAGS = $(AM_CFLAGS) $(ALSA_CFLAGS) -DMIDI_TEST
>  unit_test_midi_SOURCES = unit/test-midi.c \
>                         profiles/midi/libmidi.h \
>                         profiles/midi/libmidi.c
>  unit_test_midi_LDADD = src/libshared-glib.la \
> -                       @GLIB_LIBS@ @ALSA_LIBS@
> +                       $(GLIB_LIBS) $(ALSA_LIBS)
>  endif
>
>  if MAINTAINER_MODE
> diff --git a/Makefile.mesh b/Makefile.mesh
> index 0df7db2be..50bcc19ad 100644
> --- a/Makefile.mesh
> +++ b/Makefile.mesh
> @@ -25,7 +25,7 @@ mesh_meshd_SOURCES = $(mesh_sources) mesh/main.c
>
>  mesh_meshd_LDADD = src/shared/ecc.lo src/shared/queue.lo src/shared/io-ell.lo \
>                                 src/shared/util.lo src/shared/hci.lo src/shared/mgmt.lo \
> -                               @DBUS_LIBS@ @ELL_LIBS@ -ljson-c
> +                               $(DBUS_LIBS) $(ELL_LIBS) -ljson-c
>
>  noinst_PROGRAMS += mesh/btmesh
>
> @@ -37,6 +37,6 @@ mesh_btmesh_SOURCES = $(mesh_sources) \
>  mesh_btmesh_LDADD = src/shared/ecc.lo src/shared/queue.lo src/shared/io-ell.lo \
>                                 src/shared/util.lo src/shared/hci.lo src/shared/mgmt.lo \
>                                 src/libshared-mainloop.la \
> -                               -lreadline @ELL_LIBS@ -ljson-c
> +                               -lreadline $(ELL_LIBS) -ljson-c
>
>  endif
> diff --git a/Makefile.obexd b/Makefile.obexd
> index cd3ace425..9180d7eea 100644
> --- a/Makefile.obexd
> +++ b/Makefile.obexd
> @@ -1,8 +1,8 @@
>  if SYSTEMD
> -systemduserunitdir = @SYSTEMD_USERUNITDIR@
> +systemduserunitdir = $(SYSTEMD_USERUNITDIR)
>  systemduserunit_DATA = obexd/src/obex.service
>
> -dbussessionbusdir = @DBUS_SESSIONBUSDIR@
> +dbussessionbusdir = $(DBUS_SESSIONBUSDIR)
>  dbussessionbus_DATA = obexd/src/org.bluez.obex.service
>  endif
>
> @@ -79,12 +79,12 @@ obexd_src_obexd_SOURCES = $(btio_sources) $(gobex_sources) \
>                         obexd/src/map_ap.h
>  obexd_src_obexd_LDADD = lib/libbluetooth-internal.la \
>                         gdbus/libgdbus-internal.la \
> -                       @ICAL_LIBS@ @DBUS_LIBS@ @GLIB_LIBS@ -ldl
> +                       $(ICAL_LIBS) $(DBUS_LIBS) $(GLIB_LIBS) -ldl
>
>  obexd_src_obexd_LDFLAGS = -Wl,--export-dynamic
>
> -obexd_src_obexd_CFLAGS = $(AM_CFLAGS) @GLIB_CFLAGS@ @DBUS_CFLAGS@ \
> -                               @ICAL_CFLAGS@ -DOBEX_PLUGIN_BUILTIN \
> +obexd_src_obexd_CFLAGS = $(AM_CFLAGS) $(GLIB_CFLAGS) $(DBUS_CFLAGS) \
> +                               $(ICAL_CFLAGS) -DOBEX_PLUGIN_BUILTIN \
>                                 -DPLUGINDIR=\""$(obex_plugindir)"\" \
>                                 -fPIC -D_FILE_OFFSET_BITS=64
>
> diff --git a/Makefile.plugins b/Makefile.plugins
> index 3ab7c1c18..1c9919d71 100644
> --- a/Makefile.plugins
> +++ b/Makefile.plugins
> @@ -97,7 +97,7 @@ builtin_modules += midi
>  builtin_sources += profiles/midi/midi.c \
>                         profiles/midi/libmidi.h \
>                         profiles/midi/libmidi.c
> -builtin_ldadd += @ALSA_LIBS@
> +builtin_ldadd += $(ALSA_LIBS)
>  endif
>
>  builtin_modules += battery
> @@ -107,6 +107,6 @@ if SIXAXIS
>  plugin_LTLIBRARIES += plugins/sixaxis.la
>  plugins_sixaxis_la_SOURCES = plugins/sixaxis.c
>  plugins_sixaxis_la_LDFLAGS = $(AM_LDFLAGS) -module -avoid-version \
> -                                               -no-undefined @UDEV_LIBS@
> -plugins_sixaxis_la_CFLAGS = $(AM_CFLAGS) -fvisibility=hidden @UDEV_CFLAGS@
> +                                               -no-undefined $(UDEV_LIBS)
> +plugins_sixaxis_la_CFLAGS = $(AM_CFLAGS) -fvisibility=hidden $(UDEV_CFLAGS)
>  endif
> diff --git a/Makefile.tools b/Makefile.tools
> index 231a4b09b..cdfaaf906 100644
> --- a/Makefile.tools
> +++ b/Makefile.tools
> @@ -9,7 +9,7 @@ client_bluetoothctl_SOURCES = client/main.c \
>                                         client/advertising.c \
>                                         client/gatt.h client/gatt.c
>  client_bluetoothctl_LDADD = gdbus/libgdbus-internal.la src/libshared-glib.la \
> -                               @GLIB_LIBS@ @DBUS_LIBS@ -lreadline
> +                               $(GLIB_LIBS) $(DBUS_LIBS) -lreadline
>  endif
>
>  if MONITOR
> @@ -39,7 +39,7 @@ monitor_btmon_SOURCES = monitor/main.c monitor/bt.h \
>                                 monitor/broadcom.h monitor/broadcom.c \
>                                 monitor/tty.h
>  monitor_btmon_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-mainloop.la @UDEV_LIBS@
> +                               src/libshared-mainloop.la $(UDEV_LIBS)
>  endif
>
>  if LOGGER
> @@ -103,7 +103,7 @@ tools_mgmt_tester_SOURCES = tools/mgmt-tester.c monitor/bt.h \
>                                 emulator/bthost.h emulator/bthost.c \
>                                 emulator/smp.c
>  tools_mgmt_tester_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS)
>
>  tools_l2cap_tester_SOURCES = tools/l2cap-tester.c monitor/bt.h \
>                                 emulator/hciemu.h emulator/hciemu.c \
> @@ -111,7 +111,7 @@ tools_l2cap_tester_SOURCES = tools/l2cap-tester.c monitor/bt.h \
>                                 emulator/bthost.h emulator/bthost.c \
>                                 emulator/smp.c
>  tools_l2cap_tester_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS)
>
>  tools_rfcomm_tester_SOURCES = tools/rfcomm-tester.c monitor/bt.h \
>                                 emulator/hciemu.h emulator/hciemu.c \
> @@ -119,7 +119,7 @@ tools_rfcomm_tester_SOURCES = tools/rfcomm-tester.c monitor/bt.h \
>                                 emulator/bthost.h emulator/bthost.c \
>                                 emulator/smp.c
>  tools_rfcomm_tester_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS)
>
>  tools_bnep_tester_SOURCES = tools/bnep-tester.c monitor/bt.h \
>                                 emulator/hciemu.h emulator/hciemu.c \
> @@ -127,7 +127,7 @@ tools_bnep_tester_SOURCES = tools/bnep-tester.c monitor/bt.h \
>                                 emulator/bthost.h emulator/bthost.c \
>                                 emulator/smp.c
>  tools_bnep_tester_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS)
>
>  tools_smp_tester_SOURCES = tools/smp-tester.c monitor/bt.h \
>                                 emulator/hciemu.h emulator/hciemu.c \
> @@ -135,7 +135,7 @@ tools_smp_tester_SOURCES = tools/smp-tester.c monitor/bt.h \
>                                 emulator/bthost.h emulator/bthost.c \
>                                 emulator/smp.c
>  tools_smp_tester_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS)
>
>  tools_gap_tester_SOURCES = tools/gap-tester.c monitor/bt.h \
>                                 emulator/hciemu.h emulator/hciemu.c \
> @@ -145,7 +145,7 @@ tools_gap_tester_SOURCES = tools/gap-tester.c monitor/bt.h \
>  tools_gap_tester_LDADD =  lib/libbluetooth-internal.la \
>                                 gdbus/libgdbus-internal.la \
>                                 src/libshared-glib.la \
> -                               @GLIB_LIBS@ @DBUS_LIBS@
> +                               $(GLIB_LIBS) $(DBUS_LIBS)
>
>  tools_sco_tester_SOURCES = tools/sco-tester.c monitor/bt.h \
>                                 emulator/hciemu.h emulator/hciemu.c \
> @@ -153,10 +153,10 @@ tools_sco_tester_SOURCES = tools/sco-tester.c monitor/bt.h \
>                                 emulator/bthost.h emulator/bthost.c \
>                                 emulator/smp.c
>  tools_sco_tester_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS)
>
>  tools_hci_tester_SOURCES = tools/hci-tester.c monitor/bt.h
> -tools_hci_tester_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +tools_hci_tester_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  tools_userchan_tester_SOURCES = tools/userchan-tester.c monitor/bt.h \
>                                 emulator/hciemu.h emulator/hciemu.c \
> @@ -164,7 +164,7 @@ tools_userchan_tester_SOURCES = tools/userchan-tester.c monitor/bt.h \
>                                 emulator/bthost.h emulator/bthost.c \
>                                 emulator/smp.c
>  tools_userchan_tester_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS)
>  endif
>
>  if TOOLS
> @@ -187,7 +187,7 @@ noinst_PROGRAMS += tools/bdaddr tools/avinfo tools/avtest \
>                         tools/gatt-service profiles/iap/iapd
>
>  tools_bdaddr_SOURCES = tools/bdaddr.c src/oui.h src/oui.c
> -tools_bdaddr_LDADD = lib/libbluetooth-internal.la @UDEV_LIBS@
> +tools_bdaddr_LDADD = lib/libbluetooth-internal.la $(UDEV_LIBS)
>
>  tools_avinfo_LDADD = lib/libbluetooth-internal.la
>
> @@ -217,20 +217,20 @@ tools_btproxy_SOURCES = tools/btproxy.c monitor/bt.h
>  tools_btproxy_LDADD = src/libshared-mainloop.la
>
>  tools_btiotest_SOURCES = tools/btiotest.c btio/btio.h btio/btio.c
> -tools_btiotest_LDADD = lib/libbluetooth-internal.la @GLIB_LIBS@
> +tools_btiotest_LDADD = lib/libbluetooth-internal.la $(GLIB_LIBS)
>
>  tools_mcaptest_SOURCES = tools/mcaptest.c \
>                                 btio/btio.h btio/btio.c \
>                                 src/log.c src/log.h \
>                                 profiles/health/mcap.h profiles/health/mcap.c
> -tools_mcaptest_LDADD = lib/libbluetooth-internal.la @GLIB_LIBS@ \
> +tools_mcaptest_LDADD = lib/libbluetooth-internal.la $(GLIB_LIBS) \
>                                 src/libshared-mainloop.la -lrt
>
>  tools_bneptest_SOURCES = tools/bneptest.c \
>                                 btio/btio.h btio/btio.c \
>                                 src/log.h src/log.c \
>                                 profiles/network/bnep.h profiles/network/bnep.c
> -tools_bneptest_LDADD = lib/libbluetooth-internal.la @GLIB_LIBS@ \
> +tools_bneptest_LDADD = lib/libbluetooth-internal.la $(GLIB_LIBS) \
>                                 src/libshared-mainloop.la
>
>  tools_cltest_SOURCES = tools/cltest.c
> @@ -282,13 +282,13 @@ tools_bluemoon_LDADD = src/libshared-mainloop.la
>  tools_hex2hcd_SOURCES = tools/hex2hcd.c
>
>  tools_mpris_proxy_SOURCES = tools/mpris-proxy.c
> -tools_mpris_proxy_LDADD = gdbus/libgdbus-internal.la @GLIB_LIBS@ @DBUS_LIBS@
> +tools_mpris_proxy_LDADD = gdbus/libgdbus-internal.la $(GLIB_LIBS) $(DBUS_LIBS)
>
>  tools_gatt_service_SOURCES = tools/gatt-service.c
> -tools_gatt_service_LDADD = @GLIB_LIBS@ @DBUS_LIBS@ gdbus/libgdbus-internal.la
> +tools_gatt_service_LDADD = $(GLIB_LIBS) $(DBUS_LIBS) gdbus/libgdbus-internal.la
>
>  profiles_iap_iapd_SOURCES = profiles/iap/main.c
> -profiles_iap_iapd_LDADD = gdbus/libgdbus-internal.la @GLIB_LIBS@ @DBUS_LIBS@
> +profiles_iap_iapd_LDADD = gdbus/libgdbus-internal.la $(GLIB_LIBS) $(DBUS_LIBS)
>
>  dist_man_MANS += tools/rctest.1 tools/l2ping.1 tools/bccmd.1 tools/btattach.1
>
> @@ -313,7 +313,7 @@ tools_meshctl_SOURCES = tools/meshctl.c \
>                                 tools/mesh/onoff-model.h tools/mesh/onoff-model.c
>  tools_meshctl_LDADD = gdbus/libgdbus-internal.la src/libshared-glib.la \
>                                 lib/libbluetooth-internal.la \
> -                               @GLIB_LIBS@ @DBUS_LIBS@ -ljson-c -lreadline
> +                               $(GLIB_LIBS) $(DBUS_LIBS) -ljson-c -lreadline
>  endif
>
>  EXTRA_DIST += tools/mesh/local_node.json tools/mesh/prov_db.json
> @@ -336,7 +336,7 @@ tools_hciconfig_SOURCES = tools/hciconfig.c tools/csr.h tools/csr.c
>  tools_hciconfig_LDADD = lib/libbluetooth-internal.la
>
>  tools_hcitool_SOURCES = tools/hcitool.c src/oui.h src/oui.c
> -tools_hcitool_LDADD = lib/libbluetooth-internal.la @UDEV_LIBS@
> +tools_hcitool_LDADD = lib/libbluetooth-internal.la $(UDEV_LIBS)
>
>  tools_hcidump_SOURCES = tools/hcidump.c \
>                                 tools/parser/parser.h tools/parser/parser.c \
> @@ -365,7 +365,7 @@ tools_hcidump_SOURCES = tools/hcidump.c \
>                                 tools/parser/bpa.c
>
>  tools_sdptool_SOURCES = tools/sdptool.c src/sdp-xml.h src/sdp-xml.c
> -tools_sdptool_LDADD = lib/libbluetooth-internal.la @GLIB_LIBS@
> +tools_sdptool_LDADD = lib/libbluetooth-internal.la $(GLIB_LIBS)
>
>  tools_ciptool_LDADD = lib/libbluetooth-internal.la
>  tools_hcidump_LDADD = lib/libbluetooth-internal.la
> @@ -385,11 +385,11 @@ EXTRA_DIST += tools/rctest.1 tools/l2ping.1 tools/bccmd.1 tools/btattach.1
>  endif
>
>  if HID2HCI
> -udevdir = @UDEV_DIR@
> +udevdir = $(UDEV_DIR)
>
>  udev_PROGRAMS = tools/hid2hci
>
> -tools_hid2hci_LDADD = @UDEV_LIBS@
> +tools_hid2hci_LDADD = $(UDEV_LIBS)
>
>  dist_man_MANS += tools/hid2hci.1
>  else
> @@ -403,20 +403,20 @@ noinst_PROGRAMS += tools/btmgmt tools/obex-client-tool tools/obex-server-tool \
>  tools_obex_client_tool_SOURCES = $(gobex_sources) $(btio_sources) \
>                                                 tools/obex-client-tool.c
>  tools_obex_client_tool_LDADD = lib/libbluetooth-internal.la \
> -                                               @GLIB_LIBS@ -lreadline
> +                                               $(GLIB_LIBS) -lreadline
>
>  tools_obex_server_tool_SOURCES = $(gobex_sources) $(btio_sources) \
>                                                 tools/obex-server-tool.c
> -tools_obex_server_tool_LDADD = lib/libbluetooth-internal.la @GLIB_LIBS@
> +tools_obex_server_tool_LDADD = lib/libbluetooth-internal.la $(GLIB_LIBS)
>
>  tools_bluetooth_player_SOURCES = tools/bluetooth-player.c
>  tools_bluetooth_player_LDADD = gdbus/libgdbus-internal.la \
>                                 src/libshared-glib.la \
> -                               @GLIB_LIBS@ @DBUS_LIBS@ -lreadline
> +                               $(GLIB_LIBS) $(DBUS_LIBS) -lreadline
>
>  tools_obexctl_SOURCES = tools/obexctl.c
>  tools_obexctl_LDADD = gdbus/libgdbus-internal.la src/libshared-glib.la \
> -                               @GLIB_LIBS@ @DBUS_LIBS@ -lreadline
> +                               $(GLIB_LIBS) $(DBUS_LIBS) -lreadline
>
>  tools_btmgmt_SOURCES = tools/btmgmt.c src/uuid-helper.c client/display.c
>  tools_btmgmt_LDADD = lib/libbluetooth-internal.la src/libshared-mainloop.la \
> @@ -430,7 +430,7 @@ attrib_gatttool_SOURCES = attrib/gatttool.c attrib/att.c attrib/gatt.c \
>                                 attrib/utils.c src/log.c client/display.c \
>                                 client/display.h
>  attrib_gatttool_LDADD = lib/libbluetooth-internal.la \
> -                       src/libshared-glib.la @GLIB_LIBS@ -lreadline
> +                       src/libshared-glib.la $(GLIB_LIBS) -lreadline
>
>  endif
>  endif
> @@ -446,7 +446,7 @@ profiles_cups_bluetooth_SOURCES = profiles/cups/main.c \
>                                         profiles/cups/spp.c \
>                                         profiles/cups/hcrp.c
>
> -profiles_cups_bluetooth_LDADD = @GLIB_LIBS@ @DBUS_LIBS@ \
> +profiles_cups_bluetooth_LDADD = $(GLIB_LIBS) $(DBUS_LIBS) \
>                                 lib/libbluetooth-internal.la \
>                                 gdbus/libgdbus-internal.la
>  endif
> @@ -469,6 +469,6 @@ if BTPCLIENT
>  noinst_PROGRAMS += tools/btpclient
>
>  tools_btpclient_SOURCES = tools/btpclient.c src/shared/btp.c src/shared/btp.h
> -tools_btpclient_CFLAGS = $(AM_CFLAGS) @ELL_CFLAGS@
> -tools_btpclient_LDADD = @ELL_LIBS@ lib/libbluetooth-internal.la
> +tools_btpclient_CFLAGS = $(AM_CFLAGS) $(ELL_CFLAGS)
> +tools_btpclient_LDADD = $(ELL_LIBS) lib/libbluetooth-internal.la
>  endif
> diff --git a/android/Makefile.am b/android/Makefile.am
> index d665d1e14..d09a52d69 100644
> --- a/android/Makefile.am
> +++ b/android/Makefile.am
> @@ -12,7 +12,7 @@ android_system_emulator_LDADD = src/libshared-mainloop.la
>  noinst_PROGRAMS += android/bluetoothd-snoop
>
>  android_bluetoothd_snoop_SOURCES = android/bluetoothd-snoop.c src/log.c
> -android_bluetoothd_snoop_LDADD = src/libshared-mainloop.la @GLIB_LIBS@
> +android_bluetoothd_snoop_LDADD = src/libshared-mainloop.la $(GLIB_LIBS)
>
>  noinst_PROGRAMS += android/bluetoothd
>
> @@ -59,7 +59,7 @@ android_bluetoothd_SOURCES = android/main.c \
>                                 src/sdp-client.h src/sdp-client.c \
>                                 profiles/network/bnep.h profiles/network/bnep.c
>  android_bluetoothd_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS)
>
>  plugin_LTLIBRARIES += android/bluetooth.default.la
>
> @@ -110,7 +110,7 @@ android_avdtptest_SOURCES = android/avdtptest.c \
>                                 src/shared/log.h src/shared/log.c \
>                                 android/avdtp.h android/avdtp.c
>  android_avdtptest_CFLAGS = $(AM_CFLAGS)
> -android_avdtptest_LDADD = lib/libbluetooth-internal.la @GLIB_LIBS@
> +android_avdtptest_LDADD = lib/libbluetooth-internal.la $(GLIB_LIBS)
>
>  noinst_PROGRAMS += android/haltest
>
> @@ -165,7 +165,7 @@ android_android_tester_SOURCES = emulator/hciemu.h emulator/hciemu.c \
>  android_android_tester_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/android \
>                                 -DPLUGINDIR=\""$(android_plugindir)"\"
>  android_android_tester_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS)
>  android_android_tester_LDFLAGS = -pthread -ldl
>
>  noinst_PROGRAMS += android/ipc-tester
> @@ -178,7 +178,7 @@ android_ipc_tester_SOURCES = emulator/hciemu.h emulator/hciemu.c \
>                                 android/ipc-common.h android/ipc-tester.c
>  android_ipc_tester_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/android
>  android_ipc_tester_LDADD = lib/libbluetooth-internal.la \
> -                               src/libshared-glib.la @GLIB_LIBS@
> +                               src/libshared-glib.la $(GLIB_LIBS)
>
>  plugin_LTLIBRARIES += android/audio.a2dp.default.la
>
> @@ -193,8 +193,8 @@ android_audio_a2dp_default_la_SOURCES = android/audio-msg.h \
>                                         android/hardware/hardware.h \
>                                         android/system/audio.h
>  android_audio_a2dp_default_la_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/android \
> -                                       @SBC_CFLAGS@
> -android_audio_a2dp_default_la_LIBADD = @SBC_LIBS@
> +                                       $(SBC_CFLAGS)
> +android_audio_a2dp_default_la_LIBADD = $(SBC_LIBS)
>  android_audio_a2dp_default_la_LDFLAGS = $(AM_LDFLAGS) -module -avoid-version \
>                                         -no-undefined -pthread -lrt
>
> @@ -210,7 +210,7 @@ android_audio_sco_default_la_SOURCES = android/hal-log.h \
>                                         android/audio_utils/resampler.h \
>                                         android/system/audio.h
>  android_audio_sco_default_la_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/android
> -android_audio_sco_default_la_LIBADD = @SPEEXDSP_LIBS@
> +android_audio_sco_default_la_LIBADD = $(SPEEXDSP_LIBS)
>  android_audio_sco_default_la_LDFLAGS = $(AM_LDFLAGS) -module -avoid-version \
>                                         -no-undefined -lrt
>  unit_tests += android/test-ipc
> @@ -219,7 +219,7 @@ android_test_ipc_SOURCES = android/test-ipc.c \
>                                 src/log.h src/log.c \
>                                 android/ipc-common.h \
>                                 android/ipc.c android/ipc.h
> -android_test_ipc_LDADD = src/libshared-glib.la @GLIB_LIBS@
> +android_test_ipc_LDADD = src/libshared-glib.la $(GLIB_LIBS)
>
>  endif
>
> --
> 2.19.1

Makefile:2392: *** Recursive variable 'includedir' references itself
(eventually).  Stop.
make[3]: *** Waiting for unfinished jobs....
make[2]: *** [Makefile:9544: install-am] Error 2
make[1]: *** [Makefile:9538: install] Error 2
make: *** [Makefile:9449: distcheck] Error 1
warning: execution failed: make distcheck
You can fix the problem, and then run

Ive also got other problems when doing make distcheck with all patches
applied so Id recommend you do:

git rebase -i origin/master --exec "make distcheck"



-- 
Luiz Augusto von Dentz



[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux