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

python3: os.umask(002) to os.umask(0o002) #1414

Merged
merged 1 commit into from Jul 15, 2016

Conversation

Projects
None yet
2 participants
@ByReaL
Copy link
Contributor

commented Jul 15, 2016

What does this PR do and why is it necessary?

How was it tested? How can it be tested by the reviewer?

Any background context you want to provide?

What are the relevant tickets if any?

Screenshots (if appropriate)

Further notes

os.umask(0o002) is the python 2/3 way

python3: os.umask(002) to os.umask(0o002)
os.umask(0o002) is the python 2/3 way

@foosel foosel merged commit 3cb0453 into foosel:devel Jul 15, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

foosel added a commit that referenced this pull request Jul 15, 2016

@foosel

This comment has been minimized.

Copy link
Owner

commented Jul 15, 2016

Spotted a left over in the corresponding unit test, see commit above.

foosel added a commit that referenced this pull request Jul 15, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.