bug-texinfo
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Texinfo 7.1.0.91 pretest results


From: Patrice Dumas
Subject: Re: Texinfo 7.1.0.91 pretest results
Date: Wed, 18 Sep 2024 23:46:47 +0200

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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]