On Mon, Sep 02, 2024 at 08:02:26PM +0200, Bruno Haible wrote:
> Hi Gavin,
>
> > Could it be an issue with backslashes being used as directory separators
> > in \cygdrive\d\a\ci-check\ci-check ... ? It is strange that the
> > path is passed with forward slashes on the command line in the environment
> > variables top_builddir but is given with backslashes in the error. Is
> > there any automatic conversion of backslashes on this platform that
> > could be responsible?
>
> This looks like it is the problem, yes.
>
> Texinfo/ModulePath.pm is meant to reside in build/tp/.
>
> The line texi2any.pl:84 is meant to add this directory to @INC.
I had a look at the build log and there is
if TEXINFO_DEV_SOURCE=1
top_srcdir="/cygdrive/d/a/ci-check/ci-check/texinfo-7.1.1-20240916/build/.."
top_builddir="/cygdrive/d/a/ci-check/ci-check/texinfo-7.1.1-20240916/build"
/cygdrive/c/Strawberry/perl/bin/perl ../../../tp/texi2any.pl -c
INFO_SPECIAL_CHARS_WARNING=0 -c TREE_TRANSFORMATIONS=complete_tree_nodes_menus
-I ../../../doc/tp_api \
-o texi2any_internals.info `test -f 'texi2any_internals.texi' || echo
'../../../doc/tp_api/'`texi2any_internals.texi; \
Therefore the top_builddir
is /cygdrive/d/a/ci-check/ci-check/texinfo-7.1.1-20240916/build
and is used l 81 of texi2any.pl:
unshift @INC, File::Spec->catdir($ENV{'top_builddir'}, 'tp');
> We can see that it the value that it adds is
> \cygdrive\d\a\ci-check\ci-check\texinfo-7.1.0.91-20240901\build\tp
> but since this is a native Windows perl, it should really be
> D:\a\ci-check\ci-check\texinfo-7.1.0.91-20240901\build\tp
> (with slashes or backslahes, doesn't matter).
I think that it comes from doc/tp_api/Makefile.am
# Use the programs built in our distribution. Use absolute directory
# names so we can run from a different working directory.
MAKEINFO_ENVIRONMENT = TEXINFO_DEV_SOURCE=1 \
top_srcdir="$(abs_top_srcdir)" \
top_builddir="$(abs_top_builddir)"
Then, I think that abs_top_builddir comes from configure substitution in
Makefile.in, maybe from pwd.
I do not really understand why abs_top_srcdir is used here and not
top_srcdir, I did not really understand what the comment is about.
Maybe using top_srcdir instead of abs_top_srcdir and top_builddir
instead of abs_top_builddir could fix that issue?
--
Pat
