Skip to content

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Mar 18, 2020

test_site.test_startup_imports() is now skipped if a path of sys.path
contains a .pth file.

Sort test_site imports.

https://bugs.python.org/issue27807

test_site.test_startup_imports() is now skipped if a path of sys.path
contains a .pth file.

Sort test_site imports.
@vstinner
Copy link
Member Author

@corona10: Would you mind to review this change?

@corona10
Copy link
Member

corona10 commented Mar 18, 2020

Would you mind to review this change?

I will take a look :)

Copy link
Member

@corona10 corona10 left a comment

Choose a reason for hiding this comment

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

Just nit comment except LGTM

Co-Authored-By: Petr Viktorin <encukou@gmail.com>
@vstinner
Copy link
Member Author

@encukou: woops, nice catch :-) Thanks for the suggestion, I like to just click on a button :-)

@stratakis
Copy link
Contributor

I'll test this PR downstream

@vstinner vstinner merged commit d18de46 into python:master Mar 18, 2020
@vstinner vstinner deleted the test_site_pth branch March 18, 2020 17:27
@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-19089 is a backport of this pull request to the 3.8 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 20, 2020
…H-19060)

test_site.test_startup_imports() is now skipped if a path of sys.path
contains a .pth file.

Sort test_site imports.
(cherry picked from commit d18de46)

Co-authored-by: Victor Stinner <vstinner@python.org>
vstinner added a commit that referenced this pull request Mar 20, 2020
…H-19060) (GH-19090)

test_site.test_startup_imports() is now skipped if a path of sys.path
contains a .pth file.

Sort test_site imports.
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 20, 2020
…ythonGH-19060) (pythonGH-19090)

test_site.test_startup_imports() is now skipped if a path of sys.path
contains a .pth file.

Sort test_site imports.
(cherry picked from commit ba26bf3)

Co-authored-by: Victor Stinner <vstinner@python.org>
miss-islington added a commit that referenced this pull request Mar 20, 2020
…H-19060) (GH-19090)

test_site.test_startup_imports() is now skipped if a path of sys.path
contains a .pth file.

Sort test_site imports.
(cherry picked from commit ba26bf3)

Co-authored-by: Victor Stinner <vstinner@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants