[Buildroot] [PATCH 1/1] adding-packages-cargo.txt: remove host-cargo select

Fabrice Fontaine fontaine.fabrice at gmail.com
Sun Jun 10 13:24:09 UTC 2018


Dear Thomas,

2018-06-10 15:01 GMT+02:00 Thomas Petazzoni <thomas.petazzoni at bootlin.com>:

> Hello,
>
> 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,
>
> Thoms
> --
> Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
> Embedded Linux and Kernel engineering
> https://bootlin.com
>
Best Regards,

Fabrice
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180610/daa33b6c/attachment-0002.html>


More information about the buildroot mailing list