ENH: Bump for ITK v5.3rc04.post4 by tbirdso · Pull Request #9 · InsightSoftwareConsortium/ITKRemoteModuleBuildTestPackageAction · GitHub
Skip to content

ENH: Bump for ITK v5.3rc04.post4#9

Merged
tbirdso merged 4 commits into
mainfrom
bump-itk
Nov 18, 2022
Merged

ENH: Bump for ITK v5.3rc04.post4#9
tbirdso merged 4 commits into
mainfrom
bump-itk

Conversation

@tbirdso

@tbirdso tbirdso commented Nov 15, 2022

Copy link
Copy Markdown
Collaborator

Pending successful ITKSplitComponents test workflow run

https://github.com/InsightSoftwareConsortium/ITKSplitComponents/actions/runs/3472832027

@tbirdso tbirdso requested review from dzenanz and thewtex November 15, 2022 17:43
@tbirdso

tbirdso commented Nov 16, 2022

Copy link
Copy Markdown
Collaborator Author

@dzenanz

dzenanz commented Nov 16, 2022

Copy link
Copy Markdown
Member

I have not seen this issue before. By a quick search, it does not seem to be common.

@tbirdso

tbirdso commented Nov 16, 2022

Copy link
Copy Markdown
Collaborator Author

Perhaps some mismatch between MSVC versions? Or old ITK build artifacts were somehow swept up in the build?

@thewtex Would you please provide specs on the Windows machine used to build Windows v5.3rc04.post4 release artifacts?

@thewtex

thewtex commented Nov 16, 2022

Copy link
Copy Markdown
Member

@tbirdso yes, it is likely compiler versions -- I created the v5.3rc04.post4 on a system with VS 2022 -- the previous were created on VS 2019 (this is not the same as the MSVC toolchain version, but likely an earlier version). @tbirdso could you please try windows-2022 actions runner: https://github.com/actions/runner-images/blob/main/images/win/Windows2022-Readme.md

Resolves compiler mismatch issue where ITK v5.3rc04.post4 build archives
were built with VS22 compiler
Addresses Windows cxx build warnings due to MSVC discovery issues that
were fixed in CMake v3.21.3. The closest available tag for the
`lukka/get-cmake` action is v3.22.2, which also updates `ninja` to
1.10.2.

https://gitlab.kitware.com/cmake/cmake/-/merge_requests/6497

https://github.com/lukka/get-cmake/releases/tag/v3.22.2
@tbirdso tbirdso marked this pull request as ready for review November 18, 2022 01:15
@tbirdso

tbirdso commented Nov 18, 2022

Copy link
Copy Markdown
Collaborator Author

@tbirdso tbirdso merged commit 29ad46f into main Nov 18, 2022
tbirdso added a commit to InsightSoftwareConsortium/ITKModuleTemplate that referenced this pull request Nov 18, 2022
Updates cookiecutter CI workflow to rely on
ITKRemoteModuleBuildTestPackageAction reusable workflows for easier
maintenance.

Updates ITK reference to v5.3rc04.post4 and applies version updates
from reusable workflow.

See InsightSoftwareConsortium/ITKRemoteModuleBuildTestPackageAction#9
@thewtex thewtex deleted the bump-itk branch November 18, 2022 16:08
jhlegarreta pushed a commit to jhlegarreta/ITKModuleTemplate that referenced this pull request Jul 2, 2024
Updates cookiecutter CI workflow to rely on
ITKRemoteModuleBuildTestPackageAction reusable workflows for easier
maintenance.

Updates ITK reference to v5.3rc04.post4 and applies version updates
from reusable workflow.

See InsightSoftwareConsortium/ITKRemoteModuleBuildTestPackageAction#9
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.

3 participants