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

bs58 crate fails to build tests in 1.28 beta #51845

Closed
pietroalbini opened this issue Jun 27, 2018 · 3 comments
Closed

bs58 crate fails to build tests in 1.28 beta #51845

pietroalbini opened this issue Jun 27, 2018 · 3 comments
Labels
regression-from-stable-to-beta Performance or correctness regression from stable to beta.
Milestone

Comments

@pietroalbini
Copy link
Member

That test code doesn't make sense to me... but it was working in 1.27 somehow.

@pietroalbini pietroalbini added the regression-from-stable-to-beta Performance or correctness regression from stable to beta. label Jun 27, 2018
@pietroalbini pietroalbini added this to the 1.28 milestone Jun 27, 2018
@Nemo157
Copy link
Member

Nemo157 commented Jun 27, 2018

::decode is both a module and a function.

Throwing together a test playground indicates that the issue could be #38260, but if so I'm not sure why it was building on 1.27. I'm installing 1.27 locally right now to check.

@Nemo157
Copy link
Member

Nemo157 commented Jun 27, 2018

Yeah, I get the same error when running the tests with 1.27 locally:

error: `self` no longer imports values
   --> bs58-rs/src/decode.rs:150:19
    |
150 |     use decode::{ self, DecodeError };
    |                   ^^^^
    |
    = note: #[deny(legacy_imports)] on by default
    = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
    = note: for more information, see issue #38260 <https://github.com/rust-lang/rust/issues/38260>

Guess I need to fix that.

@pietroalbini
Copy link
Member Author

::decode is both a module and a function.

Oh, I didn't check in the lib.rs file. That explains it.

Thanks for looking into this @Nemo157! I'm closing the issue as won't fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression-from-stable-to-beta Performance or correctness regression from stable to beta.
Projects
None yet
Development

No branches or pull requests

2 participants