Skip to content

Conversation

@jacob-hughes
Copy link
Collaborator

This removes the dependency for allocation metrics in libstd

@ltratt ltratt added this pull request to the merge queue Jun 13, 2025
github-merge-queue bot pushed a commit that referenced this pull request Jun 13, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 13, 2025
This removes the dependency for allocation metrics in libstd
@jacob-hughes
Copy link
Collaborator Author

Force-pushed a fix to .builbot.sh which was using an out-of-date config name.

@ltratt ltratt added this pull request to the merge queue Jun 13, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 13, 2025
@jacob-hughes
Copy link
Collaborator Author

Oops. Ive broken the stats generation for allocation with this. Investigating..

@jacob-hughes
Copy link
Collaborator Author

I think I've had to throw my hands up and admit defeat here. See 58b7593 for an explanation. If you're happy, I won't squash it and it can be merged as-is.

@ltratt ltratt added this pull request to the merge queue Jun 13, 2025
@ltratt
Copy link
Member

ltratt commented Jun 13, 2025

Agreed.

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 13, 2025
These tests check that rustc works with various options that we don't
claim to support (e.g. no_rc, no_oom_handling). These are linked with a
bare rustc instance and not cargo which fundamentally does not work with
the way that we build Alloy.

Rather than litter lib/alloc with conditional compilation, I think it's
best to disable them for now.
@jacob-hughes
Copy link
Collaborator Author

Force-pushed a fix

@ltratt ltratt added this pull request to the merge queue Jun 16, 2025
Merged via the queue into softdevteam:master with commit 653904b Jun 16, 2025
2 checks passed
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