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

fix for certain platforms/versions not having os.O_NOFOLLOW #1344

Merged
merged 3 commits into from
Nov 26, 2013
Merged

fix for certain platforms/versions not having os.O_NOFOLLOW #1344

merged 3 commits into from
Nov 26, 2013

Conversation

qwcode
Copy link
Contributor

@qwcode qwcode commented Nov 24, 2013

fix for #849.
this is critical for the PEP453 effort in python 3.4: http://bugs.python.org/issue19749

needs confirmation from another person, as it contains a fair amount of tedium
I did add new tests.

cc @d1b

@dstufft
Copy link
Member

dstufft commented Nov 26, 2013

This may be a silly question, but can we use a tempfile.mkdtemp directory when we're not using the --no-install // --no-download options?

Either way this patch looks reasonable to me.

@qwcode
Copy link
Contributor Author

qwcode commented Nov 26, 2013

I think in the past, that split could have been done, but with #906 on the radar, I wasn't thinking in those terms.
Are you suggesting to do something now?

@dstufft
Copy link
Member

dstufft commented Nov 26, 2013

Not necessarily, just thinking out loud :)

qwcode added a commit that referenced this pull request Nov 26, 2013
fix for certain platforms/versions not having `os.O_NOFOLLOW`
@qwcode qwcode merged commit 0bd1b06 into pypa:1.5.X Nov 26, 2013
@lock lock bot added the auto-locked Outdated issues that have been locked by automation label Jun 5, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Jun 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants