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


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

Bug#989029: info: scroll-backward is buggy

Started byVincent Lefevre <vincent@vinc17.net>
First post2022-12-29 01:00 +0100
Last post2022-12-29 03:40 +0100
Articles 3 — 1 participant

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 Vincent Lefevre <vincent@vinc17.net> - 2022-12-29 01:00 +0100
    Bug#989029: info: scroll-backward is buggy Vincent Lefevre <vincent@vinc17.net> - 2022-12-29 02:50 +0100
      Bug#989029: info: scroll-backward is buggy Vincent Lefevre <vincent@vinc17.net> - 2022-12-29 03:40 +0100

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

FromVincent Lefevre <vincent@vinc17.net>
Date2022-12-29 01:00 +0100
SubjectBug#989029: info: scroll-backward is buggy
Message-ID<FHOuZ-dYtm-13@gated-at.bofh.it>
Control: unmerge -1
Control: retitle -1 gnuplot-doc: spurious menu at the end of Section 4 breaks navigation

On 2022-11-22 15:08:10 +0100, Vincent Lefevre wrote:
> 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.

This solves the inclusion of the terminal types, but unfortunately,
the buggy menu is a separate problem. In the gnuplot.info.gz file
with the patch:

[...]
4.1.54 xlib
-----------
The 'xlib' terminal driver supports the X11 Windows System.  It
generates gnuplot_x11 commands, but sends them to the output file
specified by 'set output '<filename>''.  'set term x11' is equivalent to
'set output "|gnuplot_x11 -noevents"; set term xlib'.  'xlib' takes the
same set of options as 'x11'."
* Menu:
* Bugs::
File: gnuplot.info,  Node: Bugs,  Next: Concept_Index,  Prev: Terminal_types,  Up: Top
5 Bugs
******
[...]

i.e. the menu is still there, so that going backward from Section 5
goes to Section 5.2 instead of 4.1.54.

-- 
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] | [next] | [standalone]


#1130606

FromVincent Lefevre <vincent@vinc17.net>
Date2022-12-29 02:50 +0100
Message-ID<FHQdr-dZBI-1@gated-at.bofh.it>
In reply to#1130574

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

Control: retitle -1 gnuplot-doc: spurious menu at the end of Section 4 of gnuplot.info breaks navigation in info
Control: notforwarded -1

The forward was for the missing terminal types. I have not reported
this second bug upstream yet.

On 2022-12-29 00:51:19 +0100, Vincent Lefevre wrote:
[...]
> This solves the inclusion of the terminal types, but unfortunately,
> the buggy menu is a separate problem. In the gnuplot.info.gz file
> with the patch:
> 
> [...]
> 4.1.54 xlib
> -----------
> The 'xlib' terminal driver supports the X11 Windows System.  It
> generates gnuplot_x11 commands, but sends them to the output file
> specified by 'set output '<filename>''.  'set term x11' is equivalent to
> 'set output "|gnuplot_x11 -noevents"; set term xlib'.  'xlib' takes the
> same set of options as 'x11'."
> * Menu:
> * Bugs::
> File: gnuplot.info,  Node: Bugs,  Next: Concept_Index,  Prev: Terminal_types,  Up: Top
> 5 Bugs
> ******
> [...]
> 
> i.e. the menu is still there, so that going backward from Section 5
> goes to Section 5.2 instead of 4.1.54.

I've attached a patch for this issue: this incorrect menu is added
by 3 lines in doc2texi.el, with the comment

  ; `texinfo-all-menus-update' seems
  ; to miss this one.  how odd.

Perhaps this was added as a workaround to an old bug. With my
patch that removes these 3 lines, the navigation is now correct
(going backward from Section 5 "Bugs" sends to the end of
Section 4 as expected), and I cannot see any new issue.

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


#1130610

FromVincent Lefevre <vincent@vinc17.net>
Date2022-12-29 03:40 +0100
Message-ID<FHQZP-e070-1@gated-at.bofh.it>
In reply to#1130606
Control: forwarded -1 https://sourceforge.net/p/gnuplot/bugs/2575/

On 2022-12-29 02:45:35 +0100, Vincent Lefevre wrote:
> The forward was for the missing terminal types. I have not reported
> this second bug upstream yet.

I've now reported the bug upstream.

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