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

Explicitly use edition = "2015" #2259

Closed
wants to merge 2 commits into from
Closed

Explicitly use edition = "2015" #2259

wants to merge 2 commits into from

Conversation

Kijewski
Copy link

This way cargo package won't implictily insert a resolver = "1" configuration. This fixes the inadversial MSRV bump mentioned in issue #2255.

This way `cargo package` won't implictily insert a `resolver = "1"`.
configuration. This fixes the inadversial MSRV bump mentioned in issue
[#2255].

[#2255]: #2255
@Kijewski
Copy link
Author

Oh, well, this way rustc 1.27 ..= 1.30 break. But that would be a smaller MSRV bump than 1.51. What do y'all think? Is this an acceptable solution?

@HamzaYasin1
Copy link

Now it started giving this error:

          error[E0432]: unresolved import `wasm_builder_runner`
           --> runtime/build.rs:1:5
            |
          1 | use wasm_builder_runner::WasmBuilder;
            |     ^^^^^^^^^^^^^^^^^^^ maybe a missing crate `wasm_builder_runner`?
          
          error: aborting due to previous error

@Kijewski
Copy link
Author

Kijewski commented Aug 12, 2022

Thank you for testing this, @HamzaYasin1! I don't know how to run cargo for older wasm32-unknown-unknown or wasm32-wasi targets. Can to paste needed rustup toolchain add … and/or rustup target add … commands for me?

Also, serde does not seem to contain any runtime folder in the whole project?

@HamzaYasin1
Copy link

When you run command , so it will show you all the installed version that you can use.
rustup show

    Default host: x86_64-apple-darwin
    rustup home:  /Users/algocoder/Applications/.rustup
    
    installed toolchains
    --------------------
    
    stable-x86_64-apple-darwin
    nightly-2020-08-19-x86_64-apple-darwin
    nightly-2020-10-05-x86_64-apple-darwin
    nightly-2021-11-12-x86_64-apple-darwin
    nightly-x86_64-apple-darwin (default)
    1.48-x86_64-apple-darwin
    1.45.1-x86_64-apple-darwin
    
    installed targets for active toolchain
    --------------------------------------
    
    wasm32-unknown-unknown
    x86_64-apple-darwin
    
    active toolchain
    ----------------
    
    nightly-x86_64-apple-darwin (default)
    rustc 1.65.0-nightly (20ffea693 2022-08-11)

You can see this link for toolchain changes
switch between Rust toolchains

Yes Serde doesn't contain any runtime folder as of my knowledge.

@Kijewski
Copy link
Author

Well, runtime/build.rs is missing an extern crate wasm_builder_runner; line. But the error seems unrelated to me?

@Mingun
Copy link
Contributor

Mingun commented Aug 12, 2022

Probably that code was built in 2018 edition and now it is build in 2015 edition.

@Kijewski
Copy link
Author

Probably that code was built in 2018 edition and now it is build in 2015 edition.

But what code? :) Afaik edition is not "viral". I.e. if a dependency has an older edition, it should not affect me. Or does it?

@taiki-e
Copy link
Contributor

taiki-e commented Aug 12, 2022

Considering how rust-lang/cargo#10954 was fixed, this should not be necessary and the problem should be automatically resolved when the maintainer creates a new release using the nightly toolchain after nightly-2022-08-13 (tomorrow).

@Kijewski
Copy link
Author

@taiki-e, ah, I didn't see the rust-lang/cargo#10954. Yes, that's the (only) correct solution.

@Kijewski Kijewski closed this Aug 12, 2022
@Kijewski Kijewski deleted the pr-issue-2255 branch August 12, 2022 21:03
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