* bug#58786: 29.0.50; Error with Info-minibuf-history not declared
@ 2022-10-26 3:48 Thierry Volpiatto
2022-10-26 5:36 ` Stefan Kangas
0 siblings, 1 reply; 11+ messages in thread
From: Thierry Volpiatto @ 2022-10-26 3:48 UTC (permalink / raw)
To: 58786
Info-minibuf-history is not declared hence the error when calling Info-goto-node:
(void-function Info-minibuf-history)
Evaling (defvar Info-minibuf-history nil) fixes it.
See https://github.com/emacs-helm/helm/issues/2563
Thanks.
In GNU Emacs 29.0.50 (build 1, x86_64-pc-linux-gnu, X toolkit, cairo
version 1.16.0, Xaw3d scroll bars) of 2022-10-24 built on IPad-S340
Repository revision: f7816c94b61f87919afccbedbea5270ca5db4e15
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12013000
System Description: Linux Mint 20.3
Configured using:
'configure --bindir=/usr/local/sbin/emacs-29.0.50 --with-mailutils
--with-cairo --without-dbus --without-gconf --without-gsettings
--with-x-toolkit=lucid'
Configured features:
ACL CAIRO FREETYPE GIF GLIB GMP GNUTLS GPM HARFBUZZ JPEG JSON LCMS2
LIBOTF LIBSELINUX LIBSYSTEMD LIBXML2 M17N_FLT MODULES NOTIFY INOTIFY
PDUMPER PNG RSVG SECCOMP SOUND SQLITE3 THREADS TIFF TOOLKIT_SCROLL_BARS
X11 XAW3D XDBE XIM XINPUT2 XPM LUCID ZLIB
Important settings:
value of $LANG: fr_FR.UTF-8
locale-coding-system: utf-8-unix
Major mode: ƐĽ
Minor modes in effect:
bug-reference-prog-mode: t
global-undo-tree-mode: t
undo-tree-mode: t
psession-mode: t
psession-savehist-mode: t
global-git-gutter-mode: t
git-gutter-mode: t
display-time-mode: t
winner-mode: t
helm-epa-mode: t
helm-descbinds-mode: t
helm-adaptive-mode: t
helm-mode: t
helm-minibuffer-history-mode: t
helm-ff-icon-mode: t
shell-dirtrack-mode: t
helm-popup-tip-mode: t
async-bytecomp-package-mode: t
dired-async-mode: t
minibuffer-depth-indicate-mode: t
tooltip-mode: t
global-eldoc-mode: t
eldoc-mode: t
show-paren-mode: t
mouse-wheel-mode: t
file-name-shadow-mode: t
global-font-lock-mode: t
font-lock-mode: t
column-number-mode: t
line-number-mode: t
transient-mark-mode: t
auto-composition-mode: t
auto-encryption-mode: t
auto-compression-mode: t
Load-path shadows:
None found.
Features:
(epa-mail face-remap addressbook-bookmark tv-mu4e-config mu4e-contrib
eshell esh-cmd generator esh-ext esh-opt esh-proc esh-io esh-arg
esh-module esh-groups esh-util cus-start mu4e-patch mu4e mu4e-org
mu4e-main mu4e-view gnus-art mm-uu mml2015 mm-view mml-smime smime
gnutls dig gnus-sum gnus-group gnus-undo gnus-start gnus-dbus gnus-cloud
nnimap nnmail mail-source utf7 nnoo gnus-spec gnus-int gnus-range
gnus-win mu4e-headers mu4e-compose mu4e-draft mu4e-actions smtpmail
mu4e-search mu4e-lists mu4e-bookmarks mu4e-mark mu4e-message shr
pixel-fill kinsoku url-file svg flow-fill hl-line mu4e-contacts
mu4e-update mu4e-folders mu4e-server mu4e-context mu4e-obsolete
mu4e-vars mu4e-helpers mu4e-config ido shadow sort mail-extr helm-ring
emacsbug tramp-archive tramp-gvfs dbus helm-command markdown-mode color
flymake-shellcheck flymake-proc flymake warnings sh-script smie
executable jka-compr bug-reference naquadah-theme solar cal-dst holidays
holiday-loaddefs tv-utils osm dom yaml-mode undo-tree diff queue
psession frameset log-view pcvs-util bash-completion cl-indent pcase
ffap thingatpt autocrypt-message message sendmail yank-media puny rfc822
mml mml-sec mm-decode mm-bodies mm-encode mail-parse rfc2231 rfc2047
rfc2045 mailabbrev gmm-utils mailheader autocrypt-gnus gnus nnheader
gnus-util mail-utils range mm-util mail-prsvr autocrypt-mu4e autocrypt
ietf-drums config-w3m git-gutter mule-util appt diary-lib diary-loaddefs
gud wdired dired-extension org-config ob-gnuplot org-crypt net-utils
time winner autotest-mode autoconf-mode woman man ediff ediff-merg
ediff-mult ediff-wind ediff-diff ediff-help ediff-init ediff-util
init-helm helm-ls-git vc-git diff-mode vc vc-dispatcher helm-fd epa
derived epg rfc6068 epg-config helm-epa helm-imenu imenu
helm-elisp-package helm-find helm-org org ob ob-tangle ob-ref ob-lob
ob-table ob-exp org-macro org-footnote org-src ob-comint org-pcomplete
org-list org-faces org-entities noutline outline org-version
ob-emacs-lisp ob-core ob-eval org-table oc-basic bibtex ol org-keys oc
org-compat advice org-macs org-loaddefs cal-menu calendar cal-loaddefs
helm-external isl helm-descbinds helm-wikipedia wfnames all-the-icons
all-the-icons-faces data-material data-weathericons data-octicons
data-fileicons data-faicons data-alltheicons helm-ipython helm-elisp
helm-eval edebug debug backtrace find-func python project popup
helm-bookmark helm-net xml helm-info bookmark helm-adaptive helm-mode
helm-misc helm-files image-dired image-dired-tags image-dired-external
image-dired-util xdg image-mode exif filenotify tramp tramp-loaddefs
trampver tramp-integration cus-edit pp icons wid-edit files-x
tramp-compat rx shell pcomplete parse-time iso8601 time-date ls-lisp
helm-buffers helm-occur helm-tags helm-locate helm-grep wgrep-helm wgrep
grep compile text-property-search comint ansi-osc ring helm-regexp
format-spec ansi-color helm-utils helm-help helm-types
helm-extensions-autoloads helm-config helm-autoloads helm
helm-global-bindings helm-easymenu edmacro kmacro helm-core
async-bytecomp helm-source helm-multi-match helm-lib dired-async
dired-aux dired dired-loaddefs async diminish cl-extra help-mode
mb-depth server avoid cus-load use-package use-package-ensure
use-package-delight use-package-diminish use-package-bind-key bind-key
easy-mmode use-package-core finder-inf package browse-url url url-proxy
url-privacy url-expand url-methods url-history url-cookie
generate-lisp-file url-domsuf url-util mailcap url-handlers url-parse
auth-source cl-seq eieio eieio-core cl-macs password-cache json subr-x
map byte-opt gv bytecomp byte-compile cconv url-vars cl-loaddefs cl-lib
info w3m-load rmc iso-transl tooltip eldoc paren electric uniquify
ediff-hook vc-hooks lisp-float-type elisp-mode mwheel term/x-win x-win
term/common-win x-dnd tool-bar dnd fontset image regexp-opt fringe
tabulated-list replace newcomment text-mode lisp-mode prog-mode register
page tab-bar menu-bar rfn-eshadow isearch easymenu timer select
scroll-bar mouse jit-lock font-lock syntax font-core term/tty-colors
frame minibuffer nadvice seq simple cl-generic indonesian philippine
cham georgian utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao
korean japanese eucjp-ms cp51932 hebrew greek romanian slovak czech
european ethiopic indian cyrillic chinese composite emoji-zwj charscript
charprop case-table epa-hook jka-cmpr-hook help abbrev obarray oclosure
cl-preloaded button loaddefs theme-loaddefs faces cus-face macroexp
files window text-properties overlay sha1 md5 base64 format env
code-pages mule custom widget keymap hashtable-print-readable backquote
threads inotify lcms2 dynamic-setting font-render-setting cairo
x-toolkit xinput2 x multi-tty make-network-process emacs)
Memory information:
((conses 16 845020 53069)
(symbols 48 42133 6)
(strings 32 248577 7080)
(string-bytes 1 7240706)
(vectors 16 84132)
(vector-slots 8 1813324 153395)
(floats 8 1802 103)
(intervals 56 33940 13862)
(buffers 1000 101))
<#secure method=pgpmime mode=sign>
--
Thierry
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#58786: 29.0.50; Error with Info-minibuf-history not declared
2022-10-26 3:48 bug#58786: 29.0.50; Error with Info-minibuf-history not declared Thierry Volpiatto
@ 2022-10-26 5:36 ` Stefan Kangas
2022-10-26 6:02 ` Thierry Volpiatto
0 siblings, 1 reply; 11+ messages in thread
From: Stefan Kangas @ 2022-10-26 5:36 UTC (permalink / raw)
To: Thierry Volpiatto, 58786
Thierry Volpiatto <thievol@posteo.net> writes:
> Info-minibuf-history is not declared hence the error when calling Info-goto-node:
>
> (void-function Info-minibuf-history)
>
> Evaling (defvar Info-minibuf-history nil) fixes it.
>
> See https://github.com/emacs-helm/helm/issues/2563
I can't seem to reproduce this bug on current master.
Do you have a recipe to reproduce this bug, starting from "emacs -Q"?
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#58786: 29.0.50; Error with Info-minibuf-history not declared
2022-10-26 5:36 ` Stefan Kangas
@ 2022-10-26 6:02 ` Thierry Volpiatto
2022-10-26 6:38 ` Stefan Kangas
0 siblings, 1 reply; 11+ messages in thread
From: Thierry Volpiatto @ 2022-10-26 6:02 UTC (permalink / raw)
To: Stefan Kangas; +Cc: 58786
[-- Attachment #1: Type: text/plain, Size: 1180 bytes --]
Stefan Kangas <stefankangas@gmail.com> writes:
> Thierry Volpiatto <thievol@posteo.net> writes:
>
>> Info-minibuf-history is not declared hence the error when calling Info-goto-node:
>>
>> (void-function Info-minibuf-history)
>>
>> Evaling (defvar Info-minibuf-history nil) fixes it.
>>
>> See https://github.com/emacs-helm/helm/issues/2563
>
> I can't seem to reproduce this bug on current master.
Probably the variable is defined somewhere (custom?) since long time.
> Do you have a recipe to reproduce this bug, starting from "emacs -Q"?
I could reproduce from emacs -Q as described previously.
It is simple, just grep emacs repo for
Info-minibuf-history you will see it is defvared nowhere.
The commit introducing this var is:
commit bf9364a56e618277fe72c90b3a741ade8bc0d205
Author: Lars Ingebrigtsen <larsi@gnus.org>
Date: Thu Nov 11 08:09:59 2021 +0100
Add a command to go the gnu.org version of the info page
* lisp/info.el (Info-url-for-node):
(Info-goto-node-web): New function (bug#44895).
Based on code from Drew Adams <drew.adams@oracle.com>.
--
Thierry
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 686 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#58786: 29.0.50; Error with Info-minibuf-history not declared
2022-10-26 6:02 ` Thierry Volpiatto
@ 2022-10-26 6:38 ` Stefan Kangas
2022-10-26 8:45 ` Thierry Volpiatto
0 siblings, 1 reply; 11+ messages in thread
From: Stefan Kangas @ 2022-10-26 6:38 UTC (permalink / raw)
To: Thierry Volpiatto; +Cc: 58786
Thierry Volpiatto <thievol@posteo.net> writes:
>> Do you have a recipe to reproduce this bug, starting from "emacs -Q"?
>
> I could reproduce from emacs -Q as described previously.
I couldn't reproduce it using this recipe:
0. emacs -Q
1. C-h i g
Are there any other steps required to reproduce it?
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#58786: 29.0.50; Error with Info-minibuf-history not declared
2022-10-26 6:38 ` Stefan Kangas
@ 2022-10-26 8:45 ` Thierry Volpiatto
2022-10-26 9:00 ` Thierry Volpiatto
0 siblings, 1 reply; 11+ messages in thread
From: Thierry Volpiatto @ 2022-10-26 8:45 UTC (permalink / raw)
To: Stefan Kangas; +Cc: 58786
[-- Attachment #1: Type: text/plain, Size: 1019 bytes --]
Stefan Kangas <stefankangas@gmail.com> writes:
> Thierry Volpiatto <thievol@posteo.net> writes:
>
>>> Do you have a recipe to reproduce this bug, starting from "emacs -Q"?
>>
>> I could reproduce from emacs -Q as described previously.
>
> I couldn't reproduce it using this recipe:
>
> 0. emacs -Q
> 1. C-h i g
>
> Are there any other steps required to reproduce it?
I can't reproduce anymore now.
I think vanilla Emacs completing-read fails silently when an history var
is not initialized (correct me if I am wrong) i.e. it behave just like
if it were not defined.
But, do you agree that code using completing-read with an history
variable should declare this var?
If not could you please explain me how such an history variable is defined, loaded etc...
Here we have:
(completing-read prompt #'Info-read-node-name-1 nil t nil
'Info-minibuf-history default)
and Info-minibuf-history is defvared nowhere.
Could you please fix it?
--
Thierry
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 686 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#58786: 29.0.50; Error with Info-minibuf-history not declared
2022-10-26 8:45 ` Thierry Volpiatto
@ 2022-10-26 9:00 ` Thierry Volpiatto
2022-10-26 16:11 ` Drew Adams
0 siblings, 1 reply; 11+ messages in thread
From: Thierry Volpiatto @ 2022-10-26 9:00 UTC (permalink / raw)
To: Stefan Kangas; +Cc: 58786
[-- Attachment #1: Type: text/plain, Size: 860 bytes --]
Thierry Volpiatto <thievol@posteo.net> writes:
> Stefan Kangas <stefankangas@gmail.com> writes:
>
>> Thierry Volpiatto <thievol@posteo.net> writes:
>>
>>>> Do you have a recipe to reproduce this bug, starting from "emacs -Q"?
>>>
>>> I could reproduce from emacs -Q as described previously.
>>
>> I couldn't reproduce it using this recipe:
>>
>> 0. emacs -Q
>> 1. C-h i g
>>
>> Are there any other steps required to reproduce it?
>
> I can't reproduce anymore now.
> I think vanilla Emacs completing-read fails silently when an history var
> is not initialized (correct me if I am wrong) i.e. it behave just like
> if it were not defined.
I also think that this is a bug in itself to not return an error when a
completing-read is used with an history var not initialized (at least it
should fail when hitting M-p).
--
Thierry
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 686 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#58786: 29.0.50; Error with Info-minibuf-history not declared
2022-10-26 9:00 ` Thierry Volpiatto
@ 2022-10-26 16:11 ` Drew Adams
2022-10-26 17:03 ` Thierry Volpiatto
0 siblings, 1 reply; 11+ messages in thread
From: Drew Adams @ 2022-10-26 16:11 UTC (permalink / raw)
To: Thierry Volpiatto, Stefan Kangas; +Cc: 58786@debbugs.gnu.org
My code includes just a vacuous defvar, to
declare the var but not initialize it to nil.
(defvar Info-minibuf-history)
That works, and in all Emacs versions, AFAIK.
On the other hand, we still have this in the
Elisp manual, node `Minibuffer History'. It
says that you must initialize a history var
to nil. I don't think that's correct (and I
don't think that should be needed).
If you don't specify HISTORY, then the
default history list 'minibuffer-history'
is used. For other standard history lists,
see below. You can also create your own
history list variable; just initialize it
to 'nil' before the first use.
I think the last sentence can/should be
changed to tell users to declare the var,
not that they need to initialize it to nil.
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#58786: 29.0.50; Error with Info-minibuf-history not declared
2022-10-26 16:11 ` Drew Adams
@ 2022-10-26 17:03 ` Thierry Volpiatto
2022-10-26 18:40 ` Drew Adams
2022-10-26 19:21 ` Stefan Kangas
0 siblings, 2 replies; 11+ messages in thread
From: Thierry Volpiatto @ 2022-10-26 17:03 UTC (permalink / raw)
To: Drew Adams; +Cc: 58786@debbugs.gnu.org, Stefan Kangas
[-- Attachment #1: Type: text/plain, Size: 567 bytes --]
Hello Drew,
Drew Adams <drew.adams@oracle.com> writes:
> My code includes just a vacuous defvar, to
> declare the var but not initialize it to nil.
>
> (defvar Info-minibuf-history)
You should not have to do this, this defvar should be in info.el on top
of file ideally. Initialized to nil or not I don't know, but it should
be declared anyway.
I personally initialize these history vars to nil in all my packages.
> That works, and in all Emacs versions, AFAIK.
AFAIK this variable is not used in previous emacs versions.
--
Thierry
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 686 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#58786: 29.0.50; Error with Info-minibuf-history not declared
2022-10-26 17:03 ` Thierry Volpiatto
@ 2022-10-26 18:40 ` Drew Adams
2022-10-26 19:21 ` Stefan Kangas
1 sibling, 0 replies; 11+ messages in thread
From: Drew Adams @ 2022-10-26 18:40 UTC (permalink / raw)
To: Thierry Volpiatto; +Cc: 58786@debbugs.gnu.org, Stefan Kangas
> > My code includes just a vacuous defvar, to
> > declare the var but not initialize it to nil.
> > (defvar Info-minibuf-history)
>
> You should not have to do this, this defvar should be in info.el on top
> of file ideally.
Yes, it should be in info.el.
The commit you noted was based on code I provided.
That's why I mentioned that in my code I do include
a vacuous defvar, to declare the (special) variable.
> Initialized to nil or not I don't know, but it should
> be declared anyway.
> I personally initialize these history vars to nil in all my packages.
>
> > That works, and in all Emacs versions, AFAIK.
>
> AFAIK this variable is not used in previous emacs versions.
But my code (info+.el) uses it, and that library
works in all Emacs releases, starting with 23.
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#58786: 29.0.50; Error with Info-minibuf-history not declared
2022-10-26 17:03 ` Thierry Volpiatto
2022-10-26 18:40 ` Drew Adams
@ 2022-10-26 19:21 ` Stefan Kangas
2022-10-26 20:21 ` Thierry Volpiatto
1 sibling, 1 reply; 11+ messages in thread
From: Stefan Kangas @ 2022-10-26 19:21 UTC (permalink / raw)
To: Thierry Volpiatto, Drew Adams; +Cc: 58786
close 58786 29.1
thanks
Thierry Volpiatto <thievol@posteo.net> writes:
> Drew Adams <drew.adams@oracle.com> writes:
>
>> My code includes just a vacuous defvar, to
>> declare the var but not initialize it to nil.
>>
>> (defvar Info-minibuf-history)
>
> You should not have to do this, this defvar should be in info.el on top
> of file ideally. Initialized to nil or not I don't know, but it should
> be declared anyway.
> I personally initialize these history vars to nil in all my packages.
I've now declared it (commit 68548af68a), and I'm closing this bug.
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#58786: 29.0.50; Error with Info-minibuf-history not declared
2022-10-26 19:21 ` Stefan Kangas
@ 2022-10-26 20:21 ` Thierry Volpiatto
0 siblings, 0 replies; 11+ messages in thread
From: Thierry Volpiatto @ 2022-10-26 20:21 UTC (permalink / raw)
To: Stefan Kangas; +Cc: 58786, Drew Adams
[-- Attachment #1: Type: text/plain, Size: 685 bytes --]
Stefan Kangas <stefankangas@gmail.com> writes:
> close 58786 29.1
> thanks
>
> Thierry Volpiatto <thievol@posteo.net> writes:
>
>> Drew Adams <drew.adams@oracle.com> writes:
>>
>>> My code includes just a vacuous defvar, to
>>> declare the var but not initialize it to nil.
>>>
>>> (defvar Info-minibuf-history)
>>
>> You should not have to do this, this defvar should be in info.el on top
>> of file ideally. Initialized to nil or not I don't know, but it should
>> be declared anyway.
>> I personally initialize these history vars to nil in all my packages.
>
> I've now declared it (commit 68548af68a), and I'm closing this bug.
Thanks.
--
Thierry
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 686 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2022-10-26 20:21 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-10-26 3:48 bug#58786: 29.0.50; Error with Info-minibuf-history not declared Thierry Volpiatto
2022-10-26 5:36 ` Stefan Kangas
2022-10-26 6:02 ` Thierry Volpiatto
2022-10-26 6:38 ` Stefan Kangas
2022-10-26 8:45 ` Thierry Volpiatto
2022-10-26 9:00 ` Thierry Volpiatto
2022-10-26 16:11 ` Drew Adams
2022-10-26 17:03 ` Thierry Volpiatto
2022-10-26 18:40 ` Drew Adams
2022-10-26 19:21 ` Stefan Kangas
2022-10-26 20:21 ` Thierry Volpiatto
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).