[Buildroot] [PATCH 1/1] package/weston: add configuration options for weston shells
Yann E. MORIN
yann.morin.1998 at free.fr
Fri May 28 12:01:52 UTC 2021
Martin, All,
On 2021-05-28 12:54 +0200, Martin Elshuber spake thusly:
> On 19/05/2021 12:14, Yann E. MORIN wrote:
> > On 2021-05-11 17:40 +0200, Martin Elshuber spake thusly:
> >> Enable selection of used weston shells. By default all available
> >> shells are enabled to keep the old behavior. The new configuration
> >> options enable the user to select them individually.
> > Don't we have to ensure that at least one is enabled, or is it valid to
> > disable all of them?
> I think it makes sense to allow disabling all shells, since one might
> want to add its own, non default, implementaion.
> I this case the user might want disable all shells that are included in
> the weston package.
I see your point. But then, people writting their own are also probably
be writteing their entire compositor too, probably...
But otherwise, you have a point anyway.
> I suggest to add a help comment:
> help
> if unsure say yes
That has never prevented people from saying 'n' even when they don't
know. You can't prevent anyone from shooting themselves in the foot. ;-)
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list