Use 'UnixMode' on *nix platforms instead of Windows-only 'Mode' #98
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
I had to increase the width of the
Mode
colum to10
(was7
), otherwise it wraps. We can not add a switch on the OS in the definition of this width, the XML schema would not allow that; we could create format on the fly, but that seems like significant refactoring with possible impact ongci
's latency.Otherwise, the change is a simple switch over the OS to select an appropriate property of the file to be rendered.
Related Issue
#66
Motivation and Context
It allows showing *nix file permissions on non-windows platforms.
How Has This Been Tested?
Tested on my Debian. The tests run by
./build.ps1
are all green, the imported module shows unix permissions/windows permissions depending on the platform.Screenshots (if appropriate):
Types of changes
Checklist: