Skip to content

Conversation

tbirdso
Copy link
Collaborator

@tbirdso tbirdso commented Dec 13, 2022

Replaces inline Command Prompt build setup procedures in favor of ITKPythonPackage PowerShell script.

Centralizing the Windows Python build procedures in an ITKPythonPackage script will improve ease of maintenance and allow the process to be more easily replicated on a local system for testing.

Testing in ITKSplitComponents with accompanying ITKPythonPackage changes has succeeded: https://github.com/tbirdso/ITKSplitComponents/actions/runs/3685964482/jobs/6237566090

Depends on changes in ITKPythonPackage: InsightSoftwareConsortium/ITKPythonPackage#242

@tbirdso tbirdso requested review from dzenanz and thewtex December 13, 2022 14:23
Replaces inline Command Prompt build setup procedures in favor of
ITKPythonPackage PowerShell script.

Centralizing the Windows Python build procedures in an ITKPythonPackage
script will improve ease of maintenance and allow the process to be more
easily replicated on a local system for testing.
@tbirdso tbirdso force-pushed the use-windows-download-script branch from 56b882a to 6ad6d59 Compare December 13, 2022 15:13
@tbirdso
Copy link
Collaborator Author

tbirdso commented Dec 13, 2022

Updated itk-python-package-tag to reference merge commit. Will validate one more time in ITKSplitComponents before merge.

EDIT: Testing at https://github.com/tbirdso/ITKSplitComponents/actions/runs/3686731306/jobs/6239370043

EDIT2: Windows Python packages are passing 🟢

@tbirdso tbirdso force-pushed the use-windows-download-script branch from baf56f9 to 56b882a Compare December 13, 2022 15:14
@tbirdso tbirdso marked this pull request as ready for review December 13, 2022 15:49
@tbirdso tbirdso merged commit d5cb291 into main Dec 13, 2022
@jhlegarreta jhlegarreta deleted the use-windows-download-script branch December 13, 2022 17:59
@tbirdso tbirdso linked an issue Dec 14, 2022 that may be closed by this pull request
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.

Use Windows ITKPythonPackage Download Script

2 participants