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

Rust: allow to specify more cargo configuration options #17937

Merged
merged 11 commits into from
Nov 18, 2024

Conversation

redsun82
Copy link
Contributor

@redsun82 redsun82 commented Nov 8, 2024

This allows to tweak via extractor options some aspects of the cargo configuration:

  • the target architecture
  • features (including * for all, which we must understand whether to set by default)
  • cfg overrides

Integration tests will be added in a follow-up commit.

This allows to tweak via extractor options some aspects of the cargo
configuration:
* the target architecture
* features (including `*` for all, which we must understand whether to
  set by default)
* cfg overrides

Integration tests will be added in a follow-up commit.
@redsun82 redsun82 requested a review from aibaars November 8, 2024 11:26
@github-actions github-actions bot added the Rust Pull requests that update Rust code label Nov 8, 2024
Base automatically changed from redsun82/rust-target-dir to main November 8, 2024 12:34
@redsun82 redsun82 marked this pull request as ready for review November 11, 2024 11:24
Copy link
Contributor

@aibaars aibaars left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@redsun82 redsun82 merged commit fa52d03 into main Nov 18, 2024
13 checks passed
@redsun82 redsun82 deleted the redsun82/rust-cargo-options branch November 18, 2024 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants