InsightSoftwareConsortium / ITKModuleTemplate

A template to start an ITK Module
https://itk.org/ITKSoftwareGuide/html/Book1/ITKSoftwareGuide-Book1ch9.html#x50-1430009
Apache License 2.0
12 stars 16 forks source link

ENH: Update CI build OS versions #153

Closed jhlegarreta closed 1 month ago

jhlegarreta commented 8 months ago

Update CI build OS versions: transition Ubuntu to 22.04 and macOS to 12.

dzenanz commented 5 months ago
CMake Warning (dev) at /home/runner/work/_temp/1570127931/cmake-3.24.2-linux-x86_64/share/cmake-3.24/Modules/ExternalProject.cmake:3074 (message):
  The DOWNLOAD_EXTRACT_TIMESTAMP option was not given and policy CMP0135 is
  not set.  The policy's OLD behavior will be used.  When using a URL
  download, the timestamps of extracted files should preferably be that of
  the time of extraction, otherwise code that depends on the extracted
  contents might not be rebuilt if the URL changes.  The OLD behavior
  preserves the timestamps from the archive instead, but this is usually not
  what you want.  Update your project to the NEW behavior or specify the
  DOWNLOAD_EXTRACT_TIMESTAMP option with a value of true to avoid this
  robustness issue.
Call Stack (most recent call first):
  /home/runner/work/_temp/1570127931/cmake-3.24.2-linux-x86_64/share/cmake-3.24/Modules/ExternalProject.cmake:4170 (_ep_add_download_command)
  /home/runner/work/ITKModuleTemplate/ITK/Utilities/ClangFormat/DownloadClangFormat.cmake:37 (ExternalProject_add)
  /home/runner/work/ITKModuleTemplate/ITK/CMake/ITKModuleClangFormat.cmake:14 (include)
  /home/runner/work/ITKModuleTemplate/ITK/CMake/ITKModuleMacros.cmake:9 (include)
  /home/runner/work/ITKModuleTemplate/ITK/CMake/ITKModuleExternal.cmake:140 (include)
  CMakeLists.txt:9 (include)
This warning is for project developers.  Use -Wno-dev to suppress it.

I think this is fixed in master. The auditwheel error will be addressed with next RC. I suggest waiting with these updated until then.

jhlegarreta commented 5 months ago

So depends on PR #157.

jhlegarreta commented 1 month ago

Done in https://github.com/InsightSoftwareConsortium/ITKModuleTemplate/pull/158/commits/d4afa39ba2c871d75ad1d96b1e43e004d6dbad89 PR #158.

So closing.