build-sys: fix PULSE_LOCALEDIR definition
authorTanu Kaskinen <tanuk@iki.fi>
Tue, 29 Sep 2015 08:41:51 +0000 (11:41 +0300)
committerDavid Henningsson <david.henningsson@canonical.com>
Mon, 7 Dec 2015 09:41:37 +0000 (10:41 +0100)
commit74c9c410313b1419beff90468923c574f54eda88
tree7b5d3dc3a1da5745f43bf0433bdc8b3993e54c88
parent96f4b4ffe341a4d7c4a84dd46a9c9f60d94928d2
build-sys: fix PULSE_LOCALEDIR definition

On some systems (at least Arch) DATADIRNAME is not defined. This
caused PULSE_LOCALEDIR to point to a wrong directory. This seemed like
an issue introduced in 7.0, but probably something else was updated in
Arch at the same time, causing DATADIRNAME to become undefined,
because there were no changes between 6.0 and 7.0 that could have
caused this.

After noticing that localedir is a standard variable, my first idea
was to use pulselocaledir='${localedir}' in configure.ac, but Jan
Steffens pointed out that it causes the final PULSE_LOCALEDIR to
become "${prefix}/share/locale", that is, the variables weren't fully
expanded. I then found a FAQ item in Autoconf's manual[1], which
recommends not to define any absolute installation directories in
configure.ac, because the installation directories should be possible
to change when running make. The recommended solution is to define the
constant in AM_CPPFLAGS instead, so that's what this patch does.

[1] https://www.gnu.org/software/autoconf/manual/autoconf-2.69/html_node/Defining-Directories.html
configure.ac
src/Makefile.am