Add chmod/chmodSync on *nix (and fix Cargo.toml) #1088
Merged
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.
Continuation from #673 (since no updates for quite a while and this is an important fs call)
Implements
chmod
andchmodSync
on *nix. (On windows this is noop)Prior art:
os.Chmod(name string, mode FileMode) error
fs.chmod(file, mode, callback);
os.chmod(path, mode);
std::fs::set_permissions<P: AsRef>(path: P, perm: Permissions) -> Result<()>
lchmod
is not implemented yet. It seems that Ruststd::fs
only support inspecting symlink metadata, but anyset_permissions/set_mode
calls would eventually set the permission of linked file/dir.Also, add
getopts
entry inCargo.toml
(forgot to include this in the PR yesterday)