Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ENH: Update CMake to decouple archives from example builds #399

Merged

Conversation

tbirdso
Copy link
Contributor

@tbirdso tbirdso commented Nov 10, 2022

Addresses issue where archives were not build for examples without an add_example() call, resulting in broken download links on examples.itk.org.

New behavior packages example archives for all examples to accompany Sphinx documentation. Also cleans up CMakeLists.txt for several module groups.

Closes #373.

@github-actions github-actions bot added area:Bridge Issues affecting the Bridge module area:Core Issues affecting the Core module area:Filtering Issues affecting the Filtering module area:IO Issues affecting the IO module area:Nonunit Issues affecting the Nonunit module area:Numerics Issues affecting the Numerics module area:Registration Issues affecting the Registration module area:Remotes Issues affecting the Remote module area:Segmentation Issues affecting the Segmentation module area:Video Issues affecting the Video module type:Enhancement Improvement of existing methods or implementation type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots labels Nov 10, 2022
Copy link
Member

@dzenanz dzenanz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have not tested anything here. Code looks good.

@tbirdso
Copy link
Contributor Author

tbirdso commented Nov 11, 2022

Seeing repeated failures due to SHA256 resource download failures, which are in the process of being addressed with #398.

https://open.cdash.org/viewBuildError.php?buildid=8270992

Documentation builds on my Ubuntu machine and I have verified that archives are generated based on folder structure to address #373. @thewtex Would you prefer that I wait and rebase once #398 is stable or go ahead and merge?

Copy link
Member

@thewtex thewtex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks @tbirdso !

We can merge if we can get a green build

src/Filtering/CMakeLists.txt Show resolved Hide resolved
src/Filtering/CMakeLists.txt Outdated Show resolved Hide resolved
@github-actions github-actions bot temporarily deployed to production November 11, 2022 15:32 Inactive
Addresses issue where archives were not build for examples without an
`add_example()` call, resulting in broken download links on
`examples.itk.org`.

New behavior packages example archives for all examples to accompany
Sphinx documentation. Also cleans up `CMakeLists.txt` for several module
groups.
@tbirdso tbirdso merged commit 4ffdcb9 into InsightSoftwareConsortium:master Nov 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:Bridge Issues affecting the Bridge module area:Core Issues affecting the Core module area:Filtering Issues affecting the Filtering module area:IO Issues affecting the IO module area:Nonunit Issues affecting the Nonunit module area:Numerics Issues affecting the Numerics module area:Registration Issues affecting the Registration module area:Remotes Issues affecting the Remote module area:Segmentation Issues affecting the Segmentation module area:Video Issues affecting the Video module type:Enhancement Improvement of existing methods or implementation type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: Example zip archive downloads are absent
3 participants