Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

file-list: Also display unix mode/permissions #293

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

ZNikke
Copy link
Contributor

@ZNikke ZNikke commented May 11, 2023

Users sometimes wants to know the mode (permission) flags of objects without having to show the file details of each individual item.

This patch adds the familiar unix ls-style representation to the file listing.

In order to closer resemble the familiar/expected style, the common function convertToNameValueObject() is also augmented with an optional parameter that allows displaying the file type instead of ? since the mode bits available doesn't include this information.

Fixes: #202

@ZNikke
Copy link
Contributor Author

ZNikke commented May 17, 2023

Updated patch to also be aware of LINK file type, in addition to REGULAR/DIR.

Users sometimes wants to know the mode (permission) flags of objects
without having to show the file details of each individual item.

This patch adds the familiar unix ls-style representation to the file
listing.

In order to closer resemble the familiar/expected style, the common
function convertToNameValueObject() is also augmented with an optional
parameter that allows displaying the file type instead of ? since the
mode bits available doesn't include this information.

Fixes: dCache#202

Signed-off-by: Niklas Edmundsson <[email protected]>
@kofemann kofemann merged commit 794765e into dCache:master Dec 20, 2023
@kofemann
Copy link
Member

Thanks!

@ZNikke ZNikke deleted the file-list-mode branch December 28, 2023 12:46
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.

Request: Display unix permissions on a file or directory
2 participants