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

git-commit.el has been part of magit, maybe can remove git-commit.el ? #864

Closed
Spike-Leung opened this issue Sep 4, 2024 · 5 comments
Closed

Comments

@Spike-Leung
Copy link

Hi, purcell,

Recently, I encountered a warning that it seems git-commit.el is already part of magit and doesn't need to be installed separately.

■ Emergency (emacsql): Uninstall the `git-commit' package.

The "git-commit.el" library is no longer distributed as a separate
package. It is now distributed as part of the `magit' package.

The git-commit' package now does nothing but display this warning. If it is located earlier on the load-path' than magit' is, then that prevents the proper "git-commit.el" libraries from magit' from being
loaded.

Please uninstalling `git-commit' and then restart Emacs.

Should we update the configuration here to eliminate the warning?

(when (maybe-require-package 'git-commit)

@purcell purcell closed this as completed in c6916a8 Sep 4, 2024
@purcell
Copy link
Owner

purcell commented Sep 4, 2024

Yep, fixed, thanks!

@Spike-Leung
Copy link
Author

Thank you !👍

@purcell
Copy link
Owner

purcell commented Sep 4, 2024

I guess @tarsius will remove the git-commit package recipe from MELPA soon too — Jonas, let me know if you'd like me to do so.

0xMF pushed a commit to 0xMF/emacs.d that referenced this issue Sep 4, 2024
@tarsius
Copy link
Contributor

tarsius commented Sep 4, 2024

The purpose of the fake git-commit package is to be installed as an update for people who already have an older git-commit installed. That way that transitional version/package, can inform the user that they need to uninstall it (and potentially open issues such as this one).

Without doing that, older git-commit could end up remaining installed, which would become an issue, once the git-commit.el in magit has diverged enough, from what was once published as a separate package.

I therefore intend to keep this fake package around for a while. That way, user who update their Emacs packages infrequently can also benefit from this transitional kludge.

@purcell
Copy link
Owner

purcell commented Sep 4, 2024

Cool, I figured you'd have a transition plan so I didn't want to remove the package unilaterally without checking. Will leave it as-is, thanks.

0xMF pushed a commit to 0xMF/emacs.d that referenced this issue Sep 5, 2024
0xMF pushed a commit to 0xMF/emacs.d that referenced this issue Sep 5, 2024
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