bpo-47103: Copy pgort140.dll into output directory when building PGInstrument on Windows#32083
Conversation
|
The following 2 tests in test_embed.py failed with the same error: test_init_pybuilddir_win32 |
| @@ -132,9 +132,6 @@ set PYTHONPATH=$(PySourcePath)Lib | |||
| </Target> | |||
| <Target Name="GeneratePythonBat" AfterTargets="AfterBuild"> | |||
| <PropertyGroup> | |||
There was a problem hiding this comment.
| <PropertyGroup> | |
| <PropertyGroup> | |
| <_PGOPath Condition="$(Configuration) == 'PGInstrument'">@set PATH=%PATH%%3B$(OutDir)</_PGOPath> |
There was a problem hiding this comment.
Leaving $(_PGOPath) like this fixes my errors. The way is more generic than #32146.
There was a problem hiding this comment.
No, your other PR is better. We shouldn't rely on PATH for resolving DLLs, but should copy the right one alongside the executable.
There was a problem hiding this comment.
But I see you've closed it, so I'll add a similar change to this PR.
There was a problem hiding this comment.
The additional change is perfect for PGO/non-PGO.
I can confirm PGInstrument builds work with this.
|
Thanks @zooba for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9. |
|
Thanks @zooba for the PR 🌮🎉.. I'm working now to backport this PR to: 3.10. |
|
Sorry, @zooba, I could not cleanly backport this to |
|
Sorry @zooba, I had trouble checking out the |
…strument on Windows (pythonGH-32083)
|
GH-32366 is a backport of this pull request to the 3.10 branch. |
|
GH-32407 is a backport of this pull request to the 3.9 branch. |
…strument on Windows (pythonGH-32083)
https://bugs.python.org/issue47103