[Buildroot] [PATCH 10/23 v5] docs/manual: do not hardcode name of the generated document
Samuel Martin
s.martin49 at gmail.com
Mon Sep 22 19:57:07 UTC 2014
Yann,
On Sun, Sep 14, 2014 at 1:07 PM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
> Currently, GENDOC_INNER har-codes some variables to have the name of the
s/har-codes/hard-codes/
> document in them:
> MANUAL_$(2)_ASCIIDOC_CONF
> MANUAL_$(2)_ASCIIDOC_OPTS
> MANUAL_$(2)_A2X_OPTS
> MANUAL_$(2)_INSTALL_CMDS
> ...
>
> Also, it defines some dependency on the generation rule, onto:
> manual-check-dependencies
> manual-check-dependencies-$(3)
> manual-prepare-sources
>
> This is problematic, as it is not possible to have another document
> generated with the GENDOC infra, or it would trigger the rules, and use
> the variables for our own document, 'manual'.
>
> Add a new argument to GENDOC_INNER to be the uppercase name of the
> document, much like it is done for the PKG_*_INNER functions. Replace
> any reference to 'manual' with the currently passed named of the current
> document, $(1).
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> Cc: Samuel Martin <s.martin49 at gmail.com>
> Cc: Thomas De Schampheleire <patrickdepinguin at gmail.com>
With the aformentioned typo fixed, you can add:
Reviewed-by: Samuel Martin <s.martin49 at gmail.com>
--
Samuel
More information about the buildroot
mailing list