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

update to avr-device 0.7 #618

Merged
merged 1 commit into from
Jan 5, 2025
Merged

update to avr-device 0.7 #618

merged 1 commit into from
Jan 5, 2025

Conversation

tones111
Copy link
Contributor

I'm not sure why these weren't updated with the release of avr-device 0.6. Since everything depending on these HALs need to use the same avr-device version it seems like a good idea to keep them synced.

Is there a good method to test this type of change locally? Trying to run cargo check on the main branch results in a lot of errors.

@stappersg
Copy link
Contributor

stappersg commented Jan 1, 2025 via email

@stappersg
Copy link
Contributor

stappersg commented Jan 1, 2025 via email

@tones111 tones111 mentioned this pull request Jan 1, 2025
@innermatrix
Copy link
Contributor

@tones111 FWIW I use this to bulk-compile all examples during development:

find examples -name Cargo.toml -execdir cargo build --bins \;

(Yes, we should add that to CI.)

@Rahix
Copy link
Owner

Rahix commented Jan 4, 2025

@tones111, the reason I held off upgrading to avr-device 0.6 so far was the large amount of pending change in that project which push for yet another breaking release soon (== avr-device v0.7). So I intended to skip 0.6 for the time being. The 0.6 release also doesn't include much of value for avr-hal anyway.

@Rahix
Copy link
Owner

Rahix commented Jan 4, 2025

Is there a good method to test this type of change locally?

I usually just let CI take care of this, to be honest... That's the most dependable result one can get so I prefer it to attempting local builds. Otherwise, the suggestion from @innermatrix doesn't seem too bad.

(Yes, we should add that to CI.)

CI is already doing a matrix build of all the example crates and for MCUs without examples, it does a simple build-test of the HAL crate with the right features.

@innermatrix
Copy link
Contributor

The 0.6 release also doesn't include much of value for avr-hal anyway.

@Rahix Well, it doesn't right now, but one of the reasons that I am adding ATtiny xmega support to avr-device is that I am also working on supporting those in avr-hal. Then again, that is gated on the conversation in #623 and my changes in #605 so I agree that there is no hurry to update avr-device until we make progress on those.

@tones111
Copy link
Contributor Author

tones111 commented Jan 4, 2025

My motivation for this update was to allow downstream users of the HAL crates to use newer avr-device versions. Without it cargo ends up trying to link in both 0.5 and 0.6 which errors at link time due to duplicate symbols. I'll update this to depend on 0.7.

update: re-push once 0.7 has been published

@tones111 tones111 changed the title update to avr-device 0.6 update to avr-device 0.7 Jan 4, 2025
Copy link
Owner

@Rahix Rahix left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

@Rahix Rahix merged commit 3c08979 into Rahix:main Jan 5, 2025
25 checks passed
@tones111 tones111 deleted the avr_device_06 branch January 5, 2025 03:27
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