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

Add alias for native-comp-deferred deny list #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

codingkoi
Copy link

On Emacs versions > 28 the name of the deny list has changed, but currently nix-doom-emacs is pinned to an older version of Doom Emacs that doesn't have a fix for this.

This change just adds a defvaralias in places where it doesn't exist using feature testing (instead of checking the Emacs version).

This solution was originally developed by @librephoenix.

On Emacs versions > 28 the name of the deny list has changed, but
currently `nix-doom-emacs` is pinned to an older version of Doom Emacs
that doesn't have a fix for this.

This change just adds a `defvaralias` in places where it doesn't
exist using feature testing (instead of checking the Emacs version).

This solution was originally developed by @librephoenix.
@codingkoi
Copy link
Author

This should fix nix-community/nix-doom-emacs#409.

@kaii-zen
Copy link

Thank you! Saved my sanity.

@jeremyshoemaker
Copy link

@ckiee you mentioned that you can merge this?

@ckiee
Copy link
Member

ckiee commented Oct 20, 2023

I'm not able to test currently, my NDE checkout has a bunch of other WIP changes.

@codingkoi
Copy link
Author

Oops, didn't intend to use my work account for that.

Anyway, there's no rush from me, just wanted to make sure you saw it. 😁

@librephoenix
Copy link

Thanks @codingkoi for putting the patch into a PR! I'm excited to switch back to using the official Nix Doom Emacs, rather than my unmaintained fork XD

@mrdev023
Copy link

Any news ?

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

Successfully merging this pull request may close these issues.

6 participants