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

Add rustflags in cargo config #543

Merged
merged 4 commits into from
Jul 22, 2024
Merged

Add rustflags in cargo config #543

merged 4 commits into from
Jul 22, 2024

Conversation

b-yap
Copy link
Contributor

@b-yap b-yap commented Jul 22, 2024

Defines the rustflags directly in .cargo/config.toml file.
This is required for tokio-console. https://github.com/pendulum-chain/tasks/issues/343

@b-yap b-yap requested a review from ebma July 22, 2024 14:42
Comment on lines 1 to 2
[env]
CARGO_BUILD_RUSTFLAGS="--cfg tokio_unstable"
Copy link
Member

Choose a reason for hiding this comment

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

Why are we not doing it like this? What's the difference?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh my god, good catch.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it'll behave just the same.
But yeah, I'll let it look similar to https://github.com/tokio-rs/console?tab=readme-ov-file#instrumenting-your-program

Copy link
Member

Choose a reason for hiding this comment

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

Nice, can confirm that the compilation works for me with and without the allow-debugger feature.

Copy link
Member

@ebma ebma left a comment

Choose a reason for hiding this comment

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

LGTM, let's merge.

@ebma ebma merged commit a70480c into main Jul 22, 2024
1 of 2 checks passed
@ebma ebma deleted the add-rustflags-in-cargo-config branch July 22, 2024 15:02
gianfra-t pushed a commit that referenced this pull request Jul 24, 2024
* add rustflag in cargo config

* add documentation of where the rustflags are set.

* #543 (comment)

* #543 (comment)
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.

2 participants