Ticket #2451 (closed defect: fixed)

Opened 9 years ago

Last modified 7 years ago

"Cannot chdir to" "/foo/$bar" if directory hotlist contains dollar sign

Reported by: basinilya Owned by: andrew_b
Priority: major Milestone: 4.8.2
Component: mc-core Version: 4.7.4
Keywords: Cc:
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master committed-stable

Description

Create a directory with '$'
Navigate mc to it
press C-\ to open hotlist
Press "Add current"
Chose the entry you just created and press enter

See the message:
Cannot chdir to "/foo/$bar" No such file or directory

Change History

comment:1 Changed 9 years ago by andrew_b

Works fine for me.

comment:2 Changed 9 years ago by basinilya

All rught, not just dollar sign. Try to create directory named $HOME

comment:3 Changed 9 years ago by andrew_b

Hotlist and quick cd command recognize environment variables (like $HOME) in path. Unfotunately, it is not documented. This behaviour exists for years and I wouldn't change it.

comment:4 follow-up: ↓ 5 Changed 9 years ago by basinilya

Then how to escape '$' ? \$ and %24 don't work

comment:5 in reply to: ↑ 4 Changed 9 years ago by andrew_b

Replying to basinilya:

Then how to escape '$' ? \$ and %24 don't work

This is a bug. \$ should work but doesn't.

comment:6 Changed 8 years ago by andrew_b

See also #2513.

comment:7 Changed 8 years ago by andrew_b

  • Owner set to andrew_b
  • Status changed from new to accepted
  • Branch state set to no branch
  • Blocked By 2695 added
  • Milestone changed from 4.8 to Future Releases

comment:8 Changed 7 years ago by andrew_b

  • Branch state changed from no branch to on review
  • Milestone changed from Future Releases to 4.8.2

Branch: 2451_no_subst_var (parent: 4.8.1-stable).
Initial changeset:1102d55f6b8cfe9c8ff45282cb1e4210bdf3ae5d

comment:9 Changed 7 years ago by slavazanko

  • Votes for changeset set to slavazanko

comment:10 Changed 7 years ago by angel_il

  • Votes for changeset changed from slavazanko to slavazanko angel_il
  • Branch state changed from on review to approved

comment:11 Changed 7 years ago by andrew_b

  • Blocked By 2695 removed

comment:12 Changed 7 years ago by andrew_b

  • Status changed from accepted to testing
  • Votes for changeset changed from slavazanko angel_il to committed-master
  • Resolution set to fixed
  • Branch state changed from approved to merged

Merged to master: [7721a33acfd98b046ee22bba7cd1ae5f20202453].

git log --pretty=oneline 792ae68..7721a33

comment:13 Changed 7 years ago by andrew_b

  • Keywords stable-candidate added

comment:14 Changed 7 years ago by andrew_b

  • Status changed from testing to closed
  • Keywords stable-candidate removed
  • Votes for changeset changed from committed-master to committed-master committed-stable

Cherry-picked to 4.8.1-stable.
Initial changeset:8f57c243d0b7863c5ca13b01347333119c8733dd

git log --pretty=oneline 0932051..dec6011
Note: See TracTickets for help on using tickets.