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

Set travis env to Python 3.7 #1151

Merged
merged 4 commits into from
Mar 24, 2020
Merged

Set travis env to Python 3.7 #1151

merged 4 commits into from
Mar 24, 2020

Conversation

ngzhian
Copy link
Member

@ngzhian ngzhian commented Mar 24, 2020

Bikeshed requires 3.7, see
speced/bikeshed#321 (comment)

I don't think we can run both 2.7 and 3.7 in travis, we will need to to use 2 different versions of bikeshed, but no way to specific which version of bikeshed in which version of python (could be wrong).

@ngzhian
Copy link
Member Author

ngzhian commented Mar 24, 2020

Cool! builds after some small fixes in document/core/util/mathjax2katex.py, see https://travis-ci.org/github/WebAssembly/spec/builds/666530476?utm_medium=notification&utm_source=github_status.

@ngzhian ngzhian changed the title WIP: Set travis env to Python 3.7 Set travis env to Python 3.7 Mar 24, 2020
@ngzhian ngzhian requested a review from binji March 24, 2020 21:48
@binji
Copy link
Member

binji commented Mar 24, 2020

Wow, surprisingly small amount of changes. I was expecting more effort here :)

@binji
Copy link
Member

binji commented Mar 24, 2020

@sbc100 I know you were doing some more python2 -> python3 conversions, do you see anything else we might want to change?

@@ -1,7 +1,7 @@
#! /usr/bin/env python
Copy link
Member

Choose a reason for hiding this comment

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

should this be python3 now?

Copy link
Member Author

Choose a reason for hiding this comment

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

Think this will be python3 since the travis setup only has python3. But will be useful for local builds. I will make the change.

@sbc100
Copy link
Member

sbc100 commented Mar 24, 2020

Seems reasonable.

@ngzhian ngzhian merged commit b338a49 into WebAssembly:master Mar 24, 2020
@ngzhian ngzhian deleted the py3 branch March 24, 2020 23:38
@binji
Copy link
Member

binji commented Mar 25, 2020

Hm, just noticed that there are a lot of errors in the output that look like "!!! Error processing fragment". See line 5821 here: https://travis-ci.org/github/WebAssembly/spec/builds/666836387?utm_source=github_status&utm_medium=notification. Can you take a look?

@ngzhian
Copy link
Member Author

ngzhian commented Mar 25, 2020

Woah woops, should I do a revert of this first?

@binji
Copy link
Member

binji commented Mar 25, 2020

I don't think it's urgent to fix immediately -- it looks like https://webassembly.github.io/spec still works, and we don't usually have a lot of changes to it.

@ngzhian
Copy link
Member Author

ngzhian commented Mar 25, 2020

Sg, I'm looking at it now to figure out what the error messages mean, if I can't figure it out (maybe by lunch?) I'll just send a PR to change this back, then fix it proper later.

gumb0 pushed a commit to wasmx/wasm-spec that referenced this pull request Sep 21, 2020
* Set travis env to Python 3.7

Bikeshed requires 3.7, see
speced/bikeshed#321 (comment)

* Fix import of queue for python3

* Missed 1 more Queue

* Update shebangs to python3
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.

3 participants