Skip to content

Conversation

tchebb
Copy link

@tchebb tchebb commented Sep 26, 2025

See individual commit messages for details. Please don't squash when merging, as each commit is independent.

The ln command in EFS Shell operates the same as ln on Linux: it takes
the existing file first and the new link second. But the argument names
were swapped, making the argparse help text confusing.
When we have list permission for a directory but not read permission for
a file within that directory, all metadata for that file except its name
is zeroed out. Currently, this causes a crash looking up the "File type"
string. Add a new entry to gracefully handle this case. Unknown files
now look like this:

  +---------------------------------------------------------------------------------------------------------------------------+
  + File type | Special flags | File rights | File name               | File size | Modification        | Creation            +
  +---------------------------------------------------------------------------------------------------------------------------+
  + Unknown   |               | ---------   | 'mcfg'                  | 0         | 1969-12-31 19:00:00 | 1969-12-31 19:00:00 +
  +---------------------------------------------------------------------------------------------------------------------------+
When removing a symlink, we want to know about the link itself and not
its target to select between unlink() and rmdir().
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant