On Wed, 20 Mar 2019 18:30:47 +0100, Alex Ivanov wrote:

Use $sysconfdir as prefix for ALSA_LCONF_DIR by default. Otherwise install fails on non-FHS distros.

Signed-off-by: Alex Ivanov gnidorah@ya.ru

configure.ac | 19 ++++++++++++++++++- 1 file changed, 18 insertions(+), 1 deletion(-)

diff --git a/configure.ac b/configure.ac index 5b80585..7b8dd66 100644 --- a/configure.ac +++ b/configure.ac @@ -219,6 +219,23 @@ AC_DEFINE_UNQUOTED(ALSA_DATA_DIR, "$alsadatadir", [directory containing ALSA dat ALSA_DATA_DIR="$alsadatadir" AC_SUBST(ALSA_DATA_DIR)

+dnl ALSA configuration directory +AC_ARG_WITH(alsaconfdir,

• AS_HELP_STRING([--with-alsaconfdir=dir],
• [path where ALSA configuration files are stored]),
• alsaconfdir="$withval", alsaconfdir="")

+if test -z "$alsaconfdir"; then

• eval dir="$sysconfdir"
• case "$dir" in
• /*) ;;
• *) dir="$dir"
• esac
• alsaconfdir="$dir/alsa"

+fi +AC_DEFINE_UNQUOTED(ALSA_CONF_DIR, "$alsaconfdir", [directory containing ALSA configuration files]) +ALSA_CONF_DIR="$alsaconfdir" +AC_SUBST(ALSA_CONF_DIR)

Yet another option is way too confusing, IMO.

dnl ALSA add-on global config directory AC_ARG_WITH(alsagconfdir, AS_HELP_STRING([--with-alsagconfdir=dir], @@ -237,7 +254,7 @@ AC_ARG_WITH(alsalconfdir, [path where ALSA local add-on config files are stored]), alsalconfdir="$withval", alsalconfdir="") if test -z "$alsalconfdir"; then

• alsalconfdir="/etc/alsa/conf.d"

• alsalconfdir="$ALSA_CONF_DIR/conf.d"

Let's add the $sysconfdir usage here instead of adding a new --with-*.

Care to resubmit with that change?

thanks,

Takashi