[Buildroot] [PATCH 1/9] manual: minor update in manual generation
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Sun Oct 6 09:04:26 UTC 2013
Dear Samuel Martin,
On Sat, 5 Oct 2013 21:49:06 +0200, Samuel Martin wrote:
> * minor refactoring in gen-manual-list.py
> * generate manual in the build tree
Why? This is breaking the "release:" target of the main Makefile, which
assumes that the manual is generated in $(O)/docs/manual.
> diff --git a/support/scripts/gen-manual-lists.py b/support/scripts/gen-manual-lists.py
> index 3568843..5c81e1d 100755
> --- a/support/scripts/gen-manual-lists.py
> +++ b/support/scripts/gen-manual-lists.py
> @@ -107,7 +107,7 @@ def get_symbol_parents(item, root=None, enable_choice=False):
> parent = parent.get_parent()
> if isinstance(root, kconfiglib.Menu) or \
> (enable_choice and isinstance(root, kconfiglib.Choice)):
> - parents.append("") # Dummy empty parrent to get a leading arrow ->
> + parents.append("") # Dummy empty parent to get a leading arrow ->
> parents.reverse()
> return parents
>
> @@ -126,15 +126,14 @@ def format_asciidoc_table(root, get_label_func, filter_func=lambda x: True,
> :param sub_menu: Output the column with the sub-menu path
>
> """
> - def _format_entry(label, parents, sub_menu):
> + def _format_entry(item, parents, sub_menu):
> """ Format an asciidoc table entry.
>
> """
> if sub_menu:
> - return "| {0:<40} <| {1}\n".format(label, " -> ".join(parents))
> + return "| {0:<40} <| {1}\n".format(item, " -> ".join(parents))
> else:
> - return "| {0:<40}\n".format(label)
> -
> + return "| {0:<40}\n".format(item)
> lines = []
> for item in get_symbol_subset(root, filter_func):
> if not item.is_symbol() or not item.prompts:
Also, why are those changes mixed with the change of building the
manual in $(O)/build ? They seem to be completely unrelated.
Sorry to be a bit harsh, but you're a regular contributor, so I'd
expect patches from you that are properly split in individual patches
for each feature or fix, with a more descriptive commit log. We've
already suggested you to write more detailed commit logs in the past, I
believe.
Thanks!
Thomas
--
Thomas Petazzoni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list