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
> 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.
> 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.
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)
> 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
> 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
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
> 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
> 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