[pulseaudio-discuss] [PATCH] build-sys: Stop using symdef headers for modules
Tanu Kaskinen
tanuk at iki.fi
Tue Dec 12 05:40:38 UTC 2017
On Tue, 2017-12-12 at 10:41 +0530, Arun Raghavan wrote:
>
> 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
Yes, I don't have any other issues with the patch.
--
Tanu
https://www.patreon.com/tanuk
More information about the pulseaudio-discuss
mailing list