GNU bug report logs - #61976
[elpa/yasnippet] [PATCH] Fix compilation warnings in Emacs 29

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 61976 in the body.
You can then email your comments to 61976 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: Simen Heggestøyl <simenheg <at> runbox.com>
To: Noam Postavsky <npostavs <at> gmail.com>
Cc: bug-gnu-emacs <at> gnu.org
Subject: [elpa/yasnippet] [PATCH] Fix compilation warnings in Emacs 29
Date: Sun, 05 Mar 2023 09:29:01 +0100

[Message part 1 (text/plain, inline)]

Hi. The attached patch fixes the compilation warnings in yasnippet.el
when compiling with Emacs 29.

[0001-Fix-compilation-warnings-in-Emacs-29.patch (text/x-diff, inline)]

From a5a9254f35dd85f6b945fc274fd020e887949e07 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Simen=20Heggest=C3=B8yl?= <simenheg <at> runbox.com>
Date: Sun, 5 Mar 2023 08:44:01 +0100
Subject: [PATCH] Fix compilation warnings in Emacs 29

---
 yasnippet.el | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/yasnippet.el b/yasnippet.el
index 78ef38ac39..0129087805 100644
--- a/yasnippet.el
+++ b/yasnippet.el
@@ -476,14 +476,15 @@ See also Info node `(elisp) Syntax Descriptors'.")
   '()
   "Hooks to run after a snippet exited.
 
-The hooks will be run in an environment where some variables bound to
-proper values:
+The hooks will be run in an environment where some variables are
+bound to proper values:
 
 `yas-snippet-beg' : The beginning of the region of the snippet.
 
 `yas-snippet-end' : Similar to beg.
 
-Attention: These hooks are not run when exiting nested/stacked snippet expansion!")
+Attention: These hooks are not run when exiting nested/stacked
+snippet expansion!")
 
 (defvar yas-before-expand-snippet-hook
   '()
@@ -555,7 +556,7 @@ conditions.
     sexp))
 
 (defcustom yas-keymap-disable-hook nil
-  "The `yas-keymap' bindings are disabled if any function in this list returns non-nil.
+  "Disable `yas-keymap' bindings if any function in this list returns non-nil.
 This is useful to control whether snippet navigation bindings
 override bindings from other packages (e.g., `company-mode')."
   :type 'hook)
@@ -1813,7 +1814,8 @@ Optional PROMPT sets the prompt to use."
 SNIPPETS is a list of snippet definitions, each taking the
 following form
 
- (KEY TEMPLATE NAME CONDITION GROUP EXPAND-ENV LOAD-FILE KEYBINDING UUID SAVE-FILE)
+ (KEY TEMPLATE NAME CONDITION GROUP EXPAND-ENV LOAD-FILE
+  KEYBINDING UUID SAVE-FILE)
 
 Within these, only KEY and TEMPLATE are actually mandatory.
 
@@ -2980,7 +2982,7 @@ marks it as something else (typically comment ender)."
     'again))
 
 (defun yas-longest-key-from-whitespace (start-point)
-  "As `yas-key-syntaxes' element, look for longest key between point and whitespace.
+  "As `yas-key-syntaxes' element, find longest key between point and whitespace.
 
 A newline will be considered whitespace even if the mode syntax
 marks it as something else (typically comment ender)."
@@ -3728,8 +3730,8 @@ Use as a `:filter' argument for a conditional keybinding."
   "Clears unmodified field if at field start, skips to next tab.
 
 Otherwise deletes a character normally by calling `delete-char'."
-  (interactive)
   (declare (obsolete "Bind to `yas-maybe-skip-and-clear-field' instead." "0.13"))
+  (interactive)
   (cond ((yas--maybe-clear-field-filter t)
          (yas--skip-and-clear (or field (yas-current-field)))
          (yas-next-field 1))
@@ -4740,9 +4742,11 @@ The following count as a field:
 * \"${n: text}\", for a numbered field with default text, as long as N is not 0;
 
 * \"${n: text$(expression)}, the same with a Lisp expression;
-  this is caught with the curiously named `yas--multi-dollar-lisp-expression-regexp'
+  this is caught with the curiously named
+  `yas--multi-dollar-lisp-expression-regexp'
 
-* the same as above but unnumbered, (no N:) and number is calculated automatically.
+* the same as above but unnumbered, (no N:) and number is
+  calculated automatically.
 
 When multiple expressions are found, only the last one counts."
   ;;
-- 
2.39.1

[Message part 3 (text/plain, inline)]

-- Simen

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

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Simen Heggestøyl <simenheg <at> runbox.com>, 
 João Távora <joaotavora <at> gmail.com>
Cc: "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 Noam Postavsky <npostavs <at> gmail.com>, Stefan Monnier <monnier <at> iro.umontreal.ca>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Fri, 1 Sep 2023 21:46:36 +0200

Simen Heggestøyl <simenheg <at> runbox.com> writes:

> Hi. The attached patch fixes the compilation warnings in yasnippet.el
> when compiling with Emacs 29.

Hmm.  Yasnippet seems to be maintained on Github these days, and AFAIU
we pull from there automatically.  Wouldn't installing it directly on
elpa.git lead to conflicts?

Could someone with access perhaps just push this patch to yasnippet on GitHub?

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

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: Simen Heggestøyl <simenheg <at> runbox.com>,
 "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 João Távora <joaotavora <at> gmail.com>,
 Noam Postavsky <npostavs <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Fri, 01 Sep 2023 17:10:52 -0400

>> Hi. The attached patch fixes the compilation warnings in yasnippet.el
>> when compiling with Emacs 29.
> Hmm.  Yasnippet seems to be maintained on Github these days, and AFAIU
> we pull from there automatically.  Wouldn't installing it directly on
> elpa.git lead to conflicts?

Yes, please don't push to `elpa.git` directly (at least as long as the
`:url` isn't nil in `elpa-packages`).

> Could someone with access perhaps just push this patch to yasnippet on GitHub?

I have more pending patches on `scratch/yasnippet` so you could push
yours there, and then try and get someone at the upstream to pay
attention to it?
[ See https://github.com/joaotavora/yasnippet/issues/1167  ]


        Stefan

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

From: João Távora <joaotavora <at> gmail.com>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: Simen Heggestøyl <simenheg <at> runbox.com>,
 "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 Stefan Kangas <stefankangas <at> gmail.com>, Noam Postavsky <npostavs <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Fri, 1 Sep 2023 22:16:26 +0100

On Fri, Sep 1, 2023 at 10:10 PM Stefan Monnier <monnier <at> iro.umontreal.ca> wrote:
>
> >> Hi. The attached patch fixes the compilation warnings in yasnippet.el
> >> when compiling with Emacs 29.
> > Hmm.  Yasnippet seems to be maintained on Github these days, and AFAIU
> > we pull from there automatically.  Wouldn't installing it directly on
> > elpa.git lead to conflicts?
>
> Yes, please don't push to `elpa.git` directly (at least as long as the
> `:url` isn't nil in `elpa-packages`).
>
> > Could someone with access perhaps just push this patch to yasnippet on GitHub?
>
> I have more pending patches on `scratch/yasnippet` so you could push
> yours there, and then try and get someone at the upstream to pay
> attention to it?
> [ See https://github.com/joaotavora/yasnippet/issues/1167  ]

People, I have to say I don't pay much attention to yasnippet these days.
I'm not the maintainer anymore, but Noam has also been absent for quite some
time.  I think it's time I archive the repository and it lives on in
elpa.git.

Then you can push this and all and other patches there.

WDYT?

João

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

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Noam Postavsky <npostavs <at> gmail.com>
Cc: Simen Heggestøyl <simenheg <at> runbox.com>,
 "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 Stefan Kangas <stefankangas <at> gmail.com>,
 João Távora <joaotavora <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Fri, 01 Sep 2023 17:25:35 -0400

> People, I have to say I don't pay much attention to yasnippet these days.
> I'm not the maintainer anymore, but Noam has also been absent for quite some
> time.  I think it's time I archive the repository and it lives on in
> elpa.git.
>
> Then you can push this and all and other patches there.

That would be fine by me (even better if the Github repository can
redirect people to `elpa.gnu.org` or something like that :-).

Noam?


        Stefan

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

From: Stefan Kangas <stefankangas <at> gmail.com>
To: João Távora <joaotavora <at> gmail.com>
Cc: Simen Heggestøyl <simenheg <at> runbox.com>,
 "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 Stefan Monnier <monnier <at> iro.umontreal.ca>, Noam Postavsky <npostavs <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Sat, 2 Sep 2023 01:59:17 +0200

João Távora <joaotavora <at> gmail.com> writes:

> People, I have to say I don't pay much attention to yasnippet these days.
> I'm not the maintainer anymore, but Noam has also been absent for quite some
> time.  I think it's time I archive the repository and it lives on in
> elpa.git.
>
> Then you can push this and all and other patches there.

Another alternative would be to see if there's anyone in the community
that's willing to take over its maintenance, if Noam is indeed no
longer up for it.  I think the cool kids these days put up an
advertisement on Reddit, or something like that.  The less hip option
is to announce it in the project README.  As we were recently reminded
on emacs-devel, you could even send some link to Sacha for the Emacs
newsletter.

Whichever way you decide to go, there are a couple of probably
easy-to-merge patches in the pull requests on GitHub.  Maybe you'd
want to click "Merge" on a few of those first.

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

From: João Távora <joaotavora <at> gmail.com>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: Simen Heggestøyl <simenheg <at> runbox.com>,
 "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 Stefan Monnier <monnier <at> iro.umontreal.ca>, Noam Postavsky <npostavs <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Sat, 2 Sep 2023 05:21:32 +0100

[Message part 1 (text/plain, inline)]

On Sat, Sep 2, 2023, 00:59 Stefan Kangas <stefankangas <at> gmail.com> wrote:

> João Távora <joaotavora <at> gmail.com> writes:
>
> > People, I have to say I don't pay much attention to yasnippet these days.
> > I'm not the maintainer anymore, but Noam has also been absent for quite
> some
> > time.  I think it's time I archive the repository and it lives on in
> > elpa.git.
> >
> > Then you can push this and all and other patches there.
>
> Another alternative would be to see if there's anyone in the community
> that's willing to take over its maintenance, if Noam is indeed no
> longer up for it.  I think the cool kids these days put up an
> advertisement on Reddit, or something like that.  The less hip option
> is to announce it in the project README.  As we were recently reminded
> on emacs-devel, you could even send some link to Sacha for the Emacs
> newsletter.
>

Any of those is fine with me.

>
> Whichever way you decide to go, there are a couple of probably
> easy-to-merge patches in the pull requests on GitHub.  Maybe you'd
> want to click "Merge" on a few of those first.
>

Can you make a list of the safest/most important ones? (keeping on mind
things like backward-compatibility, etc...).

João

>

[Message part 2 (text/html, inline)]

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

From: Stefan Kangas <stefankangas <at> gmail.com>
To: João Távora <joaotavora <at> gmail.com>
Cc: Simen Heggestøyl <simenheg <at> runbox.com>,
 "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 Stefan Monnier <monnier <at> iro.umontreal.ca>, Noam Postavsky <npostavs <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Sat, 2 Sep 2023 12:14:07 -0700

João Távora <joaotavora <at> gmail.com> writes:

> Can you make a list of the safest/most important ones? (keeping on mind
> things like backward-compatibility, etc...).

I don't have the bandwidth for a detailed list right now.

This was an easy one I spotted:

    https://github.com/joaotavora/yasnippet/pull/1125

This one enabling lexical-binding might be a bit more work, or no work.
No idea which.  It would be nice to have, in any case:

    https://github.com/joaotavora/yasnippet/pull/1053

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

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: Simen Heggestøyl <simenheg <at> runbox.com>,
 "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 João Távora <joaotavora <at> gmail.com>,
 Noam Postavsky <npostavs <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Sat, 02 Sep 2023 15:32:32 -0400

> This one enabling lexical-binding might be a bit more work, or no work.
> No idea which.  It would be nice to have, in any case:
>
>     https://github.com/joaotavora/yasnippet/pull/1053

The code up on `scratch/yasnippet` (inside `elpa.git`) also activates
lexical-binding (tho I can't remember if it actively makes use of it in
a way that makes it incompatible with Emacs-23).


        Stefan

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

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: Simen Heggestøyl <simenheg <at> runbox.com>,
 "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 João Távora <joaotavora <at> gmail.com>,
 Noam Postavsky <npostavs <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Mon, 04 Sep 2023 17:18:05 -0400

> Another alternative would be to see if there's anyone in the community
> that's willing to take over its maintenance, if Noam is indeed no
> longer up for it.  I think the cool kids these days put up an
> advertisement on Reddit, or something like that.  The less hip option
> is to announce it in the project README.  As we were recently reminded
> on emacs-devel, you could even send some link to Sacha for the Emacs
> newsletter.

This is actually not mutually exclusive with closing João's repository
and moving to `elpa.git`, since whenever we find new maintainers they
can still use whichever location they like for their upstream repository.


        Stefan

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

From: João Távora <joaotavora <at> gmail.com>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: Simen Heggestøyl <simenheg <at> runbox.com>,
 "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 Stefan Kangas <stefankangas <at> gmail.com>, Noam Postavsky <npostavs <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Mon, 4 Sep 2023 22:22:53 +0100

On Mon, Sep 4, 2023 at 10:18 PM Stefan Monnier <monnier <at> iro.umontreal.ca> wrote:
>
> > Another alternative would be to see if there's anyone in the community
> > that's willing to take over its maintenance, if Noam is indeed no
> > longer up for it.  I think the cool kids these days put up an
> > advertisement on Reddit, or something like that.  The less hip option
> > is to announce it in the project README.  As we were recently reminded
> > on emacs-devel, you could even send some link to Sacha for the Emacs
> > newsletter.
>
> This is actually not mutually exclusive with closing João's repository
> and moving to `elpa.git`, since whenever we find new maintainers they
> can still use whichever location they like for their upstream repository.

Yes, I'll move the repo definitely, maybe with a similar arrangement to
Eglot so at least the "GitHub pages" -- which is where the documentation
is served from -- remains alive.

João

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

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: João Távora <joaotavora <at> gmail.com>
Cc: Simen Heggestøyl
 <simenheg <at> runbox.com>, "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 Stefan Kangas <stefankangas <at> gmail.com>, Noam Postavsky <npostavs <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Mon, 04 Sep 2023 18:25:09 -0400

> Yes, I'll move the repo definitely, maybe with a similar arrangement to
> Eglot so at least the "GitHub pages" -- which is where the documentation
> is served from -- remains alive.

Sounds good.  Keeping existing links working is good.
The only important thing is to try and avoid people forking that "old"
code rather than starting from the "brand new code" that will be in
`elpa.git` (presumably the differences will be minor, at least for
the foreseeable future, but still).


        Stefan

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

From: João Távora <joaotavora <at> gmail.com>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: Simen Heggestøyl <simenheg <at> runbox.com>,
 "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 Stefan Monnier <monnier <at> iro.umontreal.ca>, Noam Postavsky <npostavs <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Tue, 5 Sep 2023 01:07:50 +0100

On Sat, Sep 2, 2023 at 8:14 PM Stefan Kangas <stefankangas <at> gmail.com> wrote:
>
> João Távora <joaotavora <at> gmail.com> writes:
>
> > Can you make a list of the safest/most important ones? (keeping on mind
> > things like backward-compatibility, etc...).
>
> I don't have the bandwidth for a detailed list right now.
>
> This was an easy one I spotted:
>
>     https://github.com/joaotavora/yasnippet/pull/1125


Thanks, I merged that one and it indeed fixes the all test failures
on Emacs28.

I still have two new test failures on latest Emacs and I suspect
it's something to do with overlay changes.

Stefan Monnier can you have a look maybe?  Could be related to your
overlay changes?

With Emacs 28 this passes:

   emacs -Q --batch -L . -l yasnippet-tests.el --eval
'(ert-run-tests-batch-and-exit "issue-824")'

With latest master it doesn't.  The issue seems to be a call to
yas-active-snippets which calls overlays-in in where BEG and END
are 1 and which returns no overlays because they have mysteriously
evaporated.  But there are no overlays with 'evaporate' properties
that I can detect.

João

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

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: Simen Heggestøyl <simenheg <at> runbox.com>,
 "Philip K." <philipk <at> posteo.net>, 61976 <at> debbugs.gnu.org,
 Noam Postavsky <npostavs <at> gmail.com>,
 João Távora <joaotavora <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Tue, 11 Feb 2025 20:37:24 -0800

Stefan Monnier <monnier <at> iro.umontreal.ca> writes:

>> People, I have to say I don't pay much attention to yasnippet these days.
>> I'm not the maintainer anymore, but Noam has also been absent for quite some
>> time.  I think it's time I archive the repository and it lives on in
>> elpa.git.
>>
>> Then you can push this and all and other patches there.
>
> That would be fine by me (even better if the Github repository can
> redirect people to `elpa.gnu.org` or something like that :-).

Was the package moved to GNU ELPA?  If so, should this patch be pushed
there?

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

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: Simen Heggestøyl <simenheg <at> runbox.com>, "Philip
 K." <philipk <at> posteo.net>, 61976-done <at> debbugs.gnu.org,
 Noam Postavsky <npostavs <at> gmail.com>,
 João Távora <joaotavora <at> gmail.com>
Subject: Re: bug#61976: [elpa/yasnippet] [PATCH] Fix compilation warnings in
 Emacs 29
Date: Fri, 14 Feb 2025 17:21:30 -0500

>>> People, I have to say I don't pay much attention to yasnippet these days.
>>> I'm not the maintainer anymore, but Noam has also been absent for quite some
>>> time.  I think it's time I archive the repository and it lives on in
>>> elpa.git.
>>>
>>> Then you can push this and all and other patches there.
>>
>> That would be fine by me (even better if the Github repository can
>> redirect people to `elpa.gnu.org` or something like that :-).
>
> Was the package moved to GNU ELPA?  If so, should this patch be pushed
> there?

No, the package still lives over on Github, but I pushed (a version of)
that patch to that repository, so we can close this.


        Stefan

Previous Next

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