Re: Texinfo 6.7 released

2019-09-28 Thread Raymond Toy
Thanks for the fix.  Everything works just fine now!

On Fri, Sep 27, 2019 at 4:54 AM Gavin Smith 
wrote:

> On Fri, Sep 27, 2019 at 7:09 AM Raymond Toy  wrote:
> > I can build texinfo from these files without any problems.
> >
> > However, if I use the git repo I can't.  I checked out the tag
> texinfo-6.7 and ran autogen.sh and get this error:
> >
> > Preparing Texinfo development infrastructure:
> >   ./tp/maintain/regenerate_file_lists.pl
> >   (cd tp && ./maintain/regenerate_docstr.sh Makefile.docstr)
> >   (cd tp/tests && ../maintain/regenerate_cmd_tests.sh Makefile.onetst .
> -base 'formatting sectioning indices nested_formats contents layout'
> -tex_html 'tex_html')
> >   aclocal -I gnulib/m4 && autoconf && autoheader && automake
> > missing file gnulib/lib/windows-mutex.c
> > configure.ac:90: error: expected source file, required through
> AC_LIBSOURCES, not found
> > gnulib/m4/gnulib-comp.m4:156: gl_INIT is expanded from...
> > configure.ac:90: the top level
> > autom4te: /usr/bin/m4 failed with exit status: 1
> > aclocal: error: echo failed with exit status: 1
>
>
> Thank for the report. What I have done is committed the files and
> moved the texinfo-6.7 tag. If you delete the local copy of the tag and
> check it out again, it should work now.
>


-- 
Ray


Re: Texinfo 6.7 released

2019-09-28 Thread Eli Zaretskii
> Date: Sat, 28 Sep 2019 13:53:35 +0300
> From: Eli Zaretskii 
> Cc: bug-texinfo@gnu.org
> 
> I will run the test suite now.

All the tests passed, with some skipped (as expected).

Thanks.



Re: Texinfo 6.7 released

2019-09-28 Thread Eli Zaretskii
> From: Gavin Smith 
> Date: Sat, 28 Sep 2019 10:56:07 +0100
> Cc: Texinfo 
> 
> On Sat, Sep 28, 2019 at 9:04 AM Eli Zaretskii  wrote:
> > For the record, the version of Perl I used to build the pretest is
> > 5.22.1.  But I'm not sure it is relevant to the issue at hand, see my
> > other message.  Maybe it was just sheer luck that Perl didn't crash
> > for me while building the pretest, or maybe the Windows port of Perl
> > was changed in later versions so as not to bump into this problem when
> > extensions allocate storage.
> 
> Another possibility is that you had TEXINFO_XS_PARSER set in the
> environment which caused the Parsetexi module to be loaded.

Does that setting disable loading the other modules, where the problem
happened?

I do have that setting in my ~/.bash_profile file on the system where
I complied 6.7, and where Perl crashed.  Does the build disable that
in some way?

I will see if I have any setting of TEXINFO_XS_PARSER on the system
where I built the pretest.



Re: Texinfo 6.7 released

2019-09-28 Thread Eli Zaretskii
> From: Gavin Smith 
> Date: Sat, 28 Sep 2019 11:20:29 +0100
> Cc: Texinfo 
> 
> > I needed to add dTHX to 'init', or it wouldn't compile.
> 
> Didn't it compile before?

It did, because there was no stderr in that function.  Once I removed
the printf's, the dTHX is no longer needed in that function.

> > --- ./tp/Texinfo/XS/parsetexi/api.c.~1~ 2019-08-25 20:11:45.0 +0300
> > +++ ./tp/Texinfo/XS/parsetexi/api.c 2019-09-28 10:50:13.319625000 +0300
> > @@ -56,7 +56,8 @@ find_locales_dir (char *builddir)
> >
> >dTHX;
> >
> > -  asprintf (, "%s/LocaleData", builddir);
> > +  s = malloc (strlen (builddir) + strlen ("/LocaleData") + 1);
> > +  sprintf (s, "%s/LocaleData", builddir);
> >dir = opendir (s);
> >if (!dir)
> >  {
> >
> >
> > and the problem went away.
> 
> Would you like to commit that change?

Done, with a comment explaining why asprintf cannot be used there.

> I don't know of a good way to avoid using the Perl version of free.

I don't think there is one.  I think we will have to be vigilant and
avoid using Gnulib functions that allocate storage without freeing it,
in files that include Perl headers.

> > If this is the right fix, then I think we
> > cannot use Gnulib's asprintf in Parsetexi, and there are a couple of
> > other places with the same problem, which we need to fix as well,
> > right?
> 
> Hopefully not: only if Perl headers are included in a file should
> there be problems.

Ah, I see that the other grep hits were indeed in files that don't
include Perl headers.  So we are safe, I think.

> I deliberately limited where the Perl headers were included: for
> example, in tree_types.h a field of a struct is declared as a void
> pointer rather than the Perl type it should be. The files where there
> could be problems are api.c and Parsetexi.xs: it shouldn't be too hard
> to avoid using asprintf there (or other functions which use the real
> version of malloc).

Right.

I will run the test suite now.



Re: Texinfo 6.7 released

2019-09-28 Thread Gavin Smith
> Maybe that's what we should do here as well.  'free' is a macro
> expanded into something implemented inside Perl, right?  And asprintf
> comes from Gnulib on MS-Windows, right?  So we have that problem I
> mentioned with allocation and free that come from different libraries.

Yes, that makes sense.

> I needed to add dTHX to 'init', or it wouldn't compile.

Didn't it compile before?

> So it looks like it indeed crashes when calling 'free' on storage
> allocated by asprintf.  With that in mind, I modified api.c as below:
>
> --- ./tp/Texinfo/XS/parsetexi/api.c.~1~ 2019-08-25 20:11:45.0 +0300
> +++ ./tp/Texinfo/XS/parsetexi/api.c 2019-09-28 10:50:13.319625000 +0300
> @@ -56,7 +56,8 @@ find_locales_dir (char *builddir)
>
>dTHX;
>
> -  asprintf (, "%s/LocaleData", builddir);
> +  s = malloc (strlen (builddir) + strlen ("/LocaleData") + 1);
> +  sprintf (s, "%s/LocaleData", builddir);
>dir = opendir (s);
>if (!dir)
>  {
>
>
> and the problem went away.

Would you like to commit that change?

I don't know of a good way to avoid using the Perl version of free.
You could have "#undef free" followed by "#define free PerlMem_free",
but this would break if perl changes the way that it overrides "free".

> If this is the right fix, then I think we
> cannot use Gnulib's asprintf in Parsetexi, and there are a couple of
> other places with the same problem, which we need to fix as well,
> right?

Hopefully not: only if Perl headers are included in a file should
there be problems. I believe you successfully built and used the
Parsetexi module before on MS-Windows when asprintf was used in other
files.

I deliberately limited where the Perl headers were included: for
example, in tree_types.h a field of a struct is declared as a void
pointer rather than the Perl type it should be. The files where there
could be problems are api.c and Parsetexi.xs: it shouldn't be too hard
to avoid using asprintf there (or other functions which use the real
version of malloc).



Re: Texinfo 6.7 released

2019-09-28 Thread Gavin Smith
On Sat, Sep 28, 2019 at 9:04 AM Eli Zaretskii  wrote:
> For the record, the version of Perl I used to build the pretest is
> 5.22.1.  But I'm not sure it is relevant to the issue at hand, see my
> other message.  Maybe it was just sheer luck that Perl didn't crash
> for me while building the pretest, or maybe the Windows port of Perl
> was changed in later versions so as not to bump into this problem when
> extensions allocate storage.

Another possibility is that you had TEXINFO_XS_PARSER set in the
environment which caused the Parsetexi module to be loaded.



Re: Texinfo 6.7 released

2019-09-28 Thread Eli Zaretskii
> From: Gavin Smith 
> Date: Fri, 27 Sep 2019 20:25:38 +0100
> Cc: Texinfo 
> 
> > I cannot check right now, but I think the version of Perl on the
> > system where I built the pretest was indeed above 5.21.5.
> >
> > What is the conditional code above about?  Why is it needed?
> 
> I don't know: that code is automatically generated by xsubpp from 
> Parsetexi.xs.

For the record, the version of Perl I used to build the pretest is
5.22.1.  But I'm not sure it is relevant to the issue at hand, see my
other message.  Maybe it was just sheer luck that Perl didn't crash
for me while building the pretest, or maybe the Windows port of Perl
was changed in later versions so as not to bump into this problem when
extensions allocate storage.



Re: Texinfo 6.7 released

2019-09-28 Thread Eli Zaretskii
> From: Gavin Smith 
> Date: Fri, 27 Sep 2019 20:23:33 +0100
> Cc: Texinfo 
> 
> This problem occurred before and at that time, we solved it by
> avoiding using "free" altogether (commit d9fd777).

Maybe that's what we should do here as well.  'free' is a macro
expanded into something implemented inside Perl, right?  And asprintf
comes from Gnulib on MS-Windows, right?  So we have that problem I
mentioned with allocation and free that come from different libraries.

> > The output is below.  Does it help?
> 
> It shows it is likely the "init" function in parsetexi/api.c where the
> problem occurs. Maybe you could try a change like the one attached and
> see exactly where the problem is occurring.

I needed to add dTHX to 'init', or it wouldn't compile.  After that,
the output is:

  rm -rf $backupdir; exit $rc
  A 1
  A 2
  A 3
  B 1
  B 2
  B 3
  B 7
  B 8
  Free to wrong pool 3f5d48 not 40f018e at ..\tp/Texinfo/XSLoader.pm line 224, 
<$fh> line 8.

So it looks like it indeed crashes when calling 'free' on storage
allocated by asprintf.  With that in mind, I modified api.c as below:

--- ./tp/Texinfo/XS/parsetexi/api.c.~1~ 2019-08-25 20:11:45.0 +0300
+++ ./tp/Texinfo/XS/parsetexi/api.c 2019-09-28 10:50:13.319625000 +0300
@@ -56,7 +56,8 @@ find_locales_dir (char *builddir)
 
   dTHX;
 
-  asprintf (, "%s/LocaleData", builddir);
+  s = malloc (strlen (builddir) + strlen ("/LocaleData") + 1);
+  sprintf (s, "%s/LocaleData", builddir);
   dir = opendir (s);
   if (!dir)
 {


and the problem went away.  If this is the right fix, then I think we
cannot use Gnulib's asprintf in Parsetexi, and there are a couple of
other places with the same problem, which we need to fix as well,
right?

Thanks.



Re: Texinfo 6.7 released

2019-09-27 Thread Gavin Smith
On Fri, Sep 27, 2019 at 8:21 PM Eli Zaretskii  wrote:
> I see this in boot_Texinfo__Parser:
>
>   XS_EXTERNAL(boot_Texinfo__Parser); /* prototype to pass 
> -Wmissing-prototypes */
>   XS_EXTERNAL(boot_Texinfo__Parser)
>   {
>   #if PERL_VERSION_LE(5, 21, 5)
>   dVAR; dXSARGS;
>   #else
>   dVAR; dXSBOOTARGSXSAPIVERCHK;
>   #endif
>   [...]
>   #if PERL_VERSION_LE(5, 21, 5)
>   XS_VERSION_BOOTCHECK;
>   #  ifdef XS_APIVERSION_BOOTCHECK
>   XS_APIVERSION_BOOTCHECK;
>   #  endif
>   #endif
>   [...]
>   #if PERL_VERSION_LE(5, 21, 5)
>   #  if PERL_VERSION_GE(5, 9, 0)
>   if (PL_unitcheckav)
>   call_list(PL_scopestack_ix, PL_unitcheckav);
>   #  endif
>   XSRETURN_YES;
>   #else
>   Perl_xs_boot_epilog(aTHX_ ax);
>   #endif
>
> I cannot check right now, but I think the version of Perl on the
> system where I built the pretest was indeed above 5.21.5.
>
> What is the conditional code above about?  Why is it needed?

I don't know: that code is automatically generated by xsubpp from Parsetexi.xs.



Re: Texinfo 6.7 released

2019-09-27 Thread Gavin Smith
This problem occurred before and at that time, we solved it by
avoiding using "free" altogether (commit d9fd777).

On Fri, Sep 27, 2019 at 8:05 PM Eli Zaretskii  wrote:
>
> > From: Gavin Smith 
> > Date: Fri, 27 Sep 2019 19:52:02 +0100
> > Cc: Texinfo 
> >
> > Maybe you could try setting TEXINFO_XS=debug to see which module is
> > being loaded when the error occurs.
>
> The output is below.  Does it help?

It shows it is likely the "init" function in parsetexi/api.c where the
problem occurs. Maybe you could try a change like the one attached and
see exactly where the problem is occurring.


debug-patch
Description: Binary data


Re: Texinfo 6.7 released

2019-09-27 Thread Eli Zaretskii
> Date: Fri, 27 Sep 2019 22:04:59 +0300
> From: Eli Zaretskii 
> Cc: bug-texinfo@gnu.org
> 
>   ..\tp\Texinfo\XS/.libs/Parsetexi.dll loaded
>   looking for boot_Texinfo__Parser
>   trying to call boot_Texinfo__Parser...
> ...succeeded
>   Free to wrong pool 3f5f20 not 40f01ec at ..\tp/Texinfo/XSLoader.pm line 
> 224, <$fh> line 8.

I see this in boot_Texinfo__Parser:

  XS_EXTERNAL(boot_Texinfo__Parser); /* prototype to pass -Wmissing-prototypes 
*/
  XS_EXTERNAL(boot_Texinfo__Parser)
  {
  #if PERL_VERSION_LE(5, 21, 5)
  dVAR; dXSARGS;
  #else
  dVAR; dXSBOOTARGSXSAPIVERCHK;
  #endif
  [...]
  #if PERL_VERSION_LE(5, 21, 5)
  XS_VERSION_BOOTCHECK;
  #  ifdef XS_APIVERSION_BOOTCHECK
  XS_APIVERSION_BOOTCHECK;
  #  endif
  #endif
  [...]
  #if PERL_VERSION_LE(5, 21, 5)
  #  if PERL_VERSION_GE(5, 9, 0)
  if (PL_unitcheckav)
  call_list(PL_scopestack_ix, PL_unitcheckav);
  #  endif
  XSRETURN_YES;
  #else
  Perl_xs_boot_epilog(aTHX_ ax);
  #endif

I cannot check right now, but I think the version of Perl on the
system where I built the pretest was indeed above 5.21.5.

What is the conditional code above about?  Why is it needed?



Re: Texinfo 6.7 released

2019-09-27 Thread Eli Zaretskii
> From: Gavin Smith 
> Date: Fri, 27 Sep 2019 19:52:02 +0100
> Cc: Texinfo 
> 
> Maybe you could try setting TEXINFO_XS=debug to see which module is
> being loaded when the error occurs.

The output is below.  Does it help?

> > This is a different system from the one where I built the pretest.
> > Here I have Perl v5.20.1, provided by ActiveState.  Could this be
> > related to this particular version of Perl?  Though I never had any
> > problems with this Perl version with previous Texinfo releases.
> 
> There is something in xspara.c in xspara_init for Perl versions >=
> 5.27.8 for "thread-safe locale handling", but I don't know if that is
> related.

At the first glance, doesn't look related to me, although I don't
really understand what this is about.  In any case, the 'free' call
there frees memory allocated with malloc in the same function, so it
doesn't look suspicious to me.

  if TEXINFO_DEV_SOURCE=1 ; export TEXINFO_DEV_SOURCE ;  top_srcdir=".." ; 
export top_srcdir ;  top_builddir=".." ; export top_builddir ; 
/d/usr/Perl/bin/perl ../tp/texi2any   -I . \
   -o texinfo.info `test -f 'texinfo.texi' || echo './'`texinfo.texi; \
  then \
rc=0; \
  else \
rc=$?; \
$restore $backupdir/* `echo "./texinfo.info" | sed 's|[^/]*$||'`; \
  fi; \
  rm -rf $backupdir; exit $rc
  checking ..\tp\maintain\lib\Text-Unidecode\lib/MiscXS.la
  checking ..\tp\maintain\lib\Unicode-EastAsianWidth\lib/MiscXS.la
  checking ..\tp\maintain\lib\libintl-perl\lib/MiscXS.la
  checking ..\tp\maintain/MiscXS.la
  checking ..\tp\Texinfo\XS\parsetexi/MiscXS.la
  checking ..\tp\Texinfo\XS/MiscXS.la
  found ..\tp\Texinfo\XS/MiscXS.la
  ..\tp\Texinfo\XS/.libs/MiscXS.dll loaded
  looking for boot_Texinfo__MiscXSXS
  trying to call boot_Texinfo__MiscXSXS...
...succeeded
  attempting to override Texinfo::Convert::Unicode::unicode_text with 
Texinfo::MiscXS::unicode_text...
...succeeded
  checking ..\tp\maintain\lib\Text-Unidecode\lib/Parsetexi.la
  checking ..\tp\maintain\lib\Unicode-EastAsianWidth\lib/Parsetexi.la
  checking ..\tp\maintain\lib\libintl-perl\lib/Parsetexi.la
  checking ..\tp\maintain/Parsetexi.la
  checking ..\tp\Texinfo\XS\parsetexi/Parsetexi.la
  checking ..\tp\Texinfo\XS/Parsetexi.la
  found ..\tp\Texinfo\XS/Parsetexi.la
  ..\tp\Texinfo\XS/.libs/Parsetexi.dll loaded
  looking for boot_Texinfo__Parser
  trying to call boot_Texinfo__Parser...
...succeeded
  Free to wrong pool 3f5f20 not 40f01ec at ..\tp/Texinfo/XSLoader.pm line 224, 
<$fh> line 8.



Re: Texinfo 6.7 released

2019-09-27 Thread Gavin Smith
On Fri, Sep 27, 2019 at 1:58 PM Eli Zaretskii  wrote:
> Did you make any changes since the last pretest?

Nothing that would have introduced this error.

> Trying to build this on MS-Windows with MinGW, Perl crashes when
> generating texinfo.info:
>
>   if TEXINFO_DEV_SOURCE=1 ; export TEXINFO_DEV_SOURCE ;  top_srcdir=".." ; 
> export top_srcdir ;  top_builddir=".." ; export top_builddir ; 
> /d/usr/Perl/bin/perl ../tp/texi2any   -I . \
>-o texinfo.info `test -f 'texinfo.texi' || echo './'`texinfo.texi; \
>   then \
> rc=0; \
>   else \
> rc=$?; \
> $restore $backupdir/* `echo "./texinfo.info" | sed 's|[^/]*$||'`; \
>   fi; \
>   rm -rf $backupdir; exit $rc
>   Free to wrong pool 3f5d48 not 40f015b at ..\tp/Texinfo/XSLoader.pm line 
> 224, <$fh> line 8.
>   Makefile:1258: recipe for target `texinfo.info' failed

Maybe you could try setting TEXINFO_XS=debug to see which module is
being loaded when the error occurs.

> This is a different system from the one where I built the pretest.
> Here I have Perl v5.20.1, provided by ActiveState.  Could this be
> related to this particular version of Perl?  Though I never had any
> problems with this Perl version with previous Texinfo releases.

There is something in xspara.c in xspara_init for Perl versions >=
5.27.8 for "thread-safe locale handling", but I don't know if that is
related.



Re: Texinfo 6.7 released

2019-09-27 Thread Eli Zaretskii
> Date: Fri, 27 Sep 2019 15:08:23 +0300
> From: Eli Zaretskii 
> Cc: bug-texinfo@gnu.org
> 
>   Free to wrong pool 3f5d48 not 40f015b at ..\tp/Texinfo/XSLoader.pm line 
> 224, <$fh> line 8.

FWIW, that line 224 of XSLoader.pm is the only line where there has
been a code change since Texinfo 6.6, see below.

Could it be that this change causes a string allocated by Perl to be
freed by a call to 'free' in the extension DLL?  On Windows, we must
not do that, as Perl could have been built with a different version of
malloc.

--- ../texinfo-6.6/tp/Texinfo/XSLoader.pm   2019-02-16 19:11:56.0 
+0200
+++ ./tp/Texinfo/XSLoader.pm2019-09-23 21:31:55.0 +0300
@@ -1,4 +1,4 @@
-# Copyright 2014, 2015, 2016, 2017, 2018 Free Software Foundation, Inc.
+# Copyright 2014-2019 Free Software Foundation, Inc.
 #
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -23,7 +23,7 @@
 
 our $TEXINFO_XS;
 
-our $VERSION = '6.6';
+our $VERSION = '6.7';
 
 our $disable_XS;
 
@@ -221,7 +221,9 @@
   # This makes it easier to refer to packages and symbols by name.
   no strict 'refs';
   
-  if (defined &{"${module}::init"} and !&{"${module}::init"} ()) {
+  if (defined &{"${module}::init"}
+  and !&{"${module}::init"} ($Texinfo::ModulePath::texinfo_uninstalled,
+ $Texinfo::ModulePath::builddir)) {
 _fatal "$module_name: error initializing";
 goto FALLBACK;
   }



Re: Texinfo 6.7 released

2019-09-27 Thread Gavin Smith
On Fri, Sep 27, 2019 at 7:09 AM Raymond Toy  wrote:
> I can build texinfo from these files without any problems.
>
> However, if I use the git repo I can't.  I checked out the tag texinfo-6.7 
> and ran autogen.sh and get this error:
>
> Preparing Texinfo development infrastructure:
>   ./tp/maintain/regenerate_file_lists.pl
>   (cd tp && ./maintain/regenerate_docstr.sh Makefile.docstr)
>   (cd tp/tests && ../maintain/regenerate_cmd_tests.sh Makefile.onetst . -base 
> 'formatting sectioning indices nested_formats contents layout' -tex_html 
> 'tex_html')
>   aclocal -I gnulib/m4 && autoconf && autoheader && automake
> missing file gnulib/lib/windows-mutex.c
> configure.ac:90: error: expected source file, required through AC_LIBSOURCES, 
> not found
> gnulib/m4/gnulib-comp.m4:156: gl_INIT is expanded from...
> configure.ac:90: the top level
> autom4te: /usr/bin/m4 failed with exit status: 1
> aclocal: error: echo failed with exit status: 1


Thank for the report. What I have done is committed the files and
moved the texinfo-6.7 tag. If you delete the local copy of the tag and
check it out again, it should work now.



Re: Texinfo 6.7 released

2019-09-27 Thread Eli Zaretskii
> From: Gavin Smith 
> Date: Mon, 23 Sep 2019 20:28:46 +0100
> 
> We have released version 6.7 of Texinfo, the GNU documentation format.
> 
> This package contains tools to produce documentation in various
> formats, including HTML and PDF, from source files in the Texinfo
> format.  Texinfo is a text-based format with commands for marking text,
> document structuring and indexing.
> 
> The highlight of this release is improved indexing facilities.
> 
>   http://ftpmirror.gnu.org/texinfo/texinfo-6.7.tar.xz
>   http://ftpmirror.gnu.org/texinfo/texinfo-6.7.tar.gz

Did you make any changes since the last pretest?

Trying to build this on MS-Windows with MinGW, Perl crashes when
generating texinfo.info:

  if TEXINFO_DEV_SOURCE=1 ; export TEXINFO_DEV_SOURCE ;  top_srcdir=".." ; 
export top_srcdir ;  top_builddir=".." ; export top_builddir ; 
/d/usr/Perl/bin/perl ../tp/texi2any   -I . \
   -o texinfo.info `test -f 'texinfo.texi' || echo './'`texinfo.texi; \
  then \
rc=0; \
  else \
rc=$?; \
$restore $backupdir/* `echo "./texinfo.info" | sed 's|[^/]*$||'`; \
  fi; \
  rm -rf $backupdir; exit $rc
  Free to wrong pool 3f5d48 not 40f015b at ..\tp/Texinfo/XSLoader.pm line 224, 
<$fh> line 8.
  Makefile:1258: recipe for target `texinfo.info' failed
  make[3]: *** [texinfo.info] Error 5
  make[3]: Leaving directory `/d/gnu/texinfo-6.7/doc'
  Makefile:1461: recipe for target `all-recursive' failed

>From searching the Internet, it sounds like the "Free to wrong pool"
error message is related to multithreading in Perl and/or fork()?

This is a different system from the one where I built the pretest.
Here I have Perl v5.20.1, provided by ActiveState.  Could this be
related to this particular version of Perl?  Though I never had any
problems with this Perl version with previous Texinfo releases.

TIA



Re: Texinfo 6.7 released

2019-09-27 Thread Raymond Toy
On Mon, Sep 23, 2019 at 12:29 PM Gavin Smith 
wrote:

> We have released version 6.7 of Texinfo, the GNU documentation format.
>
> This package contains tools to produce documentation in various
> formats, including HTML and PDF, from source files in the Texinfo
> format.  Texinfo is a text-based format with commands for marking text,
> document structuring and indexing.
>
> The highlight of this release is improved indexing facilities.
>
>   http://ftpmirror.gnu.org/texinfo/texinfo-6.7.tar.xz
>   http://ftpmirror.gnu.org/texinfo/texinfo-6.7.tar.gz


I can build texinfo from these files without any problems.

However, if I use the git repo I can't.  I checked out the tag texinfo-6.7
and ran autogen.sh and get this error:

Preparing Texinfo development infrastructure:
  ./tp/maintain/regenerate_file_lists.pl
  (cd tp && ./maintain/regenerate_docstr.sh Makefile.docstr)
  (cd tp/tests && ../maintain/regenerate_cmd_tests.sh Makefile.onetst .
-base 'formatting sectioning indices nested_formats contents layout'
-tex_html 'tex_html')
  aclocal -I gnulib/m4 && autoconf && autoheader && automake
missing file gnulib/lib/windows-mutex.c
configure.ac:90: error: expected source file, required through
AC_LIBSOURCES, not found
gnulib/m4/gnulib-comp.m4:156: gl_INIT is expanded from...
configure.ac:90: the top level
autom4te: /usr/bin/m4 failed with exit status: 1
aclocal: error: echo failed with exit status: 1

I used to be able to build from the git tree without problems.

>
> If automatic redirection fails, the list of mirrors is at:
>   https://www.gnu.org/prep/ftp.html
>
> Failing that, you can use the main server:
>   https://ftp.gnu.org/gnu/texinfo/texinfo-6.7.tar.xz
>   https://ftp.gnu.org/gnu/texinfo/texinfo-6.7.tar.gz
>
> Please email any comments to bug-texinfo@gnu.org.
> The Texinfo web page: https://www.gnu.org/software/texinfo/
>
> Full news:
> * Language:
>   . support of index subentries and sub-subentries with @subentry
>   . new commands @seeentry and @seealso in index entries
>   . no need to wrap Top node in @ifnottex - omitted automatically when
> processed with TeX
>   . UTF-8 is the default input encoding
>
> * texi2any
>   . for HTML output, mark index nodes in menus and tables of contents
> with the 'rel' attribute of the 'a' tag.
>   . TOP_NODE_UP is now only used in HTML if TOP_NODE_UP_URL is set.
> Also TOP_NODE_UP should now be formatted in the output format.
> In HTML TOP_NODE_UP should be suitable for inclusion in HTML
> element attributes, so for instance should not contain elements.
>   . support of noderename.cnf files has been removed
>   . INPUT_PERL_ENCODING, INPUT_ENCODING_NAME, NODE_FILE_EXTENSION,
> NODE_FILENAMES, SHORTEXTN and TOP_NODE_FILE removed as customization
> variables.
>   . TOP_NODE_FILE_TARGET now contains the extension.
>   . error messages translated when the XS parser module is in use
>
> * texi2dvi
>   . unconditionally run in --batch mode, i.e. without stopping if there
> is a TeX error
>   . keep on going after a TeX error if the index files changed
>   . with --tidy (or --build-dir), avoid reading index files from previous
> runs where --tidy was not used
>
> * info
>   . for a tree search (with M-/), '}' and '{' work as well as 'M-}' and
> 'M-{' to go through the results
>
> * Distribution:
>   . Several obsolete portability checks removed
>   . gettext 0.20.1, automake 1.16.1
>
>

-- 
Ray


Re: Texinfo 6.7 released

2019-09-24 Thread Andreas Schwab
On Sep 24 2019, Gavin Smith  wrote:

> On Tue, Sep 24, 2019 at 5:58 PM Andreas Schwab  wrote:
>>
>> On Sep 23 2019, Gavin Smith  wrote:
>>
>> >   https://ftp.gnu.org/gnu/texinfo/texinfo-6.7.tar.xz
>>
>> That file is still missing.  Was there any issue during upload?
>>
>> Andreas.
>
> I'm not sure; I've re-uploaded it.

Thanks, it appeared now.

Andreas.

-- 
Andreas Schwab, sch...@linux-m68k.org
GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510  2552 DF73 E780 A9DA AEC1
"And now for something completely different."



Re: Texinfo 6.7 released

2019-09-24 Thread Gavin Smith
On Tue, Sep 24, 2019 at 5:58 PM Andreas Schwab  wrote:
>
> On Sep 23 2019, Gavin Smith  wrote:
>
> >   https://ftp.gnu.org/gnu/texinfo/texinfo-6.7.tar.xz
>
> That file is still missing.  Was there any issue during upload?
>
> Andreas.

I'm not sure; I've re-uploaded it.



Re: Texinfo 6.7 released

2019-09-24 Thread Andreas Schwab
On Sep 23 2019, Gavin Smith  wrote:

>   https://ftp.gnu.org/gnu/texinfo/texinfo-6.7.tar.xz

That file is still missing.  Was there any issue during upload?

Andreas.

-- 
Andreas Schwab, sch...@linux-m68k.org
GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510  2552 DF73 E780 A9DA AEC1
"And now for something completely different."



Texinfo 6.7 released

2019-09-23 Thread Gavin Smith
We have released version 6.7 of Texinfo, the GNU documentation format.

This package contains tools to produce documentation in various
formats, including HTML and PDF, from source files in the Texinfo
format.  Texinfo is a text-based format with commands for marking text,
document structuring and indexing.

The highlight of this release is improved indexing facilities.

  http://ftpmirror.gnu.org/texinfo/texinfo-6.7.tar.xz
  http://ftpmirror.gnu.org/texinfo/texinfo-6.7.tar.gz

If automatic redirection fails, the list of mirrors is at:
  https://www.gnu.org/prep/ftp.html

Failing that, you can use the main server:
  https://ftp.gnu.org/gnu/texinfo/texinfo-6.7.tar.xz
  https://ftp.gnu.org/gnu/texinfo/texinfo-6.7.tar.gz

Please email any comments to bug-texinfo@gnu.org.
The Texinfo web page: https://www.gnu.org/software/texinfo/

Full news:
* Language:
  . support of index subentries and sub-subentries with @subentry
  . new commands @seeentry and @seealso in index entries
  . no need to wrap Top node in @ifnottex - omitted automatically when
processed with TeX
  . UTF-8 is the default input encoding

* texi2any
  . for HTML output, mark index nodes in menus and tables of contents
with the 'rel' attribute of the 'a' tag.
  . TOP_NODE_UP is now only used in HTML if TOP_NODE_UP_URL is set.
Also TOP_NODE_UP should now be formatted in the output format.
In HTML TOP_NODE_UP should be suitable for inclusion in HTML
element attributes, so for instance should not contain elements.
  . support of noderename.cnf files has been removed
  . INPUT_PERL_ENCODING, INPUT_ENCODING_NAME, NODE_FILE_EXTENSION,
NODE_FILENAMES, SHORTEXTN and TOP_NODE_FILE removed as customization
variables.
  . TOP_NODE_FILE_TARGET now contains the extension.
  . error messages translated when the XS parser module is in use

* texi2dvi
  . unconditionally run in --batch mode, i.e. without stopping if there
is a TeX error
  . keep on going after a TeX error if the index files changed
  . with --tidy (or --build-dir), avoid reading index files from previous
runs where --tidy was not used

* info
  . for a tree search (with M-/), '}' and '{' work as well as 'M-}' and
'M-{' to go through the results

* Distribution:
  . Several obsolete portability checks removed
  . gettext 0.20.1, automake 1.16.1