Skip to content

Conversation

@robertoaloi
Copy link
Contributor

@robertoaloi robertoaloi commented Jan 21, 2026

The dependency is only used as part of the profile functionality. Bump it to the latest available version.

The build was failing after the bump for certain architectures. These seems due to the false sense of cross-platform support in the previous version of the perf-event-open-sys dependency. The new version refuses to compile on unsupported platforms, where the perf structs would be incorrect. Therefore I am also restricting compilation to the supported architectures.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 21, 2026
@robertoaloi
Copy link
Contributor Author

Checking the CI failure.

@robertoaloi robertoaloi marked this pull request as draft January 21, 2026 12:00
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 21, 2026
@robertoaloi robertoaloi marked this pull request as ready for review January 21, 2026 12:37
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants