[Buildroot] [PATCH 1/1] adding-packages-cargo.txt: remove host-cargo select
fontaine.fabrice at gmail.com
Sun Jun 10 13:24:09 UTC 2018
2018-06-10 15:01 GMT+02:00 Thomas Petazzoni <thomas.petazzoni at bootlin.com>:
> On Thu, 7 Jun 2018 20:03:27 +0200, Fabrice Fontaine wrote:
> > Remove select BR2_PACKAGE_HOST_CARGO from Config.in
> > Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
> In fact, I don't agree with this change. Cargo packages are going to
> use host-cargo, so it makes sense for them to select.
> Today, it is not mandatory for a host package to be selected at the
> Config.in level to have it built, and we don't enforce this. But it is
> actually more logical to do it, and in line with our long-term idea of
> having Config.in options for all host packages.
> So, in fact I prefer your original patch, and I disagree with Baruch
> statement that we shouldn't do the select. Yes, it is not /needed/, but
> it is more correct with the select.
OK, so I will send a v3 of my patch's serie.
> Best regards,
> Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
> Embedded Linux and Kernel engineering
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the buildroot