GNU bug report logs - #36144
27.0.50; xref-show-xrefs-function changed calling convention

Previous Next

Package: emacs;

Reported by: Risto Lahe <laheristo <at> gmail.com>

Date: Sun, 9 Jun 2019 05:29:01 UTC

Severity: normal

Found in version 27.0.50

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 36144 in the body.
You can then email your comments to 36144 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#36144; Package emacs. (Sun, 09 Jun 2019 05:29:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Risto Lahe <laheristo <at> gmail.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Sun, 09 Jun 2019 05:29:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Risto Lahe <laheristo <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 27.0.50; Xref problems
Date: Sun, 9 Jun 2019 07:43:28 +0300
[Message part 1 (text/plain, inline)]
--text follows this line--
Helm-xref and ivy-xref packages have stopped working on master. Other guy
in #emacs IRC have confirmed it too.

recent commits containing “xref” in their subject:

- 49a363c875c66f3d937a7d33e1a1451227a1887d
- 1cadab78e242834adab0fae3cb1feb691d52f8c5
- 62349fe82ad42d7d2a7fb19e40860ee5d6ebd017
- e309818ecee190727d85c6f3f878c99445d06cfe
- 157d506dd73df15b6a80d06f3c15e4a899c4ef81
- 0597c1b94e5206e980865c6513e923280a3c3ce1
- cd7a0bd00802b9331229dfe4a70ea9690d2d70fc
- d0df779224f56eef777f6e75063e49bfb434f9e0
- 5ce082d636a6e321994cc807d3dea9374e17d34b
- 2843945c5fb7179b96bdab71528f57b2e4a30517


In GNU Emacs 27.0.50 (build 1, x86_64-pc-linux-gnu, X toolkit)
 of 2019-06-09 built on system
Repository revision: 1043cd30acffcc0b61da4a80dcf3f8a5ac459267
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12005000
System Description: Gentoo/Linux

Recent messages:
Loading /home/lhe/.emacs.d/custom.el (source)...done
For information about GNU Emacs and the GNU system, type C-h C-a.

Configured using:
 'configure --prefix=/usr --build=x86_64-pc-linux-gnu
 --host=x86_64-pc-linux-gnu --mandir=/usr/share/man
 --infodir=/usr/share/info --datadir=/usr/share --sysconfdir=/etc
 --localstatedir=/var/lib --disable-silent-rules
 --docdir=/usr/share/doc/emacs-vcs-27.0.9999
 --htmldir=/usr/share/doc/emacs-vcs-27.0.9999/html --libdir=/usr/lib64
 --program-suffix=-emacs-27-vcs --includedir=/usr/include/emacs-27-vcs
 --infodir=/usr/share/info/emacs-27-vcs --localstatedir=/var
 --enable-locallisppath=/etc/emacs:/usr/share/emacs/site-lisp
 --without-compress-install --without-hesiod --without-pop
 --with-file-notification=inotify --enable-acl --with-dbus
 --with-modules --without-gameuser --with-gpm --with-json
 --without-kerberos --without-kerberos5 --with-lcms2 --with-xml2
 --without-mailutils --without-selinux --with-gnutls
 --without-libsystemd --with-threads --without-wide-int --with-zlib
 --with-sound=alsa --with-x --without-ns --without-gconf --with-libgmp
 --without-gsettings --without-toolkit-scroll-bars --with-gif
 --with-jpeg --with-png --with-rsvg --with-tiff --with-xpm
 --without-imagemagick --with-xft --without-cairo --without-libotf
 --without-m17n-flt --with-x-toolkit=lucid --with-xaw3d 'CFLAGS=-O2
 -march=native -pipe' CPPFLAGS= 'LDFLAGS=-Wl,-O1 -Wl,--as-needed''

Configured features:
XAW3D XPM JPEG TIFF GIF PNG RSVG SOUND GPM DBUS GLIB NOTIFY INOTIFY ACL
GNUTLS LIBXML2 FREETYPE HARFBUZZ XFT ZLIB LUCID X11 XDBE XIM MODULES
THREADS JSON PDUMPER LCMS2 GMP

Important settings:
  value of $LC_ALL: C.UTF-8
  value of $LANG: C.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Lisp Interaction

Minor modes in effect:
  popwin-mode: t
  global-undo-tree-mode: t
  undo-tree-mode: t
  shell-dirtrack-mode: t
  evil-mode: t
  evil-local-mode: t
  helm-flx-mode: t
  helm-mode: t
  async-bytecomp-package-mode: t
  global-flycheck-mode: t
  flycheck-mode: t
  override-global-mode: t
  global-eldoc-mode: t
  eldoc-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Load-path shadows:
None found.

Features:
(shadow sort mail-extr emacsbug message rmc puny dired dired-loaddefs
rfc822 mml mml-sec epa epg gnus-util rmail rmail-loaddefs
text-property-search time-date mailabbrev gmm-utils mailheader sendmail
mail-utils winner flx helm-command helm-elisp helm-eval edebug backtrace
helm-info popwin elec-pair evil evil-keybindings evil-integration
undo-tree diff cl evil-maps evil-commands ffap reveal flyspell ispell
evil-jumps evil-command-window evil-types evil-search evil-ex shell
pcomplete comint ansi-color evil-macros evil-repeat evil-states
evil-core evil-common derived windmove thingatpt rect evil-digraphs
evil-vars helm-xref xref project ring helm-flx helm-mode helm-files
helm-buffers helm-occur helm-tags helm-locate helm-grep helm-regexp
format-spec helm-utils helm-help helm-types helm-config helm-easymenu
async-bytecomp helm helm-source eieio-compat helm-multi-match helm-lib
async edmacro kmacro general flycheck find-func rx dash doom-one-theme
doom-themes doom-themes-common quelpa-use-package advice quelpa
mm-decode mm-bodies mm-encode mail-parse rfc2231 rfc2047 rfc2045 mm-util
ietf-drums mail-prsvr lisp-mnt help-fns radix-tree diminish cl-extra
help-mode use-package use-package-ensure use-package-delight
use-package-diminish use-package-bind-key bind-key easy-mmode
use-package-core finder-inf site-gentoo mule-util info package easymenu
epg-config url-handlers url-parse auth-source cl-seq eieio eieio-core
cl-macs eieio-loaddefs password-cache json subr-x map url-vars seq
byte-opt gv bytecomp byte-compile cconv cl-loaddefs cl-lib tooltip eldoc
electric uniquify ediff-hook vc-hooks lisp-float-type 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 elisp-mode lisp-mode
prog-mode register page menu-bar rfn-eshadow isearch timer select
scroll-bar mouse jit-lock font-lock syntax facemenu font-core
term/tty-colors frame cl-generic 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 charscript charprop case-table epa-hook jka-cmpr-hook
help simple abbrev obarray minibuffer cl-preloaded nadvice loaddefs
button faces cus-face macroexp files text-properties overlay sha1 md5
base64 format env code-pages mule custom widget hashtable-print-readable
backquote threads dbusbind inotify lcms2 dynamic-setting
font-render-setting x-toolkit x multi-tty make-network-process emacs)

Memory information:
((conses 16 528418 300164)
 (symbols 48 23627 21)
 (strings 32 123849 24579)
 (string-bytes 1 3483457)
 (vectors 16 37417)
 (vector-slots 8 568457 158166)
 (floats 8 208 434)
 (intervals 56 466 536)
 (buffers 992 13))
[Message part 2 (text/html, inline)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#36144; Package emacs. (Sun, 09 Jun 2019 06:30:02 GMT) Full text and rfc822 format available.

Message #8 received at 36144 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Risto Lahe <laheristo <at> gmail.com>
Cc: 36144 <at> debbugs.gnu.org
Subject: Re: bug#36144: 27.0.50; Xref problems
Date: Sun, 09 Jun 2019 09:29:45 +0300
> From: Risto Lahe <laheristo <at> gmail.com>
> Date: Sun, 9 Jun 2019 07:43:28 +0300
> 
> Helm-xref and ivy-xref packages have stopped working on master. Other guy in #emacs IRC have confirmed
> it too.
> recent commits containing “xref” in their subject:

Was this reported to the Helm and Ivi developers?  If so, did they
conclude that the problem was in Emacs core and not in the respective
packages?  It could be, for example, that these packages rely on some
internal implementation details which were now changed; in that case,
the packages should modify their code to unbreak themselves.

If you didn't yet report this to the developers of Helm and Ivy,
please do so, and let's wait for their conclusions.

Thanks.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#36144; Package emacs. (Sun, 09 Jun 2019 09:00:03 GMT) Full text and rfc822 format available.

Message #11 received at 36144 <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Risto Lahe <laheristo <at> gmail.com>, 36144 <at> debbugs.gnu.org
Subject: Re: bug#36144: 27.0.50; Xref problems
Date: Sun, 9 Jun 2019 11:59:48 +0300
On 09.06.2019 7:43, Risto Lahe wrote:
> Helm-xref and ivy-xref packages have stopped working on master.

As a first approximation, you can add

(setq xref-show-definitions-function #'ivy-xref-show-xrefs)

to your config. Then you can ask the ivy-xref author for a dedicated 
function to use for this value.

Same for helm-xref, I guess.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#36144; Package emacs. (Sun, 09 Jun 2019 09:05:02 GMT) Full text and rfc822 format available.

Message #14 received at 36144 <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Risto Lahe <laheristo <at> gmail.com>, 36144 <at> debbugs.gnu.org
Subject: Re: bug#36144: 27.0.50; Xref problems
Date: Sun, 9 Jun 2019 12:04:18 +0300
Sorry, that's just one thing you need.

More importantly, the type of the first argument passed to hose 
functions has changes (it's now a function that returns a list).

So yeah, you should file reports on the corresponding issues trackers.

On 09.06.2019 11:59, Dmitry Gutov wrote:
> On 09.06.2019 7:43, Risto Lahe wrote:
>> Helm-xref and ivy-xref packages have stopped working on master.
> 
> As a first approximation, you can add
> 
> (setq xref-show-definitions-function #'ivy-xref-show-xrefs)
> 
> to your config. Then you can ask the ivy-xref author for a dedicated 
> function to use for this value.
> 
> Same for helm-xref, I guess.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#36144; Package emacs. (Sun, 09 Jun 2019 09:11:01 GMT) Full text and rfc822 format available.

Message #17 received at 36144 <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Eli Zaretskii <eliz <at> gnu.org>, Risto Lahe <laheristo <at> gmail.com>
Cc: 36144 <at> debbugs.gnu.org
Subject: Re: bug#36144: 27.0.50; Xref problems
Date: Sun, 9 Jun 2019 12:10:03 +0300
On 09.06.2019 9:29, Eli Zaretskii wrote:

> It could be, for example, that these packages rely on some
> internal implementation details which were now changed; in that case,
> the packages should modify their code to unbreak themselves.

We've changed the API.

Third-party packages will need to adapt. The required changes will be 
pretty minimal, though.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#36144; Package emacs. (Sun, 09 Jun 2019 14:35:01 GMT) Full text and rfc822 format available.

Message #20 received at 36144 <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Risto Lahe <laheristo <at> gmail.com>, 36144 <at> debbugs.gnu.org
Subject: Re: bug#36144: 27.0.50; Xref problems
Date: Sun, 9 Jun 2019 17:34:16 +0300
On 09.06.2019 12:04, Dmitry Gutov wrote:
> So yeah, you should file reports on the corresponding issues trackers.

FWIW, here's a PR for one of them: 
https://github.com/alexmurray/ivy-xref/pull/20




Changed bug title to '27.0.50; xref-show-xrefs-function changed calling convention' from '27.0.50; Xref problems' Request was from Noam Postavsky <npostavs <at> gmail.com> to control <at> debbugs.gnu.org. (Wed, 12 Jun 2019 13:03:01 GMT) Full text and rfc822 format available.

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#36144; Package emacs. (Sun, 05 Dec 2021 04:21:02 GMT) Full text and rfc822 format available.

Message #25 received at 36144 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Dmitry Gutov <dgutov <at> yandex.ru>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 36144 <at> debbugs.gnu.org,
 Risto Lahe <laheristo <at> gmail.com>
Subject: Re: bug#36144: 27.0.50; xref-show-xrefs-function changed calling
 convention
Date: Sun, 05 Dec 2021 05:19:53 +0100
Dmitry Gutov <dgutov <at> yandex.ru> writes:

> We've changed the API.
>
> Third-party packages will need to adapt. The required changes will be
> pretty minimal, though.

If I understand correctly, there's nothing to fix on the Emacs side,
then, and I'm closing this bug report.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no




bug closed, send any further explanations to 36144 <at> debbugs.gnu.org and Risto Lahe <laheristo <at> gmail.com> Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Sun, 05 Dec 2021 04:21:02 GMT) Full text and rfc822 format available.

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Sun, 02 Jan 2022 12:24:06 GMT) Full text and rfc822 format available.

This bug report was last modified 2 years and 76 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.