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

Ensure setup.cfg is only ASCII. CRI-169 #23872

Merged
merged 1 commit into from
May 4, 2020
Merged

Conversation

nedbat
Copy link
Contributor

@nedbat nedbat commented May 1, 2020

I don't understand why this file needs to be ASCII, or why it was only a
problem recently for one installer, since this has been in this file
since 2014. But CRI-169 has a stack trace, and it's easy enough to make
the file ASCII.

I don't understand why this file needs to be ASCII, or why it was only a
problem recently for one installer, since this has been in this file
since 2014.  But CRI-169 has a stack trace, and it's easy enough to make
the file ASCII.
@nedbat nedbat requested a review from jmbowman May 1, 2020 20:02
@edx-status-bot
Copy link

Your PR has finished running tests. There were no failures.

@jmbowman
Copy link
Contributor

jmbowman commented May 4, 2020

This may have been fixed by pypa/setuptools#1180 in setuptools 40.7.0, we're still pinning to 39.0.1 in the configuration repo. This PR is the fastest way to fix the issue, but do you think we should also upgrade setuptools and pip before the Juniper release? We were getting ready to do that very soon anyway.

@nedbat nedbat merged commit 41a9997 into master May 4, 2020
@nedbat nedbat deleted the nedbat/ascii-setup-cfg branch May 4, 2020 16:21
@nedbat
Copy link
Contributor Author

nedbat commented May 4, 2020

@jmbowman yes, if you want to do it soon, I think it would be good to do it in the next few days for Juniper. Thanks.

@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR may have caused e2e tests to fail on Stage. If you're a member of the edX org, please visit #e2e-troubleshooting on Slack to help diagnose the cause of these failures. Otherwise, it is the reviewer's responsibility. E2E tests have failed. https://gocd.tools.edx.org/go/tab/pipeline/history/deploy_to_stage

@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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.

4 participants