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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 22 Mar 2019 16:44:10 +0100,
Alex Ivanov wrote:
> 
> Hi, Takashi.
> 
> Sure. Here we go.
> 
> Signed-off-by: Alex Ivanov <gnidorah@xxxxx>

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
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux