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

cortex-m-rt v0.7.0 support #298

Closed
pdgilbert opened this issue Oct 28, 2021 · 5 comments · Fixed by #282
Closed

cortex-m-rt v0.7.0 support #298

pdgilbert opened this issue Oct 28, 2021 · 5 comments · Fixed by #282
Labels
bug Something isn't working

Comments

@pdgilbert
Copy link

(Related to #283)

I need cortex-m-rt v0.7.0 to avoid conflicts with other crates. The github version CHANGELOG.md is reporting

Support cortex-m-rt v0.7.0 but still allow v0.6.13 (#283)

but there seems to be a problem that stm32f3 is specified in Cargo.toml as

stm32f3 = { version = "0.13.2", default-features = false }

I think cortex-m-rt v0.7.0 support in stm32f3 is only introduced in 0.14. I get

error: failed to select a version for `cortex-m-rt`.
    ... required by package `stm32f3 v0.13.2`
    ... which is depended on by `stm32f3xx-hal v0.8.1 (https://github.com/stm32-rs/stm32f3xx-hal#91b47e12)`
    ... which is depended on by `rust-integration-testing-of-examples v0.1.0 (/home/paul/githubClones/rust-integration-testing/dev-testing)`
versions that meet the requirements `^0.6.10` are: 0.6.15

the package `cortex-m-rt` links to the native library `cortex-m-rt`, but it conflicts with a previous package which links to `cortex-m-rt` as well:
package `cortex-m-rt v0.7.0`
    ... which is depended on by `rust-integration-testing-of-examples v0.1.0 (/home/paul/githubClones/rust-integration-testing/dev-testing)`

failed to select a version for `cortex-m-rt` which could resolve this conflict
@Sh3Rm4n
Copy link
Member

Sh3Rm4n commented Oct 28, 2021

0.7 Support is currently only in master and not yet released: https://github.com/stm32-rs/stm32f3xx-hal/blob/master/CHANGELOG.md#added

Have you tried compiling with the current master?

Using the [patch] manifest, if you are using stm32f3xx-hal as a depency should be the easiest way to test it out.

@pdgilbert
Copy link
Author

I'm using the current master stm32f3xx-hal#91b47e12. Are you saying I need to patch too?

@pdgilbert
Copy link
Author

FWIW I tried github.com/Sh3Rm4n/stm32f3xx-hal?branch=pac-upgrade#9e59ee1ad280609c48e3361cc9f82a7ac6285c05 and all my tests are working. (At least all the ones that are not broken for other known reasons.) CI at https://github.com/pdgilbert/rust-integration-testing/actions

@Sh3Rm4n
Copy link
Member

Sh3Rm4n commented Nov 1, 2021

Ah finally I understand the problem. #283 is solving a problmen, which isn't to solve, because stm32f3 expects a fixed versino cortex-m-rt-0.7.0 for v0.14.0 and cortex-m-rt-0.6.x for v0.13.1? If than, yes #282 should fix the problem.

@Sh3Rm4n Sh3Rm4n added the bug Something isn't working label Nov 1, 2021
@pdgilbert
Copy link
Author

My guess is that is correct. (I'm still a newbie so it is just a guess.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants