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

Add #![feature(alloc)], when enabling collections and/or alloc feature #20

Closed
MathiasKoch opened this issue Apr 30, 2019 · 12 comments · Fixed by #25
Closed

Add #![feature(alloc)], when enabling collections and/or alloc feature #20

MathiasKoch opened this issue Apr 30, 2019 · 12 comments · Fixed by #25

Comments

@MathiasKoch
Copy link

error[E0658]: use of unstable library feature 'alloc': this library is unlikely to be stabilized in its current form or name (see issue #27783)
--> /home/mathias/.cargo/registry/src/6github.com-1ecc6299db9ec823/core_io-0.1.20190427/src/lib.rs:15:51
|
15 | #[cfg(all(feature="collections",no_collections))] extern crate alloc as collections;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: add #![feature(alloc)] to the crate attributes to enable

@jethrogb
Copy link
Owner

jethrogb commented Apr 30, 2019

Whoops, looks like this was accidentally broken in 81adf07

@jethrogb
Copy link
Owner

All these features need to be added back:

  • alloc
  • collections
  • str_char
  • unicode

@parasyte do you think you'd be able to fix this and #19?

@parasyte
Copy link
Contributor

I wasn't able to build with those features enabled. :( rustc complained that they were unknown.

@jethrogb
Copy link
Owner

That's weird, the crate has #![allow(stable_features,unused_features)]

@parasyte
Copy link
Contributor

I'll look at it when I get time this week.

@jethrogb
Copy link
Owner

Hmm I just tried it on nightly and indeed I can't use collections, str_char, unicode (alloc works though)

@jethrogb
Copy link
Owner

Behavior changed between:

  • nightly-2018-08-06 73c78734b
  • nightly-2018-08-14 d5a448b3f

@jethrogb
Copy link
Owner

Introduced in rust-lang/rust#52644

@jethrogb
Copy link
Owner

I guess we'll need to use the same version detection mechanism already in use for other features: https://github.com/jethrogb/rust-core_io/blob/master/build.rs#L52-L64

@parasyte
Copy link
Contributor

Cool! I didn't even know that existed. Sounds good, I should be able to bisect where all of the features needed changes. (I had to add/remove features about 3 different times IIRC)

@MathiasKoch
Copy link
Author

Any progress on this?

@parasyte
Copy link
Contributor

I started on this. It's going to take some time because I need to rebuild the patches.

bors bot added a commit that referenced this issue Jun 7, 2019
25: Fix features since nightly-2018-08-14 disallowed old features r=jethrogb a=parasyte

Fixes #20 

Co-authored-by: Jay Oster <[email protected]>
Co-authored-by: Jay Oster <[email protected]>
@bors bors bot closed this as completed in #25 Jun 7, 2019
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 a pull request may close this issue.

3 participants