[Buildroot] [PATCH v2 2/6] gendoc infra: move manual build location into $(BUILD_DIR)/manual
s.martin49 at gmail.com
Mon Aug 25 19:16:17 UTC 2014
On Fri, Aug 22, 2014 at 2:44 PM, Thomas De Schampheleire
<thomas.de.schampheleire at gmail.com> wrote:
> Hi Samuel,
> On Thu, Aug 21, 2014 at 10:25 PM, Samuel Martin <s.martin49 at gmail.com> wrote:
>> This patch reworks the mannual source preparetion by:
>> - moving the build directory under $(BUILD_DIR)/, this keeps consistency
>> with the other Buildroot infrastructure;
>> - adding a couple of targets: 'manual-rsync' and 'manual-prepare-sources',
>> to deal more efficiently the manual sources and avoid rsync-ing them on
> _with_ the manual sources
>> every single manual-* target.
>> The 'manual-rsync' target only copy 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
>> keep 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 taking
>> in account when generating the manual .
>>  http://lists.busybox.net/pipermail/buildroot/2014-August/104421.html
>> Cc: Thomas De Schampheleire <thomas.de.schampheleire at gmail.com>
>> Signed-off-by: Samuel Martin <s.martin49 at gmail.com>
>> changes v1 -> v2:
>> - remove trailing '\'
>> docs/manual/manual.mk | 24 ++++++++++++++++--------
>> 1 file changed, 16 insertions(+), 8 deletions(-)
>> diff --git a/docs/manual/manual.mk b/docs/manual/manual.mk
>> index 136f725..044557d 100644
>> --- a/docs/manual/manual.mk
>> +++ b/docs/manual/manual.mk
>> @@ -1,11 +1,20 @@
>> + $(Q)mkdir -p $@
>> +manual-rsync: $(BUILD_DIR)/$(pkgname)
>> + $(Q)$(call MESSAGE,"Preparing the manual sources...")
>> + $(Q)rsync -au docs/$(pkgname)/ $(BUILD_DIR)/$(pkgname)
> Why are you using 'rsync -au' and not 'rsync -a'?
> -u means: "-u, --update : skip files that are newer on the receiver"
> and I can't see why we would want that behavior?
> I realize that this was already present in the original code, but
> still I wonder if it's needed.
As you noticed, I just keep the original code. But that's true, we
certainly don't want the '-u' behavior.
I'll change this in the next submission (+ all the typoes pointed out
More information about the buildroot