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

COMP: Update CTK to fix build of CTK DICOM Core library #6908

Merged
merged 1 commit into from
Mar 22, 2023

Conversation

jcfr
Copy link
Member

@jcfr jcfr commented Mar 22, 2023

This commit fixes a regression introduced in 2e4cfbd (BUG: Update CTK to make DICOM browser translatable)

It ensures the build works on case-sensitive file-system by consistently including Qt headers.

List of CTK changes:

$ git shortlog c819ae737..04618a43f --no-merges
Jean-Christophe Fillion-Robin (1):
      COMP: Fix DICOM/Core build ensuring Qt headers can always be included

Related issues and pull-requests:

This commit fixes a regression introduced in 2e4cfbd (BUG: Update CTK
to make DICOM browser translatable)

It ensures the build works on case-sensitive file-system by consistently
including Qt headers.

List of CTK changes:

$ git shortlog c819ae737..04618a43f --no-merges
Jean-Christophe Fillion-Robin (1):
      COMP: Fix DICOM/Core build ensuring Qt headers can always be included
jcfr referenced this pull request Mar 22, 2023
Updated CTK to make more strings in DICOM browser translatable.

Change log:

Revision: ca5e014451e693b1f9e42250f5ce2ecfd0e7bd0f
Author: Mouhamed DIOP <[email protected]>
Date: 2023-03-01 6:47:47 PM
Message:
BUG: Mark translatable strings in DICOM/Core

see Slicer/SlicerLanguagePacks#17
see Slicer/SlicerLanguagePacks#12

Revision: 631acef7ccb35f3b965686700f9a74e234269057
Author: Mouhamed DIOP <[email protected]>
Date: 2023-03-01 4:05:20 PM
Message:
BUG: Mark translatable strings in DICOM/Widgets

see Slicer/SlicerLanguagePacks#17
see Slicer/SlicerLanguagePacks#12

Revision: dd9c6754935b7635e1fb528a73646b463f093a01
Author: Steve Pieper <[email protected]>
Date: 2023-03-15 9:10:21 AM
Message:
BUG: fix spelling error

Thanks @mohamed Alalli BILAL for finding this 👍
@jcfr jcfr merged commit fbc2ede into Slicer:main Mar 22, 2023
@jcfr jcfr deleted the update-ctk-to-fix-CTKDICOMCore-build branch March 22, 2023 15:43
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.

1 participant