Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.debian.bugs.dist > #1125936 > unrolled thread

Bug#989029: info: scroll-backward is buggy

Started byHilmar Preuße <hille42@web.de>
First post2022-11-22 10:40 +0100
Last post2022-11-22 15:40 +0100
Articles 6 — 2 participants

Back to article view | Back to linux.debian.bugs.dist

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Bug#989029: info: scroll-backward is buggy Hilmar Preuße <hille42@web.de> - 2022-11-22 10:40 +0100
    Bug#989029: info: scroll-backward is buggy Vincent Lefevre <vincent@vinc17.net> - 2022-11-22 13:20 +0100
      Bug#989029: info: scroll-backward is buggy Vincent Lefevre <vincent@vinc17.net> - 2022-11-22 13:40 +0100
        Bug#989029: info: scroll-backward is buggy Vincent Lefevre <vincent@vinc17.net> - 2022-11-22 15:00 +0100
          Bug#989029: info: scroll-backward is buggy Vincent Lefevre <vincent@vinc17.net> - 2022-11-22 15:20 +0100
            Bug#989029: info: scroll-backward is buggy Vincent Lefevre <vincent@vinc17.net> - 2022-11-22 15:40 +0100

#1125936 — Bug#989029: info: scroll-backward is buggy

FromHilmar Preuße <hille42@web.de>
Date2022-11-22 10:40 +0100
SubjectBug#989029: info: scroll-backward is buggy
Message-ID<FuxV0-5uRx-15@gated-at.bofh.it>
Control: reassign -1 gnuplot-doc

Am 24.05.2021 um 01:36 teilte Vincent Lefevre mit:

Hello,

as discussed in that bug the issue seems only to occur w/ gnuplot-doc so
the issue is specific to the gnuplot documentation. Please be so kind to
have a look at the issue.

Hilmar

> The info manual says:
>
> <DEL> ('scroll-backward')
> <PREVIOUS>
>       Shift the text in this window down.  The inverse of
>       'scroll-forward'.  If you are at the start of a node, <DEL> takes
>       you to the "previous" node, so that you can read an entire manual
>       from finish to start by repeating <DEL>.  The default scroll size
>       can be changed by invoking the
>       ('scroll-backward-page-only-set-window') command with a numeric
>       argument.
> [...]
>
> But this doesn't behave like that in the gnuplot manual from the
> gnuplot-doc 5.4.1+dfsg1-1 Debian package: Go to "Bugs". This gives
>
> ------------------------------------------------------------
> 5 Bugs
> ******
>
> Please e-mail bug reports to the gnuplot-bugs mailing list or upload the
> report to the gnuplot web site on SourceForge.  Please give complete
> information on the version of gnuplot you are using and, if possible, a
> test script that demonstrates the bug.  See 'seeking-assistance'.
>
> * Menu:
>
> * known_limitations::
> * External_libraries::
> ------------------------------------------------------------
>
> Then do a scroll-backward (with the Backspace or PageUp key).
> This gives:
>
> ------------------------------------------------------------
> 5.2 External libraries
> ======================
> [...]
> ------------------------------------------------------------
>
> which is not the "previous" node, but some later node.
>
> -- System Information:
> Debian Release: 11.0
>    APT prefers unstable-debug
>    APT policy: (500, 'unstable-debug'), (500, 'testing-security'), (500, 'stable-updates'), (500, 'unstable'), (500, 'testing'), (500, 'stable'), (1, 'experimental')
> Architecture: amd64 (x86_64)
>
> Kernel: Linux 5.10.0-7-amd64 (SMP w/8 CPU threads)
> Kernel taint flags: TAINT_PROPRIETARY_MODULE, TAINT_OOT_MODULE, TAINT_UNSIGNED_MODULE
> Locale: LANG=POSIX, LC_CTYPE=C.UTF-8 (charmap=UTF-8), LANGUAGE not set
> Shell: /bin/sh linked to /bin/dash
> Init: systemd (via /run/systemd/system)
> LSM: AppArmor: enabled
>
> Versions of packages info depends on:
> ii  install-info  6.7.0.dfsg.2-6
> ii  libc6         2.31-12
> ii  libtinfo6     6.2+20201114-2
>
> info recommends no packages.
>
> info suggests no packages.
>
> -- no debconf information
>


--
sigfault

[toc] | [next] | [standalone]


#1125950

FromVincent Lefevre <vincent@vinc17.net>
Date2022-11-22 13:20 +0100
Message-ID<FuApP-5wF4-17@gated-at.bofh.it>
In reply to#1125936
Control: retitle -1 gnuplot-doc: the list of terminals is absent from the info manual, also breaking navigation
Control: severity -1 important
Control: tags -1 - wontfix

The wontfix tag had to be removed...

The main issue is actually that the list of terminals is absent
from the info manual (the HTML and PDF manuals seem OK), which
makes the info manual *very* incomplete (this part is about 20%
of the PDF manual) as there are many options specific to
terminals, which are thus not documented; and in

  4.1 complete list of terminals

this list has been replaced by "Bugs::" in the menu, which also
breaks navigation, as explained below.

Moreover, due to the absent list of terminals, the terminal index
is obviously empty.

I suspect a bug in doc2texi.el, which generates the gnuplot.texi
file, from which the info manual is built by texinfo. More
precisely, the inclusion of the .trm files is buggy.

On 2022-11-22 10:34:26 +0100, Hilmar Preuße wrote:
> Control: reassign -1 gnuplot-doc
> 
> Am 24.05.2021 um 01:36 teilte Vincent Lefevre mit:
> 
> Hello,
> 
> as discussed in that bug the issue seems only to occur w/ gnuplot-doc so
> the issue is specific to the gnuplot documentation. Please be so kind to
> have a look at the issue.
> 
> Hilmar
> 
> > The info manual says:
> > 
> > <DEL> ('scroll-backward')
> > <PREVIOUS>
> >       Shift the text in this window down.  The inverse of
> >       'scroll-forward'.  If you are at the start of a node, <DEL> takes
> >       you to the "previous" node, so that you can read an entire manual
> >       from finish to start by repeating <DEL>.  The default scroll size
> >       can be changed by invoking the
> >       ('scroll-backward-page-only-set-window') command with a numeric
> >       argument.
> > [...]
> > 
> > But this doesn't behave like that in the gnuplot manual from the
> > gnuplot-doc 5.4.1+dfsg1-1 Debian package: Go to "Bugs". This gives
> > 
> > ------------------------------------------------------------
> > 5 Bugs
> > ******
> > 
> > Please e-mail bug reports to the gnuplot-bugs mailing list or upload the
> > report to the gnuplot web site on SourceForge.  Please give complete
> > information on the version of gnuplot you are using and, if possible, a
> > test script that demonstrates the bug.  See 'seeking-assistance'.
> > 
> > * Menu:
> > 
> > * known_limitations::
> > * External_libraries::
> > ------------------------------------------------------------
> > 
> > Then do a scroll-backward (with the Backspace or PageUp key).
> > This gives:
> > 
> > ------------------------------------------------------------
> > 5.2 External libraries
> > ======================
> > [...]
> > ------------------------------------------------------------
> > 
> > which is not the "previous" node, but some later node.

The issue comes from the fact that

  4.1 complete list of terminals

has "Bugs::" in the menu, while the "Bugs" node is not a child of
this node ("Bugs" is Section 5, not Section 4.1.x). This is not
supported by "info".

-- 
Vincent Lefèvre <vincent@vinc17.net> - Web: <https://www.vinc17.net/>
100% accessible validated (X)HTML - Blog: <https://www.vinc17.net/blog/>
Work: CR INRIA - computer arithmetic / AriC project (LIP, ENS-Lyon)

[toc] | [prev] | [next] | [standalone]


#1125952

FromVincent Lefevre <vincent@vinc17.net>
Date2022-11-22 13:40 +0100
Message-ID<FuAJc-5wM0-9@gated-at.bofh.it>
In reply to#1125950
Control: found -1 5.4.1+dfsg1-1
Control: found -1 5.4.4+dfsg1-2
Control: forcemerge -1 989028

On 2022-11-22 13:12:00 +0100, Vincent Lefevre wrote:
> The main issue is actually that the list of terminals is absent
> from the info manual (the HTML and PDF manuals seem OK), which
> makes the info manual *very* incomplete (this part is about 20%
> of the PDF manual) as there are many options specific to
> terminals, which are thus not documented; and in
[...]

and this was bug 989028, which I had reported just before this one.

-- 
Vincent Lefèvre <vincent@vinc17.net> - Web: <https://www.vinc17.net/>
100% accessible validated (X)HTML - Blog: <https://www.vinc17.net/blog/>
Work: CR INRIA - computer arithmetic / AriC project (LIP, ENS-Lyon)

[toc] | [prev] | [next] | [standalone]


#1125966

FromVincent Lefevre <vincent@vinc17.net>
Date2022-11-22 15:00 +0100
Message-ID<FuBYB-5xtA-11@gated-at.bofh.it>
In reply to#1125952
Control: tags -1 - upstream

Removed the upstream tag as I can't reproduce the issue by compiling
upstream's gnuplot 5.4.5, and Makefile.am and doc2texi.el in docs have
not changed (and the 5.4.5 release notes don't mention any fix about
the info manual).

In the gnuplot.info generated using upstream's tarball:

[...]
   Terminals marked 'legacy' are not built by default in recent gnuplot
versions and may not actually work.  @c <3 - all terminal stuff is
pulled from the .trm files

* Menu:

* aifm::
* aqua::
* be::
* epscairo::
[...]

In the gnuplot.info generated using the Debian source package:

[...]
   Terminals marked 'legacy' are not built by default in recent gnuplot
versions and may not actually work.  @c <3 - all terminal stuff is
pulled from the .trm files

* Menu:

* Bugs::

[...]

-- 
Vincent Lefèvre <vincent@vinc17.net> - Web: <https://www.vinc17.net/>
100% accessible validated (X)HTML - Blog: <https://www.vinc17.net/blog/>
Work: CR INRIA - computer arithmetic / AriC project (LIP, ENS-Lyon)

[toc] | [prev] | [next] | [standalone]


#1125969

FromVincent Lefevre <vincent@vinc17.net>
Date2022-11-22 15:20 +0100
Message-ID<FuChX-5xP8-7@gated-at.bofh.it>
In reply to#1125966

[Multipart message — attachments visible in raw view] — view raw

Control: tags -1 upstream patch

On 2022-11-22 14:46:28 +0100, Vincent Lefevre wrote:
> Removed the upstream tag as I can't reproduce the issue by compiling
> upstream's gnuplot 5.4.5, and Makefile.am and doc2texi.el in docs have
> not changed (and the 5.4.5 release notes don't mention any fix about
> the info manual).

Actually, it *is* an upstream bug. But I can reproduce it only with
a separate build directory (this is what Debian does). I suppose
that the path to the .trm files is incorrect in such a case.

doc2texi.el has

(defvar d2t-terminal-directory (expand-file-name "../term/")
  "Location of .trm files in gnuplot source tree.")

If I understand correctly, this is relative to the build directory
(since the current working directory is there), while this should have
been relative to the source directory. A general fix is not obvious:
one would have to pass $(srcdir) to the file, unless there is some
Emacs variable that already contains the information. However, for
Debian, the build directory is at some fixed place in the source tree,
so that this can be fixed by just adapting the relative path.

Patch attached.

-- 
Vincent Lefèvre <vincent@vinc17.net> - Web: <https://www.vinc17.net/>
100% accessible validated (X)HTML - Blog: <https://www.vinc17.net/blog/>
Work: CR INRIA - computer arithmetic / AriC project (LIP, ENS-Lyon)

[toc] | [prev] | [next] | [standalone]


#1125971

FromVincent Lefevre <vincent@vinc17.net>
Date2022-11-22 15:40 +0100
Message-ID<FuCBj-5xWl-21@gated-at.bofh.it>
In reply to#1125969
Control: forwarded -1 https://sourceforge.net/p/gnuplot/bugs/2568/

I've just reported a bug upstream ("with a separate build directory,
the info manual is not built correctly").

-- 
Vincent Lefèvre <vincent@vinc17.net> - Web: <https://www.vinc17.net/>
100% accessible validated (X)HTML - Blog: <https://www.vinc17.net/blog/>
Work: CR INRIA - computer arithmetic / AriC project (LIP, ENS-Lyon)

[toc] | [prev] | [standalone]


Back to top | Article view | linux.debian.bugs.dist


csiph-web