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

Release 0.8.0 to stop public dependency on syntex #374

Closed
erickt opened this issue Jun 11, 2016 · 3 comments
Closed

Release 0.8.0 to stop public dependency on syntex #374

erickt opened this issue Jun 11, 2016 · 3 comments
Assignees
Labels
Milestone

Comments

@erickt
Copy link
Member

erickt commented Jun 11, 2016

After we get the soon-to-be-released 0.7.10 out, how about we do a 0.8.0 release that also breaks serde_codegen's dependency on syntex, since that's caused a lot of havoc for people, as seen in #358?

Is it worthwhile trying to get the other v0.8.0 milestone tasks done?

This also could be done by just incrementing serde_codegen's version. If we do that, we should consider whether or not we should just pull serde_codegen and serde_macros into a separate repository, since the two aren't updated at the same time that often.

@erickt erickt added this to the v0.8.0 milestone Jun 11, 2016
@dtolnay
Copy link
Member

dtolnay commented Jun 11, 2016

I think 0.8.0 can wait. People can already use expand in 0.7.9 to avoid being broken by Syntex - the register function doesn't break them if they don't use it.

The only other actionable issue scheduled for 0.8.0 (#267) does not seem urgent now that Cargo has a workaround.

@dtolnay
Copy link
Member

dtolnay commented Jun 11, 2016

I filed #376 regarding your idea to version serde_codegen and serde_macros independently from serde.

@dtolnay dtolnay changed the title Release 0.8.0 to break public dependency on syntex Release 0.8.0 to stop public dependency on syntex Jun 12, 2016
@dtolnay
Copy link
Member

dtolnay commented Jul 17, 2016

I am closing this issue now that we have the ball rolling on an 0.8.0 release. The Syntex dependency is removed in #445.

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

No branches or pull requests

2 participants