Re: git: b4d0a174c529 - main - www/ungoogled-chromium: update to 125.0.6422.76 and implement selectable audio backend support

From: Vladimir Druzenko <vvd_at_freebsd.org>
Date: Fri, 24 May 2024 15:02:55 UTC
24.05.2024 17:17, Tomoaki AOKI пишет:
>> 24.05.2024 14:48, Robert Nagy пишет:
>>> On 24/05/24 14:45 GMT, Vladimir Druzenko wrote:
>>>> 24.05.2024 14:36, Robert Nagy пишет:
>>>>> On 24/05/24 14:07 GMT, Vladimir Druzenko wrote:
>>>>>> 24.05.2024 13:40, Robert Nagy пишет:
>>>>>>> On 24/05/24 12:51 GMT, Vladimir Druzenko wrote:
>>>>>>>>> 24.05.2024 06:47, Robert Nagy пишет:
>>>>>>>>> The branch main has been updated by rnagy:
>>>>>>>>>
>>>>>>>>> URL:https://cgit.FreeBSD.org/ports/commit/?id=b4d0a174c529e8061d838aafba1721cc317af01f
>>>>>>>>>
>>>>>>>>> commit b4d0a174c529e8061d838aafba1721cc317af01f
>>>>>>>>> Author:     Robert Nagy<rnagy@FreeBSD.org>
>>>>>>>>> AuthorDate: 2024-05-23 20:21:23 +0000
>>>>>>>>> Commit:     Robert Nagy<rnagy@FreeBSD.org>
>>>>>>>>> CommitDate: 2024-05-24 03:47:31 +0000
>>>>>>>>>
>>>>>>>>>          www/ungoogled-chromium: update to 125.0.6422.76 and implement selectable audio backend support
>>>>>>>>>          with these changes, chromium now automatically selects which
>>>>>>>>>          audio backend to use in the following order:
>>>>>>>>>          pulse (if running) -> sndio -> alsa -> fake
>>>>>>>>>          an additional command line option has been also implemented so
>>>>>>>>>          that the user can force which backend to use:
>>>>>>>>>          --audio-backend={auto,sndio,pulse,alsa}
>>>>>>>>>          Security:https://vuxml.freebsd.org/freebsd/8247af0d-183b-11ef-9f97-a8a1599412c6.html
>>>>>>>>> ---
>      (snip)
>
>>>>>>>> Please remove mandatory dependency (BUILD and LIB) from pulse and sndio and
>>>>>>>> return build options.
>>>>>>>> You can do with OPTIONS_DEFAULT what you want, but don't force install all
>>>>>>>> these dependencies even for build.
>>>>>>> I am sorry but I don't concur. Knobs are bad, there is literally no reason for someone
>>>>>>> not to install sndio (which is the most supported by chrome) by default.
>>>>>> I'm using alsa only build.
>>>>>>> Pulseaudio is only a build dependency which is completely fine.
>>>>>> No. It isn't fine. I don't want pulseaudio on my host - it break my sound.
>>>>> Pulseaudio does not start by default at all, it will not get used unless you have
>>>>> the daemon running and even then you can disable it with the new command line flag
>>>>> of chromium and use whichever audio backend you want by default.
>>>>>
>>>>>>> I am trying to remove even more build options. They make absolutely no sense.
>>>>>> You want to remove one of the most important advantages of the FreeBSD.
>>>>> It would make sense to have an option if it would pull it in as a run or lib dependency,
>>>>> but it does not, however I will look into it to see if we could build it without even
>>>>> having pulseaudio as a build dependency.
>>>> After I install pulseaudio and reboot (without start it via rc.conf) I lost
>>>> sound.
>>>> Please return back options that allow build and run this port without
>>>> installing pulseaudio and sndio.
>>> Please use --audio-backend=alsa.
>> Please return build options.
>>
>> -- 
>> Best regards,
>> Vladimir Druzenko
> You can use poudriere to build this without installing pulseaudio on
> bare-metal environment.
>
> This way, pulseaudio, which is just a BUILD_DEPENDS, is installed inside
> builder jail of poudriere and removed once building package is finished.
> No pollution on bare-metal, running environment.
>
> But if the problematic file is in the distfile of pulseaudio (means,
> not generated at build time), adding the distfile to distfiles and
> extracting somewhere in ${WRKDIR} (and patch if needed) could solve
> your frustration? Robert, how do you think about this?
>
>
> Just a FYI, I've uploaded my tips and tricks at Brew repo [1] to
> configure and use poudriere casually.
>
> [1] https://brew.bsd.cafe/TomAoki/Tips-and-Tricks

One of my installation of this port is on headless VM for export Draw.io 
diagrams to PDF. I don't need any sound on this VM.

Instead of leaving options in the port (what could be simpler?), each 
time the port is updated, I will have to manually remove unnecessary 
dependencies, add "GN_ARGS+=use_pulseaudio=false use_sndio=false" and 
add option ALSA (for desktop ON, but for VM OFF).

Just return back options please.

-- 
Best regards,
Vladimir Druzenko