Skip to content

Trying to fix CI failure#231

Merged
pacak merged 1 commit into
masterfrom
ci-exp
Dec 29, 2023
Merged

Trying to fix CI failure#231
pacak merged 1 commit into
masterfrom
ci-exp

Conversation

@pacak
Copy link
Copy Markdown
Owner

@pacak pacak commented Dec 28, 2023

The culprit lied in rust-lang/rust#116505

In short, with default features turned off, main was trivial enough to
be marked as inline function automatically which then made the symbol
weak. Since nobody was referencing it, it got stripped away.

Marking main in default-features=false config as #[inline(never)] or
replacing 1 + 1 in it's body with a simple println!("foo") call (to make
the main function sophisticated enough not to be subject to the new
automatic marking as inline) makes the test pass again.

@pacak pacak force-pushed the ci-exp branch 2 times, most recently from 58e8e65 to 8d5edea Compare December 29, 2023 01:22
The culprit lied in rust-lang/rust#116505

In short, with default features turned off, main was trivial enough to
be marked as inline function automatically which then made the symbol
weak. Since nobody was referencing it, it got stripped away.

Marking main in default-features=false config as #[inline(never)] or
replacing 1 + 1 in it's body with a simple println!("foo") call (to make
the main function sophisticated enough not to be subject to the new
automatic marking as inline) makes the test pass again.
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 this pull request may close these issues.

1 participant