From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 06/16 v3] core: introduce a generated kconfig snippet
Date: Tue, 30 Aug 2016 11:00:18 +0200 [thread overview]
Message-ID: <20160830090018.GA5815@free.fr> (raw)
In-Reply-To: <b560c751-1ce1-f45a-5ab7-2e06ba169530@mind.be>
Arnout, All,
On 2016-08-30 00:46 +0200, Arnout Vandecappelle spake thusly:
> On 30-08-16 00:16, Yann E. MORIN wrote:
> > Peter, Thomas, All,
> >
> > On 2016-08-28 22:50 +0200, Peter Korsgaard spake thusly:
> >>>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:
> >> > On Sun, 17 Jul 2016 12:34:26 +0200, Yann E. MORIN wrote:
> >> >> +# Ensure the kconfig snippet is generated (for MANUAL_GEN_LISTS, below):
> >> >> +MANUAL_DEPENDENCIES += prepare-kconfig
> >>
> >> > It's a bit crazy the complexity that we have just to allow our manual
> >> > to have the list of all packages in Buildroot. I'm not even sure this
> >> > section of the manual is that useful, to be honest. But oh well, that's
> >> > supported today, so we have to continue support it.
> >>
> >> Why do we _HAVE_ to continue to support it?
> >>
> >> If we don't find the complexity worth the added value (I personally
> >> don't, I think we could just ask people to look under package/ for the
> >> list of packages), then we should just remove it.
> >
> > I'm not entirely against removing the list of out packages from our
> > manual.
> >
> > Yet, for some users of br2-external trees, they may still want to have
> > the list of their packages in their manuals, build with the asciidoc
> > infrastructure.
> >
> > It is entirely imaginable that such users of br2-external have packages
> > annotated with some documentation for those packages, and want to
> > include that doc in their manual, so need to generate that list from
> > Kconfig.
>
> I'm not really enthousiastic about keeping complexity in our Makefiles for
> imaginary users that may want to have such and such feature :-P
>
> I'm very enthousiastic when people contribute some feature that I'm never going
> to use myself - but in that case, we know that there is at least one user (the
> contributor). But here, we really have added a feature with no known users.
Have you read what I wrote: I *am* using it.
http://lists.busybox.net/pipermail/buildroot/2016-August/171055.html
> At
> the time, it didn't add (much) complexity, but more and more it's starting to
> weigh down I think.
>
> For sure, the asciidoc support is even more difficult to understand and
> maintain than the normal package infras.
And I seem to have a good track-record at looking at it:
$ git log --follow --pretty=%an package/doc-asciidoc.mk |sort |uniq -c |sort -k1nr,2
22 Yann E. MORIN
8 Samuel Martin
7 Thomas De Schampheleire
2 Thomas Petazzoni
1 Atul Singh
1 J?r?me Pouiller
So, please, please. let's keep this asciidoc infra in place.
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
next prev parent reply other threads:[~2016-08-30 9:00 UTC|newest]
Thread overview: 95+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-17 10:34 [Buildroot] [PATCH 00/16 v3] br2-external: support multiple trees at once (branch yem/multi-br2-external-ID-7) Yann E. MORIN
2016-07-17 10:34 ` [Buildroot] [PATCH 01/16 v3] core: move pkg-utils.mk to support/ Yann E. MORIN
2016-07-24 20:43 ` Romain Naour
2016-08-27 14:11 ` Thomas Petazzoni
2016-07-17 10:34 ` [Buildroot] [PATCH 02/16 v3] core: commonalise the bundled and br2-external %_defconfig rules Yann E. MORIN
2016-07-30 20:48 ` Romain Naour
2016-08-27 14:12 ` Thomas Petazzoni
2016-08-27 14:16 ` Yann E. MORIN
2016-08-27 17:10 ` Arnout Vandecappelle
2016-07-17 10:34 ` [Buildroot] [PATCH 03/16 v3] doc/asciidoc: add possibility to define document dependencies Yann E. MORIN
2016-08-06 15:02 ` Romain Naour
2016-08-27 14:14 ` Thomas Petazzoni
2016-08-27 14:42 ` Yann E. MORIN
2016-08-27 19:58 ` Thomas Petazzoni
2016-07-17 10:34 ` [Buildroot] [PATCH 04/16 v3] core: introduce an intermediate rule before the configurators Yann E. MORIN
2016-08-06 15:13 ` Romain Naour
2016-08-27 14:47 ` Yann E. MORIN
2016-08-27 19:39 ` Thomas Petazzoni
2016-07-17 10:34 ` [Buildroot] [PATCH 05/16 v3] core: move rule to create basic directories Yann E. MORIN
2016-08-06 15:16 ` Romain Naour
2016-08-27 14:14 ` Thomas Petazzoni
2016-08-27 14:34 ` Yann E. MORIN
2016-08-27 19:41 ` Thomas Petazzoni
2016-08-27 22:07 ` Yann E. MORIN
2016-07-17 10:34 ` [Buildroot] [PATCH 06/16 v3] core: introduce a generated kconfig snippet Yann E. MORIN
2016-08-06 15:18 ` Romain Naour
2016-08-27 14:16 ` Thomas Petazzoni
2016-08-27 15:01 ` Yann E. MORIN
2016-08-28 20:50 ` Peter Korsgaard
2016-08-29 7:19 ` Thomas Petazzoni
2016-08-29 7:50 ` Peter Korsgaard
2016-08-29 10:02 ` Thomas Petazzoni
2016-08-29 22:14 ` Arnout Vandecappelle
2016-08-29 22:25 ` Yann E. MORIN
2016-08-29 22:16 ` Yann E. MORIN
2016-08-29 22:46 ` Arnout Vandecappelle
2016-08-30 9:00 ` Yann E. MORIN [this message]
2016-08-30 9:40 ` Peter Korsgaard
2016-08-30 20:01 ` Arnout Vandecappelle
2016-09-01 17:52 ` Yann E. MORIN
2016-09-01 18:57 ` Peter Korsgaard
2016-09-01 21:30 ` Arnout Vandecappelle
2016-09-01 21:51 ` Peter Korsgaard
2016-08-27 19:46 ` Thomas Petazzoni
2016-07-17 10:34 ` [Buildroot] [PATCH 07/16 v3] docs/manual: prepare-config can be used as a dependency of documents Yann E. MORIN
2016-08-06 15:19 ` Romain Naour
2016-08-27 19:58 ` Thomas Petazzoni
2016-07-17 10:34 ` [Buildroot] [PATCH 08/16 v3] core: do not hard-code inclusion of br2-external in Kconfig Yann E. MORIN
2016-08-06 15:21 ` Romain Naour
2016-08-27 20:00 ` Thomas Petazzoni
2016-08-27 22:14 ` Yann E. MORIN
2016-07-17 10:34 ` [Buildroot] [PATCH 09/16 v3] core: get rid of our dummy br2-external tree Yann E. MORIN
2016-08-06 15:30 ` Romain Naour
2016-08-06 15:37 ` Yann E. MORIN
2016-07-17 10:34 ` [Buildroot] [PATCH 10/16 v3] core: introduce per br2-external ID Yann E. MORIN
2016-08-06 15:41 ` Romain Naour
2016-08-27 20:10 ` Thomas Petazzoni
2016-08-27 22:15 ` Yann E. MORIN
2016-08-27 21:49 ` Arnout Vandecappelle
2016-08-27 21:59 ` Yann E. MORIN
2016-08-28 10:48 ` Thomas Petazzoni
2016-07-17 10:34 ` [Buildroot] [PATCH 11/16 v3] docs/manual: document the " Yann E. MORIN
2016-08-06 15:48 ` Romain Naour
2016-08-27 15:05 ` Yann E. MORIN
2016-08-27 20:13 ` Thomas Petazzoni
2016-08-27 22:19 ` Yann E. MORIN
2016-08-30 20:36 ` Arnout Vandecappelle
2016-07-17 10:34 ` [Buildroot] [PATCH 12/16 v3] core: handle .br-external in a make rule Yann E. MORIN
2016-07-24 20:05 ` Romain Naour
2016-07-25 21:13 ` Yann E. MORIN
2016-08-27 15:51 ` Yann E. MORIN
2016-08-27 20:16 ` Thomas Petazzoni
2016-08-27 22:22 ` Yann E. MORIN
2016-08-28 10:49 ` Thomas Petazzoni
2016-07-17 10:34 ` [Buildroot] [PATCH 13/16 v3] core: add support for multiple br2-external trees Yann E. MORIN
2016-07-24 20:25 ` Romain Naour
2016-07-25 21:25 ` Yann E. MORIN
2016-08-27 20:20 ` Thomas Petazzoni
2016-08-29 22:11 ` Yann E. MORIN
2016-07-17 10:34 ` [Buildroot] [PATCH 14/16 v3] support/scripts: teach gen-manual-lists about BR2_EXTERNAL Yann E. MORIN
2016-08-06 15:59 ` Romain Naour
2016-07-17 10:34 ` [Buildroot] [PATCH 15/16 v3] docs/manual: include packages from BR2_EXTERNAL if set Yann E. MORIN
2016-08-06 16:07 ` Romain Naour
2016-07-17 10:34 ` [Buildroot] [PATCH 16/16 v3] docs/manual: document multi br2-external Yann E. MORIN
2016-08-06 16:15 ` Romain Naour
2016-08-30 21:37 ` [Buildroot] [PATCH 00/16 v3] br2-external: support multiple trees at once (branch yem/multi-br2-external-ID-7) Arnout Vandecappelle
2016-08-31 6:15 ` Peter Korsgaard
2016-08-31 7:14 ` Thomas Petazzoni
2016-08-31 9:26 ` Yann E. MORIN
2016-09-01 21:43 ` Arnout Vandecappelle
2016-09-01 21:46 ` Yann E. MORIN
2016-09-01 22:04 ` Arnout Vandecappelle
2016-09-04 18:42 ` Peter Korsgaard
2016-09-04 21:14 ` Yann E. MORIN
2016-09-05 10:31 ` Peter Korsgaard
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=20160830090018.GA5815@free.fr \
--to=yann.morin.1998@free.fr \
--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