0ct0pu5/ladybird
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

Kernel: Remove misleading FIXME in DevTmpFS

Browse source 
This FIXME does not seem to apply anymore. Yes, symbolic links in all
filesystems appear to be slightly broken, but that has nothing to do
with File::absolute_path. Let's remove the wrong FIXME instead of adding
to the confusion.
This commit is contained in:
Ben Wiederhake 2021-10-30 00:54:21 +02:00 • committed by Andreas Kling
parent c05c5a7ff4
commit 9a4e125468
Notes: sideshowbarker 2024-07-18 01:43:01 +09:00
1 changed files with 0 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
Kernel/FileSystem/DevTmpFS.cpp
View file

@ -126,11 +126,6 @@ InodeMetadata DevTmpFSInode::metadata() const
break;
case Type::Link:
metadata.inode = { fsid(), index() };
// FIXME: For now, it might not be possible to change the m_mode due to
// it pointing to another device node which its absolute_path() is not
// recognizable by the VirtualFileSystem.
// When we resolve the issues with the clunky absolute_path() interface,
// this should work out of the box...
metadata.mode = S_IFLNK | m_mode;
break;
default: