[alsa-devel] [PATCH] conf: Allow for a directory to be given as a config file.

[Takashi Iwai]

At Mon, 12 Sep 2011 09:23:15 +0100,
Colin Guthrie wrote:
> 
> 'Twas brillig, and Jaroslav Kysela at 12/09/11 07:57 did gyre and gimble:
> > Date 11.9.2011 14:40, gmane at colin.guthr.ie wrote:
> >> From: Colin Guthrie <colin at mageia.org>
> >>
> >> When this is done, *.conf files can be placed in that directory and they
> >> will be processed by as if they were included directly.
> >>
> >> A directory (typically /usr/share/alsa/alsa.conf.d/) has been
> >> added into the distribution.
> > 
> > Thanks for this patch.
> > 
> > 
> >> + at hooks [
> >> +	{
> >> +		func load
> >> +		files [
> >> +			"@ALSA_CONFIG_DIR@/alsa.conf.d/"
> > 
> > I think that it might be more flexible and analogical to keep the
> > original configuration file and implement the confdir function - see the
> > datadir example.
> > 
> > Implemented:
> > 
> > 	"{ @func datadir }"
> > 
> > To be added:
> > 
> > 	"{ @func confdir }"
> > 
> > See snd_func_datadir() in confmisc.c.
> 
> Sure if you prefer it that way I can implement it. Seems a little overly
> designed to me, but it should be easy enough to support.

The big difference is that you don't hard-code the path in the file.
Even without an addition of confdir, it can be represented like:

	{
		@func concat
		strings [
			{ @func datadir }
			"/alsa.conf.d/"
		]
	}

Takashi