FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-108494: Build _testclinic_limited on Windows by vstinner · Pull Request #108589 · python/cpython · GitHub

Repository navigation

gh-108494: Build _testclinic_limited on Windows - #108589

Merged
vstinner merged 1 commit into
python:mainfrom
vstinner:ac_build_test_limited_win
Aug 28, 2023
Merged

vstinner merged 1 commit into
python:mainfrom
vstinner:ac_build_test_limited_win

Conversation

vstinner commented Aug 28, 2023 •
edited
Loading

Copy link
Copy Markdown
Member

Add _testclinic_limited project to the Visual Studio solution:

  • In PCbuild/, copy _asyncio.vcxproj to _testclinic_limited.vcxproj,
    replace RootNamespace with "_testclinic_limited".
  • In Visual Stuido, open PCbuild\pcbuild.sln, add the existing
    "_testclinic_limited" project to the solution.
  • Add a dependency from "python" project to "_testclinic_limited"
    project.
  • Save and exit VS.
  • Add "_testclinic_limited" to
    in PCbuild\pcbuild.proj.

vstinner force-pushed the ac_build_test_limited_win branch from 46c8d54 to 4c79ccb Compare August 28, 2023 23:20
vstinner changed the title gh-108494: AC: Build _testclinic_limited on Windows gh-108494: Build _testclinic_limited on Windows Aug 28, 2023
vstinner marked this pull request as ready for review August 28, 2023 23:21
vstinner requested a review from a team as a code owner August 28, 2023 23:21
Add _testclinic_limited project to the Visual Studio solution:

* In PCbuild/, copy _asyncio.vcxproj to _testclinic_limited.vcxproj,
  replace RootNamespace with "_testclinic_limited".
* In Visual Stuido, open PCbuild\pcbuild.sln, add the existing
  "_testclinic_limited" project to the solution.
* Add a dependency from "python" project to "_testclinic_limited"
  project.
* Save and exit VS.
* Add "_testclinic_limited" to <TestModules Include="...">
  in PCbuild\pcbuild.proj.
vstinner force-pushed the ac_build_test_limited_win branch from 4c79ccb to 07d21ec Compare August 28, 2023 23:26

Copy link
Copy Markdown
Member Author

The _testclinic_limited is now build as expected on Windows. Logs of the Windows x64 GHA CI:

  _testclinic_limited.c
     Creating library D:\a\cpython\cpython\PCbuild\amd64\_testclinic_limited_d.lib and object D:\a\cpython\cpython\PCbuild\amd64\_testclinic_limited_d.exp
  _testclinic_limited.vcxproj -> D:\a\cpython\cpython\PCbuild\amd64\_testclinic_limited_d.pyd

vstinner enabled auto-merge (squash) August 28, 2023 23:33
vstinner merged commit 5c68cba into python:main Aug 28, 2023
vstinner deleted the ac_build_test_limited_win branch August 28, 2023 23:56
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to 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


Back | FazBrowse Home | New Git URL