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

Why is emacsWithPackagesFromUsePackage part of the "emacs" overlay? #350

Open
cmm opened this issue Aug 5, 2023 · 2 comments
Open

Why is emacsWithPackagesFromUsePackage part of the "emacs" overlay? #350

cmm opened this issue Aug 5, 2023 · 2 comments

Comments

@cmm
Copy link
Contributor

cmm commented Aug 5, 2023

(as opposed to the "package" overlay, that is)

The reason I'm asking is that I'm interested in this repo for the purpose of deriving an Emacs that uses a released core Emacs (i.e. one from unadulterated Nixpkgs) + latest MELPA packages as required by my config.

Thus it would make sense for me to opt out of the "emacs" overlay and use only the "package" one, which is not currently an option because emacsWithPackagesFromUsePackage is in the former and not in the latter.

This is admittedly a theoretical problem right now because the "emacs" overlay does not shadow Nixpkgs's "emacs" attribute, but will become an actual problem when Nixpkgs (shortly, hopefully) grows its own "emacs-pgtk" attribute (NixOS/nixpkgs#246203). The coming name clash can be solved by renaming this overlay's "emacs-pgtk" to something else so it does not shadow anything, but still I wonder whether emacsWithPackagesFromUsePackage attribute is just in the wrong overlay, since putting it in "package" would make the whole issue moot.

@jian-lin
Copy link
Contributor

The coming name clash can be solved by renaming this overlay's "emacs-pgtk" to something else so it does not shadow anything

Agreed. Considering existing names in this overlay, emacs-git-pgtk is a good candidate.

putting it in "package" would make the whole issue moot

This introduces another issue for users who want to use emacsWithPackagesFromUsePackage together with elisp packages from Nixpkgs instead of this overlay. I think this issue affects more users.

emacsWithPackagesFromUsePackage is actually a wrapper like emacs.pkgs.withPackage. It is not an Emacs variant nor a Emacs lisp package. Another solution is to put emacsWithPackagesFromUsePackage into a separate wrapper overlay.

Unfortunately, both of these solutions are breaking changes.

WDYT?

@jian-lin
Copy link
Contributor

cc @adisbladis @leungbk

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

2 participants