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 > te

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.

Re: paragraph justification issue on @url with "texi2dvi --pdf" in texinfo.tex 2019-09-20.22

2019-09-28 Thread Gavin Smith
On Wed, Sep 25, 2019 at 3:03 PM Vincent Lefevre wrote: > This fixes some cases, but not others. I've attached a url-break.texi > file with 4 cases. And I've tested 4 different texinfo.tex solutions > (the old one, the new one, the new one with this patch, and another > one with just a \allowbreak)

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 functio

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 se

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 pretes

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: t

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 i