Skip to content
This repository was archived by the owner on Jun 26, 2020. It is now read-only.

Update the minimum supported Rust version to 1.29.#569

Merged
sunfishcode merged 2 commits intobytecodealliance:masterfrom
sunfishcode:rust-1.29
Oct 31, 2018
Merged

Update the minimum supported Rust version to 1.29.#569
sunfishcode merged 2 commits intobytecodealliance:masterfrom
sunfishcode:rust-1.29

Conversation

@sunfishcode
Copy link
Member

We no longer need the Ubuntu LTS restriction, so now the only only
constraint I'm aware of is Firefox's policy. Fortunately, that tracks
the latest stable delayed by only two weeks. So this puts is at
Rust 1.29 now.

If anyone is aware of other important constraints that we should follow,
please bring them up!


language: rust
rust:
- stable
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did you remove stable?

Copy link
Member Author

Choose a reason for hiding this comment

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

We don't want to immediately update to stable when new versions are released because we're following Firefox's policy.

Copy link
Contributor

Choose a reason for hiding this comment

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

Get it

@bjorn3
Copy link
Contributor

bjorn3 commented Oct 24, 2018

[…]
======  Checking python source files  ======
   === flake8: 3.6.0 (mccabe: 0.6.1, pycodestyle: 2.4.0, pyflakes: 2.0.0) CPython 3.4.3 on Linux ===
./base/instructions.py:1145:-316: W605 invalid escape sequence '\p'
./cdsl/instructions.py:349:-5: W605 invalid escape sequence '\s'
./cdsl/test_ti.py:111:-405: W605 invalid escape sequence '\i'
./cdsl/ti.py:435:-296: W605 invalid escape sequence '\ '
./cdsl/ti.py:435:-29: W605 invalid escape sequence '\ '
./cdsl/typevar.py:124:-28: W605 invalid escape sequence '\i'
./cdsl/xform.py:119:-129: W605 invalid escape sequence '\i'
./semantics/__init__.py:27:-254: W605 invalid escape sequence '\i'
./semantics/__init__.py:27:-71: W605 invalid escape sequence '\i'
[…]

We no longer need the Ubuntu LTS restriction, so now the only only
constraint I'm aware of is Firefox's policy. Fortunately, that tracks
the latest stable delayed by only two weeks. So this puts is at
Rust 1.29 now.
@sunfishcode
Copy link
Member Author

A new version of flake8 added some new warnings. These are now fixed.

This re-introduces several cleanups that we previously deferred for not
supporting Rust 1.25.
@lachlansneff
Copy link
Contributor

This is great! Once this lands, I'll update preopt to use rustc_apfloat.

@sunfishcode
Copy link
Member Author

Ok, let's land this!

@sunfishcode sunfishcode merged commit 8382f9e into bytecodealliance:master Oct 31, 2018
@sunfishcode sunfishcode deleted the rust-1.29 branch October 31, 2018 19:54
This was referenced Oct 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants