Skip to content

COMP: Update CTK to include build-system fixes#7318

Merged
jcfr merged 1 commit intoSlicer:mainfrom
jcfr:update-ctk-build-system-fixes
Oct 30, 2023
Merged

COMP: Update CTK to include build-system fixes#7318
jcfr merged 1 commit intoSlicer:mainfrom
jcfr:update-ctk-build-system-fixes

Conversation

@jcfr
Copy link
Member

@jcfr jcfr commented Oct 30, 2023

See commontk/CTK@b9ef9b8...4888deb

List of CTK changes:

$ git shortlog b9ef9b8aa..4888deb1d --no-merges
Jean-Christophe Fillion-Robin (4):
      COMP: Fix CTK PluginFramework library build
      COMP: Fix configuration without explicitly setting CTK_QT_VERSION
      COMP: Fix CMake 3.26 warning in ctkLinkerAsNeededFlagCheck project
      COMP: Add install rules for PluginFramework CMake modules

See commontk/CTK@b9ef9b8...4888deb

List of CTK changes:

$ git shortlog b9ef9b8aa..4888deb1d --no-merges
Jean-Christophe Fillion-Robin (4):
      COMP: Fix CTK PluginFramework library build
      COMP: Fix configuration without explicitly setting CTK_QT_VERSION
      COMP: Fix CMake 3.26 warning in ctkLinkerAsNeededFlagCheck project
      COMP: Add install rules for PluginFramework CMake modules
@jcfr jcfr requested review from jamesobutler, pieper and sjh26 October 30, 2023 17:08
Copy link
Member

@pieper pieper left a comment

Choose a reason for hiding this comment

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

I didn't try building it but it looks fine to me.

@jcfr jcfr merged commit 8116678 into Slicer:main Oct 30, 2023
@jcfr jcfr deleted the update-ctk-build-system-fixes branch October 30, 2023 18:25
@jcfr jcfr added the backport:5.x Identify pull request expected to be backported to the current 5.x release branch. label Nov 5, 2023
@jcfr jcfr removed the backport:5.x Identify pull request expected to be backported to the current 5.x release branch. label Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants