git does not use sensible-editor when EDITOR not defined

Bug #569979 reported by Deactivated User
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
git (Ubuntu)
Invalid
Undecided
Unassigned
git-core (Debian)
Fix Released
Unknown

Bug Description

Binary package hint: git-core

git does not use sensible-editor as a fallback when $EDITOR is not defined, instead falling back on the system editor. Git should try $EDITOR, sensible-editor, and then editor in that order instead of $EDITOR, editor as it does now.

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in git-core (Ubuntu):
status: New → Confirmed
Revision history for this message
Richard Laager (rlaager) wrote :

Despite the Debian bug which says this was fixed in 1:1.5.3~rc4-1, this is still a problem in 1:1.9.1-1ubuntu0.1 on Ubuntu Trusty.

affects: git-core (Ubuntu) → git (Ubuntu)
Revision history for this message
Anders Kaseorg (andersk) wrote :

Git uses /usr/bin/editor as the fallback editor, in accordance with Debian Policy:

https://www.debian.org/doc/debian-policy/ch-customized-programs.html#s11.4

“Thus, every program that launches an editor or pager must use the EDITOR or PAGER environment variable to determine the editor or pager the user wishes to use. If these variables are not set, the programs /usr/bin/editor and /usr/bin/pager should be used, respectively.

If it is very hard to adapt a program to make use of the EDITOR or PAGER variables, that program may be configured to use /usr/bin/sensible-editor and /usr/bin/sensible-pager as the editor or pager program respectively.”

Since Git does make use of the EDITOR variable, this policy requires Git to fall back to /usr/bin/editor, not /usr/bin/sensible-editor.

Changed in git (Ubuntu):
status: Confirmed → Invalid
Changed in git-core (Debian):
status: Unknown → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.