mbox series

[00/10] Slimming BlueZ defaults

Message ID 20210731203814.68125-1-buildroot@heine.tech
Headers show
Series Slimming BlueZ defaults | expand

Message

Michael Nosthoff July 31, 2021, 8:38 p.m. UTC
As discussed in the follow-up of the previous patches to the BlueZ Package
I did the following:

* Make the build of the tools optional
* remove the 'default y' line from the new options
* skim threw the packages and see which ones would require the now optional plugins

None of the modified packages failed to build with the new defaults. But looking at
what these packages do at runtime I suppose they require either the audio or the HID
plugins.

If anyone knows one (or more) of those altered packages in detail I would be happy
to get feedback on the changes.

What I left out for now is how to communicate these changes to the user on upgrade.

Regards,
Michael

Michael Nosthoff (10):
  package/bluez5_utils: make tools optional
  package/bluez5_utils: drop default y
  package/cwiid: select BlueZ HID plugin
  package/bluez-alsa: select BlueZ audio plugins
  package/brltty: enable bluez HID plugin
  package/gstreamer1/gst1-plugins-bad: enable BlueZ Audio
  package/pulseaudio: enable BlueZ Audio Plugins
  package/pipewire: enable BlueZ Audio Plugins
  package/supertuxkart: enable BlueZ HID Plugin
  package/kodi: enable BlueZ Audio and HID plugins

 package/bluez-alsa/Config.in                  |  1 +
 package/bluez5_utils/Config.in                | 22 +++++++++++++------
 package/bluez5_utils/bluez5_utils.mk          | 13 +++++++++--
 package/brltty/Config.in                      |  1 +
 package/cwiid/Config.in                       |  1 +
 package/gstreamer1/gst1-plugins-bad/Config.in |  1 +
 package/kodi/Config.in                        |  2 ++
 package/pipewire/Config.in                    |  1 +
 package/pulseaudio/Config.in                  |  1 +
 package/supertuxkart/Config.in                |  1 +
 10 files changed, 35 insertions(+), 9 deletions(-)

Comments

Arnout Vandecappelle Aug. 2, 2021, 3:41 p.m. UTC | #1
On 31/07/2021 22:38, Michael Nosthoff wrote:
> As discussed in the follow-up of the previous patches to the BlueZ Package
> I did the following:
> 
> * Make the build of the tools optional
> * remove the 'default y' line from the new options
> * skim threw the packages and see which ones would require the now optional plugins

 Series applied to master, thanks!


> None of the modified packages failed to build with the new defaults. But looking at
> what these packages do at runtime I suppose they require either the audio or the HID
> plugins.
> 
> If anyone knows one (or more) of those altered packages in detail I would be happy
> to get feedback on the changes.

 I've applied the series to master regardless. If the automatic select bothers
someone, they can send a patch.

> 
> What I left out for now is how to communicate these changes to the user on upgrade.

 I'm cooking an RFC patch for that. Stay tuned.

 Regards,
 Arnout

> 
> Regards,
> Michael
> 
> Michael Nosthoff (10):
>   package/bluez5_utils: make tools optional
>   package/bluez5_utils: drop default y
>   package/cwiid: select BlueZ HID plugin
>   package/bluez-alsa: select BlueZ audio plugins
>   package/brltty: enable bluez HID plugin
>   package/gstreamer1/gst1-plugins-bad: enable BlueZ Audio
>   package/pulseaudio: enable BlueZ Audio Plugins
>   package/pipewire: enable BlueZ Audio Plugins
>   package/supertuxkart: enable BlueZ HID Plugin
>   package/kodi: enable BlueZ Audio and HID plugins
> 
>  package/bluez-alsa/Config.in                  |  1 +
>  package/bluez5_utils/Config.in                | 22 +++++++++++++------
>  package/bluez5_utils/bluez5_utils.mk          | 13 +++++++++--
>  package/brltty/Config.in                      |  1 +
>  package/cwiid/Config.in                       |  1 +
>  package/gstreamer1/gst1-plugins-bad/Config.in |  1 +
>  package/kodi/Config.in                        |  2 ++
>  package/pipewire/Config.in                    |  1 +
>  package/pulseaudio/Config.in                  |  1 +
>  package/supertuxkart/Config.in                |  1 +
>  10 files changed, 35 insertions(+), 9 deletions(-)
>