Skip to content

internal: No std testing - #6257

Queued
Person-93 wants to merge 6 commits into
PyO3:mainfrom
Person-93:no_std_testing
Queued

internal: No std testing#6257
Person-93 wants to merge 6 commits into
PyO3:mainfrom
Person-93:no_std_testing

Conversation

@Person-93

@Person-93 Person-93 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

This is part of the WIP no_std support.

I am not yet working on CI as that will need a separate discussion so that maintainers can decide how it should be handled. For now, I want to locally run something like this:

PYO3_WIP_NO_STD=1 nox -s test-rust

I've tried making the noxfile aware of the features required for no_std builds, but I am not sure I've got it quite right.

@Person-93 Person-93 changed the title No std testing internal: No std testing Jul 30, 2026
@Person-93
Person-93 marked this pull request as ready for review July 30, 2026 04:30
@Person-93
Person-93 marked this pull request as draft July 30, 2026 05:14
@Person-93
Person-93 marked this pull request as ready for review July 30, 2026 05:27

@davidhewitt davidhewitt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, looks correct to me!

@davidhewitt
davidhewitt added this pull request to the merge queue Aug 5, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 5, 2026
@Person-93

Copy link
Copy Markdown
Contributor Author

I do now know why these changes cause the ffi-check to fail on some platforms. Please help.

@davidhewitt

Copy link
Copy Markdown
Member

#6302 should have fixed, plz merge main / rebase

@Person-93

Copy link
Copy Markdown
Contributor Author

#6302 should have fixed, plz merge main / rebase

@davidhewitt done

@davidhewitt
davidhewitt added this pull request to the merge queue Aug 7, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 7, 2026
@Person-93

Copy link
Copy Markdown
Contributor Author

@davidhewitt I think that was a spurious issue with cargo-careful.

@davidhewitt
davidhewitt added this pull request to the merge queue Aug 7, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 7, 2026
@Person-93

Copy link
Copy Markdown
Contributor Author

@davidhewitt, it looks like it's not a spurious issue, but for some reason it's not finding the std that cargo-careful compiled for it.

@davidhewitt

Copy link
Copy Markdown
Member

Will attempt to debug later

@davidhewitt

Copy link
Copy Markdown
Member

Hoping queuing behind #6310 now lands this...

@davidhewitt
davidhewitt added this pull request to the merge queue Aug 8, 2026
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