Ticket #337 (accepted defect) — at Version 6

Opened 16 years ago

Last modified 15 years ago

wrong $PS1 parser in command line input

Reported by: styx Owned by: styx
Priority: trivial Milestone:
Component: mc-core Version: 4.6.2
Keywords: commited-mc-4.6 commited-master Cc:
Blocked By: Blocking:
Branch state: Votes for changeset:

Description (last modified by styx) (diff)

​https://bugs.launchpad.net/ubuntu/+source/mc/+bug/330633

patch provided by Sergey Nizovtsev.

based on: mc-4.6
branch: 337_ps1_fix
changeset:

Change History

comment:1 Changed 16 years ago by styx

  • Status changed from new to accepted

Changed 16 years ago by styx

patch

comment:2 Changed 16 years ago by styx

  • Keywords vote-styx added
  • Description modified (diff)

comment:3 Changed 16 years ago by slavazanko

  • Keywords vote-slavazanko approved added

Looks good. My vote here.

comment:4 Changed 16 years ago by slavazanko

  • Keywords rework added; vote-styx vote-slavazanko approved removed

Sorry, vote removed.
​http://www.patoche.org/LTT/shell/00000076.html
​http://www.faqs.org/docs/Linux-mini/Xterm-Title.html
​http://www.xfree86.org/current/ctlseqs.html

Need to fix all cases, IMHO

comment:5 Changed 16 years ago by slyfox

Patch looks wrong: it does not seem to handle correctly such pathological case:

PS1=esc(OSC)...esc(ST) ${GOOD_DATA1} esc(OSC)...esc(BEL) ${GOOD_DATA2}

As I understand it will drop all leaving ${GOOD_DATA2}

comment:6 Changed 16 years ago by styx

  • Description modified (diff)

now based on mc-4.6

Changed 16 years ago by snizovtsev

New version

Note: See TracTickets for help on using tickets.