[Buildroot] [PATCH v2 6/6] gendoc infra: Move the manual-clean target outside of the infra
Thomas De Schampheleire
patrickdepinguin at gmail.com
Sun Aug 24 13:06:50 UTC 2014
On Thu, Aug 21, 2014 at 10:25 PM, Samuel Martin <s.martin49 at gmail.com> wrote:
> Signed-off-by: Samuel Martin <s.martin49 at gmail.com>
>
> ---
> changes v1 -> v2:
> - no change
> ---
> docs/manual/manual.mk | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/docs/manual/manual.mk b/docs/manual/manual.mk
> index 437eccf..74e591f 100644
> --- a/docs/manual/manual.mk
> +++ b/docs/manual/manual.mk
> @@ -140,10 +140,12 @@ $(call GENDOC_INNER,$(pkgname),pdf,pdf,pdf,PDF,\
> --dblatex-opts "-P latex.output.revhistory=0 -x '--maxvars 100000'")
> $(call GENDOC_INNER,$(pkgname),text,text,text,text)
> $(call GENDOC_INNER,$(pkgname),epub,epub,epub,ePUB)
> +endef
> +
> clean: $(pkgname)-clean
> +
> $(pkgname)-clean:
> - $$(Q)$$(RM) -rf $$(BUILD_DIR)/$(pkgname)
> -endef
> + $(Q)$(RM) -rf $(BUILD_DIR)/$(pkgname)
>
> MANUAL_SOURCES = $(sort $(wildcard docs/manual/*.txt) $(wildcard docs/images/*))
> $(eval $(call GENDOC))
> --
This kind of defeats the purpose of having a GENDOC infra: suppose we
add a new type of document next to 'manual'. In this case, we will
have to create a new directory docs/XXXX/ and make sure that the
GENDOC infra moves to another file, say package/pkg-gendoc.mk
Then, we want XXXX-clean to exist too, which is why it should be within GENDOC.
If we don't care about such a case, then we could also simply remove
GENDOC and only keep GENDOC_INNER.
In that case we don't even need $(pkgname).
Best regards,
Thomas
More information about the buildroot
mailing list