diff mbox series

[-,JACK,plugin,1/1] configure: use /etc instead of /etc

Message ID 20190322154410.10712-1-gnidorah@ya.ru (mailing list archive)
State New, archived
Headers show
Series [-,JACK,plugin,1/1] configure: use /etc instead of /etc | expand

Commit Message

Alex Ivanov March 22, 2019, 3:44 p.m. UTC
Hi, Takashi.

Sure. Here we go.

Signed-off-by: Alex Ivanov <gnidorah@ya.ru>
---
 configure.ac | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Takashi Iwai March 22, 2019, 5:52 p.m. UTC | #1
On Fri, 22 Mar 2019 16:44:10 +0100,
Alex Ivanov wrote:
> 
> Hi, Takashi.
> 
> Sure. Here we go.
> 
> Signed-off-by: Alex Ivanov <gnidorah@ya.ru>

Erm, please keep more context in the patch description.

> ---
>  configure.ac | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 5b80585..b52923c 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -237,7 +237,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="$sysconfdir/alsa/conf.d"
>  fi

I tested this change, and it resulted in a bad string
  #define ALSA_LCONF_DIR "${prefix}/etc/alsa/conf.d"

Actually the same bug is found in other places, e.g.
  #define ALSA_DATA_DIR "${prefix}/share/alsa"

Hmm.


Takashi
diff mbox series

Patch

diff --git a/configure.ac b/configure.ac
index 5b80585..b52923c 100644
--- a/configure.ac
+++ b/configure.ac
@@ -237,7 +237,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="$sysconfdir/alsa/conf.d"
 fi
 AC_DEFINE_UNQUOTED(ALSA_LCONF_DIR, "$alsalconfdir", [directory containing local ALSA add-on config files])
 ALSA_LCONF_DIR="$alsalconfdir"