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

[Marco Wahl <marcowahlsoft@gmail.com>]

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