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

Tolerate execute bit on owner when writing file #173

Merged
merged 1 commit into from
Oct 24, 2019

Conversation

kevin-bates
Copy link
Member

Some filesystems (CIFS) auto-enable execute bits on files. As
a result, we should tolerate execute on owner (but only owner).

Special thanks to @hroncok for providing the missing tidbits to this issue! (Pun intended)

Closes: #172

Some filesystems (CIFS) auto-enable execute bits on files.  As
a result, we should tolerate execute on owner (but only owner).

Closes: jupyter#172
@hroncok
Copy link
Contributor

hroncok commented Oct 16, 2019

I won't be able to recheck this until next Wednesday.
Maybe @MarekSuchanek can.

@MarekSuchanek
Copy link

I won't be able to recheck this until next Wednesday.
Maybe @MarekSuchanek can.

I can on Friday and then Tuesday...

@kevin-bates
Copy link
Member Author

@hroncok, @MarekSuchanek - thank you! We will likely move forward with the merge, but should probably hold off on building a release until we know this looks okay. Thanks for your help!

@hroncok
Copy link
Contributor

hroncok commented Oct 23, 2019

I've verified that this works. Thank You.

@kevin-bates
Copy link
Member Author

Great, thank you @hroncok!

@MSeal, @minrk - looks like we could use a 4.6.1 release once this is merged. I'm happy to create the change log entry.

Copy link
Contributor

@MSeal MSeal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can help get a patch release going for this @kevin-bates if you want to make the changelog entry.

@kevin-bates
Copy link
Member Author

@MSeal - Changelog PR: #174

Thanks!

@MSeal MSeal merged commit 5a3c69c into jupyter:master Oct 24, 2019
@MSeal
Copy link
Contributor

MSeal commented Oct 24, 2019

@kevin-bates Done.

@meeseeksmachine
Copy link

This pull request has been mentioned on Jupyter Community Forum. There might be relevant details there:

https://discourse.jupyter.org/t/kernel-error-in-jupyter-notebook/4728/3

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.

paths.secure_write fails on CIFS volumes
5 participants