Skip to content

Conversation

tbirdso
Copy link
Contributor

@tbirdso tbirdso commented Dec 12, 2022

Separates step for pulling build artifacts into a standalone script that is called by the central download-and-build script. The modular approach allows ITK modules with more complex build steps (OpenCL, etc) to use the download script directly instead of forking and altering the download script, thus reducing the maintenance burden for those ITK modules.

Local test has passed, will test on ITKSplitComponents before merge.

EDIT: Testing at https://github.com/InsightSoftwareConsortium/ITKSplitComponents/actions/runs/3676857336

Separates step for pulling build artifacts into a standalone script that
is called by the central download-and-build script. The modular approach
allows ITK modules with more complex build steps (OpenCL, etc) to use the
download script directly instead of forking and altering the download
script, thus reducing the maintenance burden for those ITK modules.
@tbirdso tbirdso requested review from thewtex and dzenanz December 12, 2022 14:42
@tbirdso
Copy link
Contributor Author

tbirdso commented Dec 12, 2022

This change is in support of InsightSoftwareConsortium/ITKCLEsperanto#50 which previously forked ITKPythonPackage's manylinux download script.

@tbirdso
Copy link
Contributor Author

tbirdso commented Dec 12, 2022

ITKSplitComponents Linux tests passed: https://github.com/InsightSoftwareConsortium/ITKSplitComponents/actions/runs/3676857336/attempts/1

Windows failures are unrelated as this PR solely deals with Linux scripts. I believe failures are due to these changes being in my fork at tbirdso/ITKPythonPackage.

@tbirdso tbirdso merged commit 2df8737 into InsightSoftwareConsortium:master Dec 12, 2022
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.

2 participants