Ticket #3923 (closed defect: fixed)

Opened 7 years ago

Last modified 6 years ago

"Cannot create target file" when target has backslash space in the name

Reported by: sergio Owned by: andrew_b
Priority: major Milestone: 4.8.22
Component: mc-core Version: 4.8.21
Keywords: Cc:
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master

Description

$ mkdir /tmp/test

test
$ mc
goto '/tmp/test\ test'
trying to copy into '/tmp/test\ test' will give:
Cannot create target file "/tmp/test test/test"
No such file or directory (2)

Change History

comment:1 Changed 7 years ago by sergio

$ mkdir /tmp/test\\\ test

comment:2 Changed 7 years ago by andrew_b

This is unpronounceable for me in 4.8.21. Or your description is incomplete.

Version 0, edited 7 years ago by andrew_b (next)

comment:3 follow-up: ↓ 4 Changed 7 years ago by sergio

This bug is present on git. Just tested.

$ mkdir /tmp/test\\\ test
$ touch /tmp/test
$ mc
left panel: /tmp
right panel: /tmp/test\ test
copy test from /tmp to /tmp/test\ test

comment:4 in reply to: ↑ 3 Changed 7 years ago by andrew_b

Replying to sergio:

This bug is present on git.

Yes, confirmed.
Sorry, I was inattentive last night.

comment:5 Changed 7 years ago by andrew_b

Related to #3135.

comment:6 Changed 7 years ago by andrew_b

The buggy commit is [ef5d346dd9a11b65efbd213be300d2602c949876].

comment:7 Changed 6 years ago by andrew_b

  • Status changed from new to accepted
  • Owner set to andrew_b
  • Branch state changed from no branch to on review
  • Milestone changed from Future Releases to 4.8.22

Branch: 3923_escape_fileop_filename
Initial changeset:aea06fbc7a177cfd463ce6487d4bb0fe8e657863

The autocompletion is still not working correctly for such file/dir names. But this is a task of another ticket.

comment:8 Changed 6 years ago by andrew_b

  • Votes for changeset set to andrew_b
  • Branch state changed from on review to approved

comment:9 Changed 6 years ago by andrew_b

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

Merged to master: [6cdd3b7ad635e3469e2e3c82fcb70d3b215d6035].

git log --oneline 0075f36..6cdd3b7

comment:10 Changed 6 years ago by andrew_b

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.