Skip to content

bpo-33764: Appveyor fixes #7364

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

Merged
merged 3 commits into from
Jun 4, 2018
Merged

bpo-33764: Appveyor fixes #7364

merged 3 commits into from
Jun 4, 2018

Conversation

zware
Copy link
Member

@zware zware commented Jun 3, 2018

@zware
Copy link
Member Author

zware commented Jun 3, 2018

It appears that AppVeyor is not accepting builds from us at the moment; I'm not sure why. I've tested this on my own clone (zware#8), though, and from a sample size of one it appears to both speed up the build and prevent the crash.

@terryjreedy if this comes unblocked and I haven't already done it, please feel free to merge.

@vstinner vstinner changed the title Appveyor fixes bpo-33764: Appveyor fixes Jun 4, 2018
@vstinner
Copy link
Member

vstinner commented Jun 4, 2018

I closed my PR #7368 as a duplicate of this one.

@vstinner
Copy link
Member

vstinner commented Jun 4, 2018

And now we cannot merge this PR because of https://bugs.python.org/issue33765 ? AppVeyor, you are not funny with us :-( Is a Python release manager allowed to merge such change? @ned-deily?

@ned-deily ned-deily merged commit 47a6c79 into python:master Jun 4, 2018
@miss-islington
Copy link
Contributor

Thanks @zware for the PR, and @ned-deily for merging it 🌮🎉.. I'm working now to backport this PR to: 2.7, 3.6, 3.7.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-7369 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 4, 2018
* Prevent spurious message if taking a shortcut
* Fix YAML style
* Disable largefile tests
(cherry picked from commit 47a6c79)

Co-authored-by: Zachary Ware <[email protected]>
@miss-islington
Copy link
Contributor

Sorry, @zware and @ned-deily, I could not cleanly backport this to 2.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 47a6c79f0986ce9a84d0efc3c565b28255042911 2.7

@miss-islington miss-islington assigned ned-deily and unassigned zware Jun 4, 2018
@vstinner
Copy link
Member

vstinner commented Jun 4, 2018

It would be nice to see a new AppVeyor build with this new configuration, before merging backports.

@miss-islington
Copy link
Contributor

Sorry, @zware and @ned-deily, I could not cleanly backport this to 3.6 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 47a6c79f0986ce9a84d0efc3c565b28255042911 3.6

ned-deily pushed a commit that referenced this pull request Jun 4, 2018
* Prevent spurious message if taking a shortcut
* Fix YAML style
* Disable largefile tests
(cherry picked from commit 47a6c79)

Co-authored-by: Zachary Ware <[email protected]>
ned-deily pushed a commit to ned-deily/cpython that referenced this pull request Jun 4, 2018
* Prevent spurious message if taking a shortcut
* Fix YAML style
* Disable largefile tests
@bedevere-bot
Copy link

GH-7379 is a backport of this pull request to the 3.6 branch.

ned-deily pushed a commit to ned-deily/cpython that referenced this pull request Jun 4, 2018
* Prevent spurious message if taking a shortcut
* Fix YAML style
* Disable largefile tests
@bedevere-bot
Copy link

GH-7380 is a backport of this pull request to the 2.7 branch.

ned-deily added a commit that referenced this pull request Jun 4, 2018
* Prevent spurious message if taking a shortcut
* Fix YAML style
* Disable largefile tests
ned-deily added a commit that referenced this pull request Jun 4, 2018
* Prevent spurious message if taking a shortcut
* Fix YAML style
* Disable largefile tests
@zware zware deleted the appveyor_fixes branch June 22, 2018 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants