Skip to content

Conversation

@MabezDev
Copy link
Member

This is temporary measure, as the problem cannot be solved cleanly right now.

The issue is that the msrv check uses the stable compiler, which uses a stable cargo. With a stable cargo, the unstable build-std option is not respected within .cargo/config.toml. This means core is never rebuilt with the atomic cfg flags so we get this error when building log version 0.4.19. The 0.4.19 release uses the atomic cfg flags instead of a custom build script, so by switching back to 0.4.18 we can avoid this issue... for now at least.

@MabezDev MabezDev marked this pull request as ready for review June 20, 2023 12:43
@MabezDev MabezDev force-pushed the pin-log branch 2 times, most recently from 835ceab to 2f74819 Compare June 20, 2023 12:55
This is temporary measure, as the problem cannot be solved cleanly right
now.

The issue is that the msrv check uses the stable compiler, which uses a
stable cargo. With a stable cargo, the unstable `build-std` option is
not respected within `.cargo/config.toml`. This means `core` is never
rebuilt with the atomic cfg flags so we get this error when building log
version 0.4.19. The 0.4.19 release uses the atomic cfg flags instead of
a custom build script, so by switching back to 0.4.188888888 we can avoid this
issue... for now at least.
Copy link
Contributor

@bjoernQ bjoernQ left a comment

Choose a reason for hiding this comment

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

LGTM

@jessebraham jessebraham merged commit bd2f0fe into main Jun 20, 2023
@jessebraham jessebraham deleted the pin-log branch August 24, 2023 13:35
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.

4 participants