[pulseaudio-discuss] [PATCH] build-sys: Stop using symdef headers for modules
Arun Raghavan
arun at arunraghavan.net
Tue Dec 12 05:11:45 UTC 2017
On Sat, 9 Dec 2017, at 01:19 PM, Tanu Kaskinen wrote:
> On Thu, 2017-12-07 at 05:39 +0530, Arun Raghavan wrote:
> > This removes the symdef header generation m4 magic in favour of a
> > simpler macro method, allowing us to skip one unnecessary build step
> > while moving to meson, and removing an 11 year old todo!
> > ---
> > src/.gitignore | 1 -
> > src/Makefile.am | 271 ++++++++--------------
> > src/modules/alsa/module-alsa-card.c | 1 -
> > src/modules/alsa/module-alsa-sink.c | 1 -
> > src/modules/alsa/module-alsa-source.c | 1 -
> > src/modules/bluetooth/module-bluetooth-discover.c | 2 -
> > src/modules/bluetooth/module-bluetooth-policy.c | 2 -
> > src/modules/bluetooth/module-bluez4-device.c | 1 -
> > src/modules/bluetooth/module-bluez4-discover.c | 1 -
> > src/modules/bluetooth/module-bluez5-device.c | 2 -
> > src/modules/bluetooth/module-bluez5-discover.c | 2 -
> > src/modules/dbus/module-dbus-protocol.c | 2 -
> > src/modules/echo-cancel/module-echo-cancel.c | 2 -
> > src/modules/gconf/module-gconf.c | 2 -
> > src/modules/jack/module-jack-sink.c | 2 -
> > src/modules/jack/module-jack-source.c | 2 -
> > src/modules/jack/module-jackdbus-detect.c | 2 -
> > src/modules/macosx/module-bonjour-publish.c | 2 -
> > src/modules/macosx/module-coreaudio-detect.c | 2 -
> > src/modules/macosx/module-coreaudio-device.c | 2 -
> > src/modules/module-allow-passthrough.c | 2 -
> > src/modules/module-always-sink.c | 2 -
> > src/modules/module-always-source.c | 2 -
> > src/modules/module-augment-properties.c | 2 -
> > src/modules/module-card-restore.c | 2 -
> > src/modules/module-cli.c | 2 -
> > src/modules/module-combine-sink.c | 2 -
> > src/modules/module-combine.c | 2 -
> > src/modules/module-console-kit.c | 2 -
> > src/modules/module-default-device-restore.c | 2 -
> > src/modules/module-defs.h.m4 | 35 ---
> > src/modules/module-detect.c | 2 -
> > src/modules/module-device-manager.c | 2 -
> > src/modules/module-device-restore.c | 2 -
> > src/modules/module-equalizer-sink.c | 2 -
> > src/modules/module-esound-compat-spawnfd.c | 2 -
> > src/modules/module-esound-compat-spawnpid.c | 2 -
> > src/modules/module-esound-sink.c | 2 -
> > src/modules/module-filter-apply.c | 2 -
> > src/modules/module-filter-heuristics.c | 2 -
> > src/modules/module-hal-detect-compat.c | 2 -
> > src/modules/module-intended-roles.c | 2 -
> > src/modules/module-ladspa-sink.c | 1 -
> > src/modules/module-lirc.c | 2 -
> > src/modules/module-loopback.c | 2 -
> > src/modules/module-match.c | 2 -
> > src/modules/module-mmkbd-evdev.c | 2 -
> > src/modules/module-native-protocol-fd.c | 2 -
> > src/modules/module-null-sink.c | 2 -
> > src/modules/module-null-source.c | 2 -
> > src/modules/module-pipe-sink.c | 2 -
> > src/modules/module-pipe-source.c | 2 -
> > src/modules/module-position-event-sounds.c | 2 -
> > src/modules/module-protocol-stub.c | 30 ---
> > src/modules/module-remap-sink.c | 2 -
> > src/modules/module-remap-source.c | 2 -
> > src/modules/module-rescue-streams.c | 2 -
> > src/modules/module-role-cork.c | 2 -
> > src/modules/module-role-ducking.c | 2 -
> > src/modules/module-rygel-media-server.c | 2 -
> > src/modules/module-sine-source.c | 2 -
> > src/modules/module-sine.c | 2 -
> > src/modules/module-solaris.c | 2 -
> > src/modules/module-stream-restore.c | 2 -
> > src/modules/module-suspend-on-idle.c | 2 -
> > src/modules/module-switch-on-connect.c | 2 -
> > src/modules/module-switch-on-port-available.c | 2 -
> > src/modules/module-systemd-login.c | 2 -
> > src/modules/module-tunnel-sink-new.c | 2 -
> > src/modules/module-tunnel-source-new.c | 2 -
> > src/modules/module-tunnel.c | 6 -
> > src/modules/module-udev-detect.c | 2 -
> > src/modules/module-virtual-sink.c | 2 -
> > src/modules/module-virtual-source.c | 2 -
> > src/modules/module-virtual-surround-sink.c | 2 -
> > src/modules/module-volume-restore.c | 2 -
> > src/modules/module-waveout.c | 2 -
> > src/modules/module-zeroconf-discover.c | 2 -
> > src/modules/module-zeroconf-publish.c | 2 -
> > src/modules/oss/module-oss.c | 1 -
> > src/modules/raop/module-raop-discover.c | 1 -
> > src/modules/raop/module-raop-sink.c | 2 -
> > src/modules/rtp/module-rtp-recv.c | 2 -
> > src/modules/rtp/module-rtp-send.c | 2 -
> > src/modules/x11/module-x11-bell.c | 2 -
> > src/modules/x11/module-x11-cork-request.c | 2 -
> > src/modules/x11/module-x11-publish.c | 2 -
> > src/modules/x11/module-x11-xsmp.c | 2 -
> > src/pulsecore/module.h | 29 +++
> > todo | 1 -
> > 90 files changed, 124 insertions(+), 407 deletions(-)
> > delete mode 100644 src/modules/module-defs.h.m4
>
> > -
> > -EXTRA_DIST += $(SYMDEF_FILES)
> > -BUILT_SOURCES += $(SYMDEF_FILES) builddirs
> > -
> > -$(SYMDEF_FILES): modules/module-defs.h.m4
> > - $(AM_V_at)$(MKDIR_P) modules
> > - $(AM_V_GEN)$(M4) -Dfname="$@" $< > $@
> > -
> > .PHONY: builddirs
> > builddirs:
> > $(AM_V_at)$(MKDIR_P) daemon modules
>
> If you remove builddirs from BUILT_SOURCES, you can also remove the
> builddirs target definition, because nothing is referencing it any
> more.
>
> However, is it safe to remove builddirs from BUILT_SOURCES? The
> builddirs target creates the daemon and modules directories, which
> already exist in the source tree, so it's obviously not needed when the
> build directory is the same as the source directory, but maybe it's
> needed when the build directory is different than the source directory?
I always build in a separate tree, so will try and remove builddirs if
it works (my understanding is that those directories will be created if
necessary by the build system anyway).
> > if HAVE_ESOUND
> > module_esound_protocol_tcp_la_SOURCES = modules/module-protocol-stub.c
> > -module_esound_protocol_tcp_la_CFLAGS = -DUSE_TCP_SOCKETS -DUSE_PROTOCOL_ESOUND $(AM_CFLAGS)
> > +module_esound_protocol_tcp_la_CFLAGS = -DUSE_TCP_SOCKETS -DUSE_PROTOCOL_ESOUND $(AM_CFLAGS) -DPA_MODULE_NAME=module_esound_protocol_tcp
> > module_esound_protocol_tcp_la_LDFLAGS = $(MODULE_LDFLAGS)
> > module_esound_protocol_tcp_la_LIBADD = $(MODULE_LIBADD) libprotocol-esound.la
> >
> > module_esound_protocol_unix_la_SOURCES = modules/module-protocol-stub.c
> > -module_esound_protocol_unix_la_CFLAGS = -DUSE_UNIX_SOCKETS -DUSE_PROTOCOL_ESOUND $(AM_CFLAGS)
> > +module_esound_protocol_unix_la_CFLAGS = -DUSE_UNIX_SOCKETS -DUSE_PROTOCOL_ESOUND $(AM_CFLAGS) -DPA_MODULE_NAME=module_esound_protocol_uniix
>
> uniix -> unix
Fixed, thanks.
Is this an ack with the above two things
More information about the pulseaudio-discuss
mailing list