[Buildroot] [PATCH 4 of 7] packages: remove support for documentation on target

Thomas De Schampheleire patrickdepinguin at gmail.com
Thu Feb 6 12:15:10 UTC 2014


On Thu, Feb 6, 2014 at 12:40 PM, Thomas Petazzoni
<thomas.petazzoni at free-electrons.com> wrote:
> Dear Thomas De Schampheleire,
>
> On Thu, 6 Feb 2014 12:34:00 +0100, Thomas De Schampheleire wrote:
>
>> >  But of course that's a large change, since all 846 autotools packages
>> > would have to be checked...
>> >
>> >  So let's start with not adding anything to the global CONF_OPT :-)
>>
>> With this conclusion, I'd say we mark the last patch as Rejected (the
>> one removing the separate DISABLE_DOCUMENTATION variable).
>
> I don't really come to the same conclusion. Arnout said to not add more
> things to the global CONF_OPT.
>
> The existing options in DISABLE_DOCUMENTATION are already part of the
> CONF_OPT, so having a small refactoring that removes
> DISABLE_DOCUMENTATION and makes it part of the global CONF_OPT doesn't
> change anything in terms of removing those global options and moving
> them to a per-package solution.

My reasoning was that if we are going to migrate the global options to
per-package options, then it is not needed to change anything about
this now. Maybe this is too swift a conclusion...

I have no very strong opinion about the last patch, so feel free to
apply or reject it.

Best regards,
Thomas



More information about the buildroot mailing list