From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 4 of 7] packages: remove support for documentation on target
Date: Thu, 6 Feb 2014 09:37:24 +0100 [thread overview]
Message-ID: <20140206093724.52c0fb0f@skate> (raw)
In-Reply-To: <52F2AB5A.8000006@mind.be>
Dear Arnout Vandecappelle,
On Wed, 05 Feb 2014 22:21:30 +0100, Arnout Vandecappelle wrote:
> > WARNING: options --disable-foo --enable-bar unsupported
> >
> > If you have gazillions of options that are unsupported, then you may
> > very well miss an option passed by your .mk file on which you made a
> > typo, for example.
> >
> > So having a long list of options to disable documentation is not
> > something that I personally like that much, but I believe Arnout does
> > not share the same opinion :)
>
> I have a mixed opinion about this...
>
> I do think that the configure options should ideally be as exact as
> possible. However, documentation is something that is wont to cause hard
> to track build failures
Not sure what you meant here. Will those build failures be hard to
track, or *not* hard to track ?
Remember that the Free Electrons autobuilder builds run in a chroot
that has only the minimal hard dependencies required by Buildroot. This
typically allows us to catch invisible dependencies on asciidoc or
other documentation utilities.
>, e.g. trying to run system asciidoc with host
> python, or system makeinfo with host-perl. So if we remove it from the
> defaults, then we should make sure that _all_ autotools packages disable
> their documentation.
Indeed.
> Hm, now I think about that, it would actually be a good idea to do
> that... We currently still miss some of these cases because we assume
> --disable-documentation works, but if we make a habit of paying attention
> to it, then there is less risk of this happening.
So your conclusion is that instead of passing those options globally,
we should pass them in a per-package basis, so that we remember to
always verify that the documentation disabling has been done?
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2014-02-06 8:37 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-05 10:50 [Buildroot] [PATCH 0 of 7] Remove remaining deprecated packages for 2014.02 Thomas De Schampheleire
2014-02-05 10:50 ` [Buildroot] [PATCH 1 of 7] xstroke: remove deprecated package Thomas De Schampheleire
2014-02-05 10:50 ` [Buildroot] [PATCH 2 of 7] autoconf: remove deprecated target package Thomas De Schampheleire
2014-02-05 10:50 ` [Buildroot] [PATCH 3 of 7] automake: " Thomas De Schampheleire
2014-02-05 10:50 ` [Buildroot] [PATCH 4 of 7] packages: remove support for documentation on target Thomas De Schampheleire
2014-02-05 12:49 ` Thomas Petazzoni
2014-02-05 13:10 ` Thomas De Schampheleire
2014-02-05 13:16 ` Thomas Petazzoni
2014-02-05 13:28 ` Thomas De Schampheleire
2014-02-05 13:34 ` Thomas Petazzoni
2014-02-05 21:21 ` Arnout Vandecappelle
2014-02-06 8:37 ` Thomas Petazzoni [this message]
2014-02-06 10:26 ` Arnout Vandecappelle
2014-02-06 11:34 ` Thomas De Schampheleire
2014-02-06 11:40 ` Thomas Petazzoni
2014-02-06 12:15 ` Thomas De Schampheleire
2014-02-06 13:33 ` Arnout Vandecappelle
2014-02-06 22:19 ` Peter Korsgaard
2014-02-05 10:50 ` [Buildroot] [PATCH 5 of 7] ccache: remove deprecated target package Thomas De Schampheleire
2014-02-05 10:50 ` [Buildroot] [PATCH 6 of 7] gdb: remove deprecated versions 7.2.x and 7.3.x Thomas De Schampheleire
2014-02-05 10:50 ` [Buildroot] [PATCH 7 of 7] kernel-headers: remove deprecated versions 3.1, 3.3, 3.5 Thomas De Schampheleire
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140206093724.52c0fb0f@skate \
--to=thomas.petazzoni@free-electrons.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox