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.

Sign up for GitHub

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

Jump to bottom

Remove limitation on clap version #28

Closed
pellico opened this issue Jun 14, 2024 · 1 comment · Fixed by #43
Closed

Remove limitation on clap version #28

pellico opened this issue Jun 14, 2024 · 1 comment · Fixed by #43
Assignees
@pellico
Milestone
Release 0.3.0

Comments

@pellico
Copy link
Contributor

pellico commented Jun 14, 2024

Problem
Presently clap version is capped to 4.4 to avoid test failures when suing Aurix Compiler.
clap > 4.4 requires rustc v 1.74 while rust aurix compiler toolchain includes rustc v 1.72

Solution

Toolchain used to compile tests of svd2pac shall be decoupled from toolchain used to check the generated pac.
@pellico pellico added this to the Release 0.3.0 milestone Jun 14, 2024
@pellico
Copy link
Contributor Author

pellico commented Jun 21, 2024

This issue will be resolved when rust-lang/cargo#9930 is implemented

pellico referenced this issue Sep 11, 2024
@jnkr-ifx
[tests] Fail on invalid cfg(feature) directives
72e037d 
The current implementation does not generate valid feature flags for
derived peripherals, but this was not caught by the tests because rustc
only emits a warning rather than an error.
@pellico pellico self-assigned this Sep 24, 2024
@pellico pellico linked a pull request Sep 24, 2024 that will close this issue
Removed limitation on clap version. #43
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pellico pellico

Labels
None yet
Projects
None yet
Milestone
Release 0.3.0
Development

Successfully merging a pull request may close this issue.

Removed limitation on clap version. Infineon/svd2pac
1 participant
@pellico