Skip to content

Update for latest libsyntax changes - #303

Merged
erickt merged 3 commits into
serde-rs:masterfrom
antrik:rustup
May 4, 2016
Merged

erickt merged 3 commits into
serde-rs:masterfrom
antrik:rustup

Conversation

@antrik

@antrik antrik commented May 2, 2016

Copy link
Copy Markdown
Contributor

This one is mostly about updating dependencies on syntex, aster, and quasi -- along with a small code patch for adapting to the actual interface changes.

antrik added 3 commits May 2, 2016 07:08
rust-lang/rust@9108fb7
dropped the `map()` method (among other things) -- so we need to adapt
our code.

This change should be backwards-compatible.
This was renamed upstream in
rust-lang/rust@6c44bea

It's an incompatible change requiring new versions of syntex, aster, and
quasi.
Dependencies were updated in previous commit. (For supporting current
Nightly Rust.)

I guess this shouldn't affect our client interface -- which is why these
dependency bumps only get patch level version number updates for Serde,
I presume?
@antrik

antrik commented May 2, 2016

Copy link
Copy Markdown
Contributor Author

And finally, this one depends on serde-deprecated/aster#78 , serde-deprecated/syntex#44 , as well as serde-deprecated/quasi#41

@KodrAus

KodrAus commented May 4, 2016

Copy link
Copy Markdown

Does this build now?

@dtolnay

dtolnay commented May 4, 2016

Copy link
Copy Markdown
Member

@oli-obk @erickt does one of you have permission to retrigger the Travis build?

@erickt
erickt merged commit 8378267 into serde-rs:master May 4, 2016
@erickt

erickt commented May 4, 2016

Copy link
Copy Markdown
Member

@dtolnay: I've merged this in. Hopefully it'll pass travis now.

@dtolnay

dtolnay commented May 4, 2016

Copy link
Copy Markdown
Member

Looks like it passed! Let's cut a release to unblock everybody.

@erickt

erickt commented May 4, 2016

Copy link
Copy Markdown
Member

@dtolnay: 0.7.4 has (finally) been released! Thanks so much for all your work getting this out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants