[Buildroot] [PATCH v2 1/3] infra: Move --enable/--disable-debug to package/Makefile.in

[Thomas Petazzoni <thomas.petazzoni@free-electrons.com>]

Dear Yann E. MORIN,

On Sun, 19 Oct 2014 17:15:12 +0200, Yann E. MORIN wrote:

> > Patches 1 and 2 applied. For patch 3, I'm a bit hesitant: there are
> > only two packages using --disable-gtk-doc-html. Does it really make
> > sense to factorize that in the package infrastructure?
> > 
> > Cc'ing Yann to get his input, since he Reviewed-by the patc in question.
> 
> Well, my Reviewed-by means just that: I reviewed it, and found it was
> correct. It does not mean I think it should go in.
> 
> However, I do believe it should go in. The more generic stuff we move
> into the infra, the easier it will get to review packages.
> 
> And since we already have four documentation-rleated switches, it makes
> sense to add another one. Further more, we do have --disable-gtk-doc, so
> adding --disable-gtk-doc-html really makes sense, for consistency.

Well, my personal opinion is that --disable-gtk-doc already shouldn't
have gone to the common package infra. It's far from being a generic
autoconf option, and therefore it generates warnings for many
many ./configure invocations. But the general opinion didn't agree with
mine :)

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com