Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 850760 - >=dev-vcs/git-2.35.1: remove deprecated GNU Emacs support (along with the "emacs" USE flag)
Summary: >=dev-vcs/git-2.35.1: remove deprecated GNU Emacs support (along with the "em...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Robin Johnson
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-06-09 15:27 UTC by Maciej Barć
Modified: 2022-06-11 06:31 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Maciej Barć gentoo-dev 2022-06-09 15:27:29 UTC
USE=emacs in dev-vcs/git has little to no effect, except possible annoyances, that is because the Emacs Lisp libraries it ships with cannot be used.

This is the content of "/usr/share/emacs/site-lisp/git/git.el":
> (error "git.el no longer ships with git. It's recommended to
> replace its use with Magit, or simply delete references to git.el
> in your initialization file(s). See contrib/emacs/README in git's
> sources (https://github.com/git/git/blob/master/contrib/emacs/README)
> for suggested alternatives and for why this happened. Emacs's own
> VC mode and Magit are viable alternatives.")
Above file will throw a error while attempted to be loaded.

Checked with version dev-vcs/git-2.35.1.

Also see:
https://github.com/git/git/commit/6d5ed4836db94c07498e34ddebc31abbe532c8e1
https://github.com/git/git/blob/master/contrib/emacs/git.el
https://github.com/git/git/blob/master/contrib/emacs/git-blame.el
Comment 2 Maciej Barć gentoo-dev 2022-06-09 16:53:47 UTC
When removing "emacs" use flag we can instead rely on "optfeature" to inform the users about possibility of using magit - "app-emacs/magit" & print "einfo" that Emacs' "vc" can also be used.
Comment 3 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-06-10 02:10:31 UTC
Nice spot.
Comment 4 Ulrich Müller gentoo-dev 2022-06-10 17:45:50 UTC
Right, we had already disabled the bundled elisp files for >=emacs-22 in 2007, see bug 194690. It was partially reenabled on user's request in bug 235393.

This no longer makes sense, so let's just remove the emacs flag. IMHO no need for optfeature.
Comment 5 Larry the Git Cow gentoo-dev 2022-06-11 06:31:06 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=b84958e4b536ffc773078d98a57bf28ec0024018

commit b84958e4b536ffc773078d98a57bf28ec0024018
Author:     Ulrich Müller <ulm@gentoo.org>
AuthorDate: 2022-06-11 06:26:00 +0000
Commit:     Ulrich Müller <ulm@gentoo.org>
CommitDate: 2022-06-11 06:30:59 +0000

    dev-vcs/git: Drop emacs USE flag
    
    Closes: https://bugs.gentoo.org/850760
    Signed-off-by: Ulrich Müller <ulm@gentoo.org>

 dev-vcs/git/files/50git-gentoo.el | 13 -------------
 dev-vcs/git/git-2.32.0-r1.ebuild  | 26 +++-----------------------
 dev-vcs/git/git-2.33.1.ebuild     | 26 +++-----------------------
 dev-vcs/git/git-2.34.1-r1.ebuild  | 26 +++-----------------------
 dev-vcs/git/git-2.34.1.ebuild     | 26 +++-----------------------
 dev-vcs/git/git-2.35.1.ebuild     | 26 +++-----------------------
 dev-vcs/git/git-2.35.2.ebuild     | 26 +++-----------------------
 dev-vcs/git/git-2.35.3.ebuild     | 26 +++-----------------------
 dev-vcs/git/git-2.36.0.ebuild     | 26 +++-----------------------
 dev-vcs/git/git-2.36.1.ebuild     | 26 +++-----------------------
 dev-vcs/git/git-9999-r1.ebuild    | 26 +++-----------------------
 dev-vcs/git/git-9999-r2.ebuild    | 26 +++-----------------------
 dev-vcs/git/git-9999-r3.ebuild    | 26 +++-----------------------
 dev-vcs/git/git-9999.ebuild       | 26 +++-----------------------
 14 files changed, 39 insertions(+), 312 deletions(-)