Skip to content
This repository was archived by the owner on Aug 25, 2025. It is now read-only.

Conversation

@0x7CFE
Copy link
Contributor

@0x7CFE 0x7CFE commented Jun 13, 2018

Recent changes in nightly broke wee_alloc build. In an attempt to solve the issues I prepared the following PR.

Unfortunately, not all changes are resolved at the moment, so I would really appreciate any suggestions and comments on the remaining issues.

Currently there are three of them:

  • definition of unknown language item panic_fmt at example/src/lib.rs:21:1
  • duplicate lang item found: oom at example/src/lib.rs:31:1
  • duplicate lang item found: eh_personality at example/src/lib.rs:39:1

Aside of that, the rest of the code compiles successfully.

@@ -1,5 +1,5 @@
use const_init::ConstInit;
use core::alloc::{AllocErr, Opaque};
use core::alloc::{AllocErr, u8};
Copy link
Contributor Author

@0x7CFE 0x7CFE Jun 13, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, that was too verbose :) Gonna fix that in a sec.

@ZackPierce
Copy link
Collaborator

For the first issue, I would refer to https://users.rust-lang.org/t/psa-breaking-change-panic-fmt-language-item-removed-in-favor-of-panic-implementation/17875 , regarding the switch from panic_fmt to panic_implementation. Seems like a relatively straightforward change.

@0x7CFE
Copy link
Contributor Author

0x7CFE commented Jun 13, 2018

Seems like a relatively straightforward change.

@ZackPierce, thank you for the link. I think I can handle that.

@0x7CFE
Copy link
Contributor Author

0x7CFE commented Jun 13, 2018

@ZackPierce, I've fixed all issues and now ./build.sh seems to compile without any warnings or errors.

As for the remaining two issues, looks like it was me accidentally running normal cargo +nightly check instead of ./build.sh :)

Copy link
Member

@pepyakin pepyakin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@ZackPierce ZackPierce self-requested a review June 13, 2018 17:06
Copy link
Collaborator

@ZackPierce ZackPierce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@pepyakin pepyakin merged commit 4cfd38a into rustwasm:master Jun 13, 2018
@0x7CFE 0x7CFE deleted the dk-alloc-fix branch June 14, 2018 04:35
0x7CFE added a commit to paritytech/substrate that referenced this pull request Jun 15, 2018
Revision 4e9f23f points to a master after rustwasm/wee_alloc#45 was merged
rphmeier pushed a commit to paritytech/polkadot that referenced this pull request Aug 14, 2018
Revision 4e9f23f points to a master after rustwasm/wee_alloc#45 was merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants