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

rtic-macros: fix #[cfg] for hardware and software tasks #894

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dalegaard
Copy link
Contributor

Hi,

Disabling hardware and software tasks via #[cfg] flags was broken. Added test case to verify, and fixed codegen to output missing cfgs.

Best regards,
Lasse

@dalegaard dalegaard force-pushed the fix_cfg_tasks branch 3 times, most recently from b5b0370 to 1f7e46e Compare February 24, 2024 12:35
@dalegaard dalegaard force-pushed the fix_cfg_tasks branch 2 times, most recently from c861b93 to afd506f Compare March 18, 2024 12:22
@dalegaard
Copy link
Contributor Author

Hi,

Updated this to work with changes in 2.1 release.

Best regards,
Lasse

Disabling hardware and software tasks via `#[cfg]` flags was broken.
Added test case to verify, and fixed codegen to output missing cfgs.
@dalegaard
Copy link
Contributor Author

Rebased to master.

Best regards,
Lasse

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