Ticket #3412 (closed defect: fixed)

Opened 9 years ago

Last modified 8 years ago

Wrong device name with symbolic link in information panel

Reported by: narcan Owned by: andrew_b
Priority: minor Milestone: 4.8.16
Component: mc-core Version: 4.8.13
Keywords: Cc: rpnpif@…, onlyjob@…
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master

Description

Hello,

Please see bug: ​https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=746990

to reproduce it:
"
Let 2 ext4 partitions : /dev/sdb3 and /dev/sdb4.
/dev/sdb3 is mounted on /mnt/dir3 and /dev/sdb4 is mounted
on /mnt/dir4
Do these bash commands :
cd /mnt/dir3
mkdir mytry3
cd mytry3
echo "try 1" > myfile
mkdir /mnt/dir4/mytry4
cd /mnt/dir4/mytry4
echo "try 1" > myfile
ln -s /mnt/dir3/mytry3 /mnt/dir4/mytry3
mc
Ctrl-x i
Select /mnt/dir4/mytry4/myfile
On the informations panel, device is /dev/sdb4. Good.
Now, select /.. on the top.
Select mytry3
Select myfile
The displayed device is : /dev/sdb4 ! FALSE !
It was not updated.
It should be /dev/sdb3.
"

Many thanks

Denis

Change History

comment:1 Changed 9 years ago by egmont

Confirmed. Similarly, the Filesystem, Type, Free space/nodes aren't updated either.

comment:2 Changed 8 years ago by onlyjob

  • Cc onlyjob@… added

comment:3 Changed 8 years ago by and

Thanks Alain and Denis for reporting

when Cd follow symlink option is enabled, mc save internal different CWD
which don't point to real path device

patch attached

Changed 8 years ago by and

comment:4 Changed 8 years ago by and

  • Milestone changed from Future Releases to 4.8.16

comment:5 Changed 8 years ago by andrew_b

  • Status changed from new to accepted
  • Owner set to andrew_b

Branch: 3412_device_link_info
changeset:c1ea489753489080c2b32cfb94c81169ab5f22b4

comment:6 Changed 8 years ago by andrew_b

  • Votes for changeset set to andrew_b
  • Branch state changed from no branch to approved

comment:7 Changed 8 years ago by andrew_b

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

Merged to master: [c1ea489753489080c2b32cfb94c81169ab5f22b4].

comment:8 Changed 8 years ago by andrew_b

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