GH-48947 [CI][Python] Install pymanager.msi instead of pymanager.msix to fix docker rebuild on Windows wheels #48948
Conversation
|
|
|
@github-actions crossbow submit wheel-windows-cp314-cp314-amd64 |
|
Revision: 367321b Submitted crossbow builds: ursacomputing/crossbow @ actions-34019f34e1
|
|
@github-actions crossbow submit wheel-windows-cp314-cp314-amd64 |
|
Revision: 3f780d1 Submitted crossbow builds: ursacomputing/crossbow @ actions-040b1e8dd6
|
|
@github-actions crossbow submit wheel-windows-cp314-cp314-amd64 |
|
Revision: 62b1e05 Submitted crossbow builds: ursacomputing/crossbow @ actions-e948eca8ff
|
|
@github-actions crossbow submit wheel-windows-cp314-cp314-amd64 |
|
Revision: 9bb6af8 Submitted crossbow builds: ursacomputing/crossbow @ actions-a9e694da5d
|
github-actions
bot
added
awaiting changes
|
@github-actions crossbow submit wheel-windows-cp314-cp314-amd64 |
github-actions
bot
added
awaiting change review
|
Revision: ec82969 Submitted crossbow builds: ursacomputing/crossbow @ actions-41b004ef76
|
|
@github-actions crossbow submit wheel-windows-cp314-cp314-amd64 |
|
Revision: 6a253a0 Submitted crossbow builds: ursacomputing/crossbow @ actions-69aeada46a
|
|
@github-actions crossbow submit wheel-windows-cp314* |
|
Revision: 986f43c Submitted crossbow builds: ursacomputing/crossbow @ actions-bdf62292ce
|
Yes, I had issues with installing the manager for the test step though it worked for the build step, see: The only difference when working on the linked PR has been the docker image (different for build and test step). |
|
@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64 |
|
Revision: 416297b Submitted crossbow builds: ursacomputing/crossbow @ actions-dbcf48b598
|
|
@github-actions crossbow submit wheel-windows-* |
|
Revision: 416297b Submitted crossbow builds: ursacomputing/crossbow @ actions-c79f074728
|
raulcd
changed the title
|
@AlenkaF @pitrou this PR is only to fix the ability of rebuilding our wheels which I need for the meson-python work I am doing. I am also working on top of this PR on a different one to reduce the amout of dockerfiles. This will reduce the current 5 dockerfiles to 3 (1 base, 1 build, 1 test) instead of (2 base, 1 build, 2 test) but this is a different issue. |
|
After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit 790ed2c. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. It also includes information about 4 possible false positives for unstable benchmarks that are known to sometimes produce them. |
|
After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit 790ed2c. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. It also includes information about 4 possible false positives for unstable benchmarks that are known to sometimes produce them. |
3 participants
Rationale for this change
As soon as we have to rebuild our Windows docker images they will fail installing python-manager-25.0.msix
What changes are included in this PR?
pymanager.msito install python version instead ofpymanager.msixwhich has problems on Docker.pymanager installcommand to use newer API (old command fails with missing flags)Are these changes tested?
Yes via archery
Are there any user-facing changes?
No