-
-
Notifications
You must be signed in to change notification settings - Fork 131
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
chmodSync
throws EISDIR for directories
#558
Labels
Comments
Looks like the same issue as #391. |
This was referenced Oct 25, 2022
williamstein
added a commit
to sagemathinc/memfs-js
that referenced
this issue
Oct 26, 2022
- this is also upstream streamich#558
williamstein
added a commit
to williamstein/memfs
that referenced
this issue
Oct 26, 2022
🎉 This issue has been resolved in version 3.4.10 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Running
chmodSync
on a directory fails with EISDIR:The error is:
Error: EISDIR: illegal operation on a directory, open '/foo'
.(Try it: https://runkit.com/dabbott/memfs-chmodsync/1.0.0)
The same calls seem to work with node's built-in
fs
, as long as it's a directory where you have permission to write, so I'm guessing this is an issue withchmodSync
?If this is a bug I can try to submit a PR. I haven't looked through the code yet, but seems like it'd be a quick change.
Running
chmod
(not sync) works. Also, runningopenSync
and thenfchmodSync
works, as a workaround:The text was updated successfully, but these errors were encountered: