On Mon, Aug 05, 2024 at 07:03:08PM +0200, Bruno Haible wrote: > Patrice Dumas wrote: > Find attached two patches, to generate these two distributed files in the > source directory. Tested.
Thanks for your continued testing and feedback. I've committed Patrice's fix and further fixes including one of those two patches (as df28d148ab26612). (I realised too late that I missed the commit summary line but it should be clear from the rest of the commit message what the commit is for.) In the other commit I made (a78e787fe3a) it was a simpler fix than the one you sent - hopefully it doesn't make a difference. (I want to keep it as simple and comprehensible as possible so there is not so much for someone to make sense of in the future when fixing any other problems - personally, my understanding of these issues and willingness to fix them is going down over time.) > Additionally, there is one more issue: > > 2) Texinfo/ModulePath.pm is no longer found in a particular situation > (not sure whether this situation is supported). > > How to reproduce: > > $ ./configure > $ make maintainer-clean > $ ./configure > $ make distcheck > ... > make distdir-am > make[4]: Entering directory '/TEXINFO/texinfo/doc' > Updating ./version.texi > Updating ./version-stnd.texi > Updating ./version-texi2any_api.texi > TEXINFO_DEV_SOURCE=1 top_srcdir=".." top_builddir=".." /usr/bin/perl -I > ../Pod-Simple-Texinfo/lib/ ../Pod-Simple-Texinfo/pod2texi.pl > --base-level=subsubsection --no-section-nodes --headings-as-sections > --subdir=. /TEXINFO/texinfo/Pod-Simple-Texinfo/pod2texi.pl > /dev/null > Can't locate Texinfo/ModulePath.pm in @INC (you may need to install the > Texinfo::ModulePath module) (@INC contains: ../tp ../Pod-Simple-Texinfo/lib/ > /etc/perl /usr/local/lib/x86_64-linux-gnu/perl/5.34.0 > /usr/local/share/perl/5.34.0 /usr/lib/x86_64-linux-gnu/perl5/5.34 > /usr/share/perl5 /usr/lib/x86_64-linux-gnu/perl-base > /usr/lib/x86_64-linux-gnu/perl/5.34 /usr/share/perl/5.34 > /usr/local/lib/site_perl) at ../Pod-Simple-Texinfo/pod2texi.pl line 75. > BEGIN failed--compilation aborted at ../Pod-Simple-Texinfo/pod2texi.pl line > 112. This failure is for building pod2texi.texi. It requires running the pod2texi program, but this is not ready to run yet. It is probably easy enough to make work by adding more Makefile rules but I wonder how many similar problems could exist. One idea is to run pod2texi instead of pod2texi.pl, setting TEXINFO_DEV_SOURCE. There would be a rule for regenerating pod2texi by running "make" in "Pod-Simple-Texinfo". ModulePath.pm would be a dependency of pod2texi. This would avoid having to add rules to regenerate ModulePath.pm in multiple places, which is less obvious than a rule to generate pod2texi. I can probably try to implement this in the next day or two.
