From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yann E. MORIN Date: Sun, 7 Sep 2014 17:03:57 +0200 Subject: [Buildroot] [PATCH v3 2/4] gendoc infra: move manual build location into $(BUILD_DIR)/manual In-Reply-To: <1409078947-5093-2-git-send-email-s.martin49@gmail.com> References: <1409078947-5093-1-git-send-email-s.martin49@gmail.com> <1409078947-5093-2-git-send-email-s.martin49@gmail.com> Message-ID: <20140907150357.GG31418@free.fr> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Samuel, All, On 2014-08-26 20:49 +0200, Samuel Martin spake thusly: > This patch reworks the manual source preparation by: > - moving the build directory under $(BUILD_DIR)/, this keeps consistency > with the other Buildroot infrastructures; > - adding a couple of targets: 'manual-rsync' and 'manual-prepare-sources', > to deal more efficiently with the manual sources and avoid rsync-ing > them on every single manual-* target. > > The 'manual-rsync' target only copies the manual sources under git, while > the 'manual-prepare-sources' also takes care of the generated ones. These > targets are now run only once, and the manual build is no longer cleaned > after each manual format generation. > > Now, the 'manual-clean' target only remove the manual build directory, but > keeps the output one $(O)/output/doc/manual unchanged. > > Doing so (moving the manual build directory and keeping it between 2 > manual format generation) ensures that all generated sources are taken > in account when generating the manual [1]. > > [1] http://lists.busybox.net/pipermail/buildroot/2014-August/104421.html > > Cc: Thomas De Schampheleire > Signed-off-by: Samuel Martin Woot! I totally love this patch! :-) It will make it so much easier to build a manual-like document from br2-external! :-) But see a comment below... > diff --git a/docs/manual/manual.mk b/docs/manual/manual.mk > index 136f725..e86f872 100644 > --- a/docs/manual/manual.mk > +++ b/docs/manual/manual.mk > @@ -1,11 +1,20 @@ > +$(BUILD_DIR)/$(pkgname): > + $(Q)mkdir -p $@ > + > +manual-rsync: $(BUILD_DIR)/$(pkgname) > + $(Q)$(call MESSAGE,"Preparing the manual sources...") > + $(Q)rsync -a docs/$(pkgname)/ $(BUILD_DIR)/$(pkgname) Would it be possible to still have this as part of GENDOC_INNER, so it is useable to generate another document? I think I'll cary this patch series in my branch, and do some additional (probably cumulative, so it is not a show-blocker) changes. So: Reviewed-by: "Yann E. MORIN" Thanks again for this work! :-) Regards, Yann E. MORIN. > # Packages included in BR2_EXTERNAL are not part of buildroot, so they > # should not be included in the manual. > -manual-update-lists: manual-check-dependencies-lists > +manual-update-lists: manual-check-dependencies-lists $(BUILD_DIR)/$(pkgname) > $(Q)$(call MESSAGE,"Updating the manual lists...") > - $(Q)BR2_DEFCONFIG="" TOPDIR=$(TOPDIR) O=$(O)/docs/manual/.build \ > + $(Q)BR2_DEFCONFIG="" TOPDIR=$(TOPDIR) O=$(BUILD_DIR)/$(pkgname) \ > BR2_EXTERNAL=$(TOPDIR)/support/dummy-external \ > python -B $(TOPDIR)/support/scripts/gen-manual-lists.py > > +manual-prepare-sources: manual-rsync manual-update-lists > + > # we can't use suitable-host-package here because that's not available in > # the context of 'make release' > manual-check-dependencies: > @@ -65,14 +74,13 @@ $$(O)/docs/$(1)/$(1).$(4): docs/$(1)/$(1).txt \ > $$($$(call UPPERCASE,$(1))_SOURCES) \ > manual-check-dependencies \ > manual-check-dependencies-$(3) \ > - manual-update-lists > + manual-prepare-sources > $$(Q)$$(call MESSAGE,"Generating $(5) $(1)...") > - $$(Q)mkdir -p $$(@D)/.build > - $$(Q)rsync -au docs/$(1)/*.txt $$(@D)/.build > + $$(Q)mkdir -p $$(@D) > $$(Q)a2x $(6) -f $(2) -d book -L -r $$(TOPDIR)/docs/images \ > --asciidoc-opts="$$(MANUAL_$(2)_ASCIIDOC_OPTS)" \ > - -D $$(@D) $$(@D)/.build/$(1).txt > - -$$(Q)rm -rf $$(@D)/.build > + -D $$(@D) \ > + $$(BUILD_DIR)/$(1)/$(1).txt > endef > > ################################################################################ > @@ -94,7 +102,7 @@ $(call GENDOC_INNER,$(pkgname),text,text,text,text) > $(call GENDOC_INNER,$(pkgname),epub,epub,epub,ePUB) > clean: $(pkgname)-clean > $(pkgname)-clean: > - $$(Q)$$(RM) -rf $$(O)/docs/$(pkgname) > + $$(Q)$$(RM) -rf $$(BUILD_DIR)/$(pkgname) > .PHONY: $(pkgname) $(pkgname)-clean manual-update-lists > endef > > -- > 2.1.0 > > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot -- .-----------------.--------------------.------------------.--------------------. | 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. | '------------------------------^-------^------------------^--------------------'