[pulseaudio-discuss] [PATCH] Add HDMI 5.1 and 7.1 profiles to default.conf as well

[Alexander E. Patrakov]

2014-07-30 16:48 GMT+06:00 David Henningsson <david.henningsson at canonical.com>:
>
>
> On 2014-07-30 12:34, Alexander E. Patrakov wrote:
>>
>> 2014-07-30 16:31 GMT+06:00 David Henningsson
>> <david.henningsson at canonical.com>:
>>>
>>>
>>>
>>> On 2014-07-30 11:59, Alexander E. Patrakov wrote:
>>>>
>>>> Due to these complications, I don't think that the entire "include"
>>>> idea makes sense. I will try an alternative approach - a shell script
>>>> that writes the conf files that share the common parts.
>>>
>>>
>>>
>>> Since we're planning to merge extra-hdmi.conf with default.conf (right?),
>>> and force-speaker*.conf is probably going away at some not too distant
>>> future, we're left with default.conf only.
>>>
>>> So common parts between what files...?
>>
>>
>> I am going to generate default.conf and force-speaker*.conf, with the
>> HDMI part merged into both. Thats because force-speaker* are not going
>> away today, according to Tanu.
>>
>> And even then, the shell script makes sense in order to avoid
>> copy-pasting the same HDMI-related things 8 times.
>
>
> If you want to make a shell script, run it locally, and turn the result into
> a patch, that's your business. If you want to make the shell script part of
> your patch, I think that's a bad idea - the disadvantages of the additional
> complexity weighs heavier than the benefit of less copy-paste.

I disagree, but will do as you say.

> Also, I don't think force-speaker*.conf ever needs more than first HDMI set
> (i e hdmi:%f,0 in stereo, 5.1 and 7.1).
> Unless you have an alsa-info counterexample?

I have no counterexample and will take this consideration into account. Thanks.

-- 
Alexander E. Patrakov