Ticket #7448 (closed defect: fixed)

Opened 13 years ago

Last modified 13 years ago

Do not configure a text-mode editor

Reported by: baum Owned by: lasgouttes
Priority: normal Milestone: 1.6.10
Component: configure Version: 1.6.9
Severity: normal Keywords:
Cc: sven@…

Description

LyX searches for 'sensible-editor' as default editor for text files. Since this can actually be a text mode editor, LyX becomes unusable if the user trues to edit a text file (see bug #7404). Therefore, LyX should ensure that it only chooses GUI editors and not editors with a terminal based UI.

Change History

comment:1 Changed 13 years ago by sanda

so should we kill sensible-editor?

comment:2 Changed 13 years ago by lasgouttes

Yes, probably.

comment:3 Changed 13 years ago by sanda

  • Cc sven@… added

i see this is debian specific. Sven is it possible to ask for X-only variant of sensible-editor?

comment:4 Changed 13 years ago by sven

I don't think so. There is rather old bug about this issue ​http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=243461

Looking at the sensible-editor script it's not very flexible nor creative in finding an approriate editor. (I somehow feel guilty to have pushed this mess upstream. Lesson learned.)

comment:5 Changed 13 years ago by sanda

ok, i will kill this from our sources.

comment:6 Changed 13 years ago by sanda

  • Status changed from new to closed
  • Resolution set to fixed

r38350

comment:7 Changed 13 years ago by spitz

  • Keywords fixedintrunk added
  • Status changed from closed to reopened
  • Version changed from 2.0.0svn to 1.6.9
  • Resolution fixed deleted
  • Milestone set to 1.6.10

the setting is also in branch. We should kill it there as well.

comment:8 Changed 13 years ago by spitz

  • Keywords fixedinbranch added

backported at r38351.

comment:9 Changed 13 years ago by spitz

  • Keywords fixedintrunk fixedinbranch removed
  • Status changed from reopened to closed
  • Resolution set to fixed

close these. LyX 2.0 is ready and we have a new stable branch (1.6.10 will be released in a few days).

Note: See TracTickets for help on using tickets.