[Buildroot] [PATCH 1/1] bluez5_utils: Add config option to install "bdaddr"
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Wed Feb 15 08:42:04 UTC 2017
Hello,
On Wed, 15 Feb 2017 08:39:11 +0200, Baruch Siach wrote:
> > How many tools are provided by bluez5_utils? If it's just a very small
> > set of tools, it's OK to have one option for each. But if there are
> > many small tools, we definitely don't want to have one option for each.
>
> The bluez5_utils-5.43 tools/ subdirectory has 56 executables, totaling 5.7MB.
> Some are covered by covered by --enable-tools which we enable unconditionally,
> some other are installed by --enable-experimental, and yet others are only
> built by --enable-experimental. bdaddr falls in the last category.
>
> > In any case, we need to provide a better guideline to Vishal, because
> > right now, we are not explaining how the patch should be fixed. If we
> > introduce something like BR2_PACKAGE_BLUEZ5_UTILS_TOOLS, then what
> > should be done with BR2_PACKAGE_BLUEZ5_UTILS_CLIENT or
> > BR2_PACKAGE_BLUEZ5_UTILS_GATTTOOL for example ?
>
> All should go under BR2_PACKAGE_BLUEZ5_UTILS_TOOLS, as well as --enable-tools,
> IMO.
Well, 5.7 MB and 56 executables is definitely not a small amount, so I
would tend to instead accept a more fine-grained selection of the tools.
However, if indeed --enable-tools is optional, we shouldn't enable it
unconditional, but have an option for that.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
More information about the buildroot
mailing list