Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

evil-iedit/emacs 31.01: defadvice obsolte #16776

Closed
dannywillems opened this issue Jan 2, 2025 · 2 comments
Closed

evil-iedit/emacs 31.01: defadvice obsolte #16776

dannywillems opened this issue Jan 2, 2025 · 2 comments

Comments

@dannywillems
Copy link

With Emacs 31.0, we're getting this warning:

Warning: ‘defadvice’ is an obsolete macro (as of 30.1); use ‘advice-add’ or ‘define-advice’

Full message:

../../../.emacs.d/elpa/31.0/develop/evil-iedit-state-20220219.1432/evil-iedit-state-autoloads.el: Warning: ‘defadvice’ is an obsolete macro (as of 30.1); use ‘advice-add’ or ‘define-advice’

Coming from here: https://github.com/syl20bnr/evil-iedit-state/blob/master/evil-iedit-state.el.

Is it a plugin we can get rid of?

@bcc32
Copy link
Collaborator

bcc32 commented Jan 3, 2025

This package is still used (e.g., by pressing SPC s e). I use it pretty regularly for things like variable renaming, for example.

I think we should just update the package to use define-advice instead. Looks like somebody has opened a pull request to do so: syl20bnr/evil-iedit-state#39.

@smile13241324
Copy link
Collaborator

I have made a quick fork and incorporated the required changes from the PR of @durantschoon. When the package is updated it will be pulled from our fork instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants