> Thanks for the report. I found it was due to the reassignment
> of \rightskip inside @multitable.
>
> I've attempted a fix in commit beb67a5f97219. I checked the output
> for some of the @multitable's in the Texinfo manuals but it would be
> good if you could check that the output looks fine
On Fri, Apr 08, 2022 at 10:50:19PM +0300, Eli Zaretskii wrote:
> > Date: Fri, 8 Apr 2022 15:53:31 +0200
> > From: Tomas Kalibera
> >
> > Would this be an acceptable fix? (turn OSTYPE to uppercase when setting
> > MSYSTEM)
> >
> > MSYSTEM=$(echo $OSTYPE | tr [a-z] [A-Z]) uname | $EGREP -iv
> From: Gavin Smith
> Date: Fri, 8 Apr 2022 15:12:19 +0100
> Cc: Eli Zaretskii , bug-texinfo@gnu.org
>
> On Fri, Apr 08, 2022 at 03:53:31PM +0200, Tomas Kalibera wrote:
> >
> > On 4/8/22 15:31, Gavin Smith wrote:
> > > On Fri, Apr 08, 2022 at 03:15:01PM +0200, Tomas Kalibera wrote:
> > > > > Wha
> Date: Fri, 8 Apr 2022 15:53:31 +0200
> From: Tomas Kalibera
>
> Would this be an acceptable fix? (turn OSTYPE to uppercase when setting
> MSYSTEM)
>
> MSYSTEM=$(echo $OSTYPE | tr [a-z] [A-Z]) uname | $EGREP -iv
> 'cygwin|msys' >/dev/null; then
>
> It was suggested by Markus Mützel in a
> From: Gavin Smith
> Date: Fri, 8 Apr 2022 14:31:32 +0100
> Cc: Eli Zaretskii , bug-texinfo@gnu.org
>
> > It works with Msys2 and cygwin for me, but breaks with MSYSTEM, where
>
> What is MSYSTEM? Is this the original msys?
>
> > $ uname
> > MINGW32_NT-6.2
> >
> > $ MSYSTEM=$OSTYPE uname
> >
> From: Gavin Smith
> Date: Fri, 8 Apr 2022 13:56:31 +0100
> Cc: Tomas Kalibera , bug-texinfo@gnu.org
>
> What happens if we don't set MSYSTEM at all?
Then you cannot distinguish between MSYS and MinGW, because 'uname'
will return a value that doesn't tell which one is it. So you have a
problem
On 4/8/22 18:30, Gavin Smith wrote:
On Fri, Apr 08, 2022 at 06:07:01PM +0200, Tomas Kalibera wrote:
Works for me on all those 3 systems (just needed to add \ at the end of the
line after "/dev/null "). And checking OSTYPE against "msys" is a common
pattern in Msys2 patches, so I assume it is c
On Fri, Apr 08, 2022 at 06:07:01PM +0200, Tomas Kalibera wrote:
> Works for me on all those 3 systems (just needed to add \ at the end of the
> line after "/dev/null "). And checking OSTYPE against "msys" is a common
> pattern in Msys2 patches, so I assume it is correct for Msys2.
Commited in df78
On 4/8/22 17:25, Gavin Smith wrote:
On Fri, Apr 08, 2022 at 04:30:12PM +0200, Tomas Kalibera wrote:
I am not sure what is the correct name of it. I executed
C:\MinGW\msys\1.0\msys.bat to get the shell.
It uses : as path separator.
Have you found that running texi2dvi fails in such an environ
On Fri, Apr 08, 2022 at 04:30:12PM +0200, Tomas Kalibera wrote:
> I am not sure what is the correct name of it. I executed
> C:\MinGW\msys\1.0\msys.bat to get the shell.
> It uses : as path separator.
>
> > Have you found that running texi2dvi fails in such an environment, with
> > my patch above?
On Fri, Apr 08, 2022 at 05:02:11PM +0200, Patrice Dumas wrote:
> > The ignoring starts at '@node Top' and continues until the next @node or
> > @part command.
>
> Then it is different from texi2any. In texi2any in the following
> @part part Intro and @chapter intro content is ignored:
>
> @node
On Fri, Apr 08, 2022 at 02:52:06PM +0100, Gavin Smith wrote:
> On Fri, Apr 08, 2022 at 03:36:59PM +0200, Patrice Dumas wrote:
> > On Fri, Apr 08, 2022 at 02:22:30PM +0100, Gavin Smith wrote:
> > > On Sun, Mar 20, 2022 at 11:47:41PM +0100, Patrice Dumas wrote:
> > > > On Sun, Mar 13, 2022 at 07:11:1
On 4/8/22 16:12, Gavin Smith wrote:
On Fri, Apr 08, 2022 at 03:53:31PM +0200, Tomas Kalibera wrote:
On 4/8/22 15:31, Gavin Smith wrote:
On Fri, Apr 08, 2022 at 03:15:01PM +0200, Tomas Kalibera wrote:
What happens if we don't set MSYSTEM at all?
diff --git a/util/texi2dvi b/util/texi2dvi
ind
On Fri, Apr 08, 2022 at 03:53:31PM +0200, Tomas Kalibera wrote:
>
> On 4/8/22 15:31, Gavin Smith wrote:
> > On Fri, Apr 08, 2022 at 03:15:01PM +0200, Tomas Kalibera wrote:
> > > > What happens if we don't set MSYSTEM at all?
> > > >
> > > > diff --git a/util/texi2dvi b/util/texi2dvi
> > > > index
On 4/8/22 15:31, Gavin Smith wrote:
On Fri, Apr 08, 2022 at 03:15:01PM +0200, Tomas Kalibera wrote:
What happens if we don't set MSYSTEM at all?
diff --git a/util/texi2dvi b/util/texi2dvi
index 1f42b41907..c506bbad37 100755
--- a/util/texi2dvi
+++ b/util/texi2dvi
@@ -85,7 +85,7 @@ IFS="$space
On Fri, Apr 08, 2022 at 03:36:59PM +0200, Patrice Dumas wrote:
> On Fri, Apr 08, 2022 at 02:22:30PM +0100, Gavin Smith wrote:
> > On Sun, Mar 20, 2022 at 11:47:41PM +0100, Patrice Dumas wrote:
> > > On Sun, Mar 13, 2022 at 07:11:11PM +, Gavin Smith wrote:
> > > >
> > > > Do you think you can d
On Fri, Apr 08, 2022 at 02:22:30PM +0100, Gavin Smith wrote:
> On Sun, Mar 20, 2022 at 11:47:41PM +0100, Patrice Dumas wrote:
> > On Sun, Mar 13, 2022 at 07:11:11PM +, Gavin Smith wrote:
> > >
> > > Do you think you can do something similar for texi2any, i.e. not include
> > > the @part line i
On Fri, Apr 08, 2022 at 03:15:01PM +0200, Tomas Kalibera wrote:
> > What happens if we don't set MSYSTEM at all?
> >
> > diff --git a/util/texi2dvi b/util/texi2dvi
> > index 1f42b41907..c506bbad37 100755
> > --- a/util/texi2dvi
> > +++ b/util/texi2dvi
> > @@ -85,7 +85,7 @@ IFS="$space$tab$newline"
On Sun, Mar 20, 2022 at 11:47:41PM +0100, Patrice Dumas wrote:
> On Sun, Mar 13, 2022 at 07:11:11PM +, Gavin Smith wrote:
> >
> > Do you think you can do something similar for texi2any, i.e. not include
> > the @part line in the preceding node?
>
> Should be done now. Note that this also mea
On 4/8/22 14:56, Gavin Smith wrote:
On Wed, Apr 06, 2022 at 05:24:01PM +0300, Eli Zaretskii wrote:
Date: Wed, 6 Apr 2022 14:39:30 +0200
From: Tomas Kalibera
texi2dvi on Msys2 doesn't find tex, because it incorrectly detects ";"
as path separator (it is ":").
The problem is in the following
On Wed, Apr 06, 2022 at 05:24:01PM +0300, Eli Zaretskii wrote:
> > Date: Wed, 6 Apr 2022 14:39:30 +0200
> > From: Tomas Kalibera
> >
> > texi2dvi on Msys2 doesn't find tex, because it incorrectly detects ";"
> > as path separator (it is ":").
> >
> > The problem is in the following code. With M
21 matches
Mail list logo
