Ticket #231 (testing enhancement: fixed) — at Version 10

Opened 15 years ago

Last modified 7 years ago

Initial Friendly Interactive SHell support

Reported by: me@… Owned by: slyfox
Priority: minor Milestone: Future Releases
Component: mc-core Version: master
Keywords: Cc: zaytsev, SanskritFritz, elpresidento@…, mooffie@…
Blocked By: Blocking:
Branch state: no branch Votes for changeset:

Description (last modified by slyfox) (diff)

Here comes a patch to make fish (​http://fishshell.org/) work together with MC. As I am not an MC user myself I'm not sure if there are things broken, but it is at least a start.

merged as changeset:1b5ae972bbdfe4aaa7f94b4248fc030c5319993c

Change History

Changed 15 years ago by pekdon

Patch for fish shell support

comment:1 Changed 15 years ago by angel_il

  • Keywords fish removed
  • Milestone changed from 4.7 to 4.7.0-pre1

comment:2 Changed 15 years ago by slyfox

  • Keywords review vote-slyfox added

branch:231_bg_shell_fish_support
changeset:128160fef1d44ed8891c0e7e09b994cda3590ddf

Pushed in slightly (cosmetic) modified form.
Patch looks absolutely noninvasive, so no reason not to vote for it :]

comment:3 Changed 15 years ago by slyfox

  • Status changed from new to assigned
  • Owner set to slyfox

comment:4 Changed 15 years ago by angel_il

  • Keywords vote-angel_il added

comment:5 Changed 15 years ago by slavazanko

  • Keywords vote-slavazanko approved added; review removed

hm... I think, need to handle envirovement variable $fish_prompt_normal

I don't see correct prompt.

My test case:

$ export SHELL=/bin/fish
$ fish

slavaz@slavaz-nb /h/s> /path/to/test/mc

I see only '$' symbol as prompt. After pressing CTRL+O no subshel is running. Is I'm wrong or this bug?

comment:6 Changed 15 years ago by slavazanko

  • Keywords review added; vote-slavazanko approved removed

Ops, sorry. :)

comment:7 follow-up: ↓ 8 Changed 15 years ago by angel_il

maybe do this optional ?
--enable-fish

comment:8 in reply to: ↑ 7 Changed 15 years ago by slyfox

Replying to angel_il:

maybe do this optional ?
--enable-fish

For what purpose? What does new code break? Why do we need to uglify this tiny patch with more #ifdefs?

comment:9 Changed 15 years ago by angel_il

  • Keywords approved added

comment:10 Changed 15 years ago by slyfox

  • Keywords committed-master added; review vote-slyfox vote-angel_il approved removed
  • Status changed from assigned to testing
  • Resolution set to fixed
  • Description modified (diff)

Merged as-is. changeset:1b5ae972bbdfe4aaa7f94b4248fc030c5319993c

Nice shell prompt needs slightly more explorations.

Note: See TracTickets for help on using tickets.