GNU bug report logs - #62162
29.0.60; xref.el doesn't compile on Emacs 26, because multi-pair setq-local

Previous Next

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

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

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

From: João Távora <joaotavora <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 29.0.60; xref.el doesn't compile on Emacs 26, because multi-pair
 setq-local
Date: Mon, 13 Mar 2023 13:51:55 +0000

Hi, 

In a test run of the Eglot test suite (which tests Eglot on Emacs 26.3)

   https://github.com/joaotavora/eglot/actions/runs/4405538673/jobs/7716553677#step:7:453

I'm reasonably sure that one of the failures has to do with a multi-pair
call to `setq-local` in xref.el which appeared in this commit

  commit 3573ebfa6d94411257ffefdf9eb72f508dbe502c
  Author: Juri Linkov <juri <at> linkov.net>
  Date:   Wed Nov 23 10:38:28 2022 +0200

That construct should not be used in xref.el because it isn't available
on Emacs 26, which xref purports to require.  In turn, this breaks Eglot
(and any other packge requiring xref) on those platforms.

João

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

From: Juri Linkov <juri <at> linkov.net>
To: João Távora <joaotavora <at> gmail.com>
Cc: 62162 <at> debbugs.gnu.org, dgutov <at> yandex.ru
Subject: Re: bug#62162: 29.0.60; xref.el doesn't compile on Emacs 26,
 because multi-pair setq-local
Date: Mon, 13 Mar 2023 19:40:40 +0200

close 62162 29.0.60
thanks

> I'm reasonably sure that one of the failures has to do with a multi-pair
> call to `setq-local` in xref.el which appeared in this commit
>
>   commit 3573ebfa6d94411257ffefdf9eb72f508dbe502c
>   Author: Juri Linkov <juri <at> linkov.net>
>   Date:   Wed Nov 23 10:38:28 2022 +0200
>
> That construct should not be used in xref.el because it isn't available
> on Emacs 26, which xref purports to require.  In turn, this breaks Eglot
> (and any other packge requiring xref) on those platforms.

Now this is fixed in emacs-29.  I hope Dmitry will decide
when to bump the xref package version.

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

From: João Távora <joaotavora <at> gmail.com>
To: Juri Linkov <juri <at> linkov.net>
Cc: 62162 <at> debbugs.gnu.org, dgutov <at> yandex.ru
Subject: Re: bug#62162: 29.0.60; xref.el doesn't compile on Emacs 26,
 because multi-pair setq-local
Date: Mon, 13 Mar 2023 23:38:54 +0000

reopen 62162
thanks

Juri Linkov <juri <at> linkov.net> writes:

> close 62162 29.0.60
> thanks
>
>> I'm reasonably sure that one of the failures has to do with a multi-pair
>> call to `setq-local` in xref.el which appeared in this commit
>>
>>   commit 3573ebfa6d94411257ffefdf9eb72f508dbe502c
>>   Author: Juri Linkov <juri <at> linkov.net>
>>   Date:   Wed Nov 23 10:38:28 2022 +0200
>>
>> That construct should not be used in xref.el because it isn't available
>> on Emacs 26, which xref purports to require.  In turn, this breaks Eglot
>> (and any other packge requiring xref) on those platforms.
>
> Now this is fixed in emacs-29.  

Thanks but, I think this should have been fixed in master instead,
because that's where the :core packages are built from, AFAIK, and
emacs-29 doesn't have a problem with that construct.  It'll make no
difference soon enough, hopefully.

> I hope Dmitry will decide when to bump the xref package version.

I think you could have bumped the version, too.  Is there any reason not
to?

Closing this is premature, as the problem stands: the upcoming Eglot
1.12 will have this problem on Emacs 26.  Re-opening.  I'll bump xref.el
myself in a day or so if no-one beats me to it, and then close.

João

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

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: João Távora <joaotavora <at> gmail.com>,
 Juri Linkov <juri <at> linkov.net>
Cc: 62162 <at> debbugs.gnu.org
Subject: Re: bug#62162: 29.0.60; xref.el doesn't compile on Emacs 26, because
 multi-pair setq-local
Date: Tue, 14 Mar 2023 02:29:37 +0200

On 14/03/2023 01:38, João Távora wrote:
> reopen 62162
> thanks
> 
> Juri Linkov <juri <at> linkov.net> writes:
> 
>> close 62162 29.0.60
>> thanks
>>
>>> I'm reasonably sure that one of the failures has to do with a multi-pair
>>> call to `setq-local` in xref.el which appeared in this commit
>>>
>>>    commit 3573ebfa6d94411257ffefdf9eb72f508dbe502c
>>>    Author: Juri Linkov <juri <at> linkov.net>
>>>    Date:   Wed Nov 23 10:38:28 2022 +0200
>>>
>>> That construct should not be used in xref.el because it isn't available
>>> on Emacs 26, which xref purports to require.  In turn, this breaks Eglot
>>> (and any other packge requiring xref) on those platforms.
>>
>> Now this is fixed in emacs-29.
> 
> Thanks but, I think this should have been fixed in master instead,
> because that's where the :core packages are built from, AFAIK, and
> emacs-29 doesn't have a problem with that construct.  It'll make no
> difference soon enough, hopefully.

True enough, on both counts.

>> I hope Dmitry will decide when to bump the xref package version.
> 
> I think you could have bumped the version, too.  Is there any reason not
> to?

Extra courtesy never hurts, and after I responded (on emacs-devel) Juri 
bumped the version 40 minutes after the commit with the fix.

> Closing this is premature, as the problem stands: the upcoming Eglot
> 1.12 will have this problem on Emacs 26.  Re-opening.  I'll bump xref.el
> myself in a day or so if no-one beats me to it, and then close.

We're now waiting for the changes to be merged to master. You could do 
it yourself, if you are in a hurry.

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

From: João Távora <joaotavora <at> gmail.com>
To: Dmitry Gutov <dgutov <at> yandex.ru>
Cc: 62162 <at> debbugs.gnu.org, Juri Linkov <juri <at> linkov.net>
Subject: Re: bug#62162: 29.0.60; xref.el doesn't compile on Emacs 26, because
 multi-pair setq-local
Date: Tue, 14 Mar 2023 00:43:35 +0000

On Tue, Mar 14, 2023 at 12:29 AM Dmitry Gutov <dgutov <at> yandex.ru> wrote:

> and after I responded (on emacs-devel) Juri
> bumped the version 40 minutes after the commit with the fix.

I missed that. Great.

> > Closing this is premature, as the problem stands: the upcoming Eglot
> > 1.12 will have this problem on Emacs 26.  Re-opening.  I'll bump xref.el
> > myself in a day or so if no-one beats me to it, and then close.
>
> We're now waiting for the changes to be merged to master. You could do
> it yourself, if you are in a hurry.

I don't know how, and I'm not in that much of a hurry.  I think one day
without newest Eglot 1.12 in Emacs 26 is bearable.

Thanks,
João

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

From: João Távora <joaotavora <at> gmail.com>
To: Dmitry Gutov <dgutov <at> yandex.ru>
Cc: 62162 <at> debbugs.gnu.org, Juri Linkov <juri <at> linkov.net>
Subject: Re: bug#62162: 29.0.60; xref.el doesn't compile on Emacs 26, because
 multi-pair setq-local
Date: Tue, 14 Mar 2023 18:27:48 +0000

On Tue, Mar 14, 2023 at 12:43 AM João Távora <joaotavora <at> gmail.com> wrote:

> I don't know how, and I'm not in that much of a hurry.  I think one day
> without newest Eglot 1.12 in Emacs 26 is bearable.

I found out that it's not that hard using the admin/automerge script, which
builds and runs tests.  I did that, the merge was clean, and all tests passed,
so I pushed to master.

João

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

From: João Távora <joaotavora <at> gmail.com>
To: Dmitry Gutov <dgutov <at> yandex.ru>, 62162-done <at> debbugs.gnu.org
Cc: Juri Linkov <juri <at> linkov.net>
Subject: Re: bug#62162: 29.0.60; xref.el doesn't compile on Emacs 26, because
 multi-pair setq-local
Date: Wed, 15 Mar 2023 00:47:10 +0000

Newest xref now in GNU ELPA.  Closing.

On Tue, Mar 14, 2023 at 6:27 PM João Távora <joaotavora <at> gmail.com> wrote:
>
> On Tue, Mar 14, 2023 at 12:43 AM João Távora <joaotavora <at> gmail.com> wrote:
>
> > I don't know how, and I'm not in that much of a hurry.  I think one day
> > without newest Eglot 1.12 in Emacs 26 is bearable.
>
> I found out that it's not that hard using the admin/automerge script, which
> builds and runs tests.  I did that, the merge was clean, and all tests passed,
> so I pushed to master.

Previous Next

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