Re: [PATCH] Fix parallel build failure for Texinfo manual

[Marco Wahl]

Ulrich Mueller <ulm@gentoo.org> writes:

>>>>>> On Mon, 20 Dec 2021, Marco Wahl wrote:
>
>> Possibly a split of function org-make-manuals in org-make-manual and
>> org-make-guide and further create two single targets instead of the
>> current double target is more clear.
>
>> See the patch.
>
>> WDYT?
>
> Sure, that's much cleaner. (I didn't dare to change the elisp code
> because it is also more intrusive.)
>
>> -org.texi orgguide.texi:     org-manual.org org-guide.org
>> +org.texi:   org-manual.org org-guide.org
>
> Shouldn't it have only org-manual.org as prerequisite?
>
>>      $(BATCH)                                      \
>>        --eval '(add-to-list `load-path "../lisp")' \
>>        --eval '(load "../mk/org-fixup.el")'        \
>> -      --eval '(org-make-manuals)'
>> +      --eval '(org-make-manual)'
>> +
>> +orgguide.texi:      org-manual.org org-guide.org
>
> Similar here, only org-guide.org?
>
>> +    $(BATCH)                                      \
>> +      --eval '(add-to-list `load-path "../lisp")' \
>> +      --eval '(load "../mk/org-fixup.el")'        \
>> +      --eval '(org-make-guide)'

Yes, thanks.

The issue should be fixed with a slightly more defensive change than
stated in the last patch (by keeping function org-make-manuals.)


Thanks again,
-- 
Marco