Skip to content

Conversation

@neonene
Copy link
Contributor

@neoneneneonene commented Jan 27, 2025

This reverts the temporal commit b00e125.

The issue with test_init_pyvenv_cfg failing is closed as resolved.

cc @ned-deily

@vstinner
Copy link
Member

test_embed is slow and mostly spawns sub-processes. I'm not convinced that it's an useful tests to train the PGO build.

cc @pablogsal@corona10@markshannon

@neonene
Copy link
ContributorAuthor

I'd prefer a more explicit announcement if test_embed will not be used in PGO. Also, I'd prefer older versions (3.10 to 3.13) to have the same conditions.

@vstinner
Copy link
Member

I'd prefer a more explicit announcement if test_embed will not be used in PGO. Also, I'd prefer older versions (3.10 to 3.13) to have the same conditions.

I suggest to also remove test_embed from 3.12 and 3.13 in this case, and document the change in the Build category.

@neonene
Copy link
ContributorAuthor

neonene commented Feb 4, 2025

I suggest to also remove test_embed from 3.12 and 3.13 in this case, and document the change in the Build category.

I have opened #129660.

@neoneneneonene closed this Feb 4, 2025
@neoneneneonene deleted the restore_embed branch February 5, 2025 15:25
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@neonene@vstinner