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: Fix DICOM/Core build ensuring Qt headers can always be included #1074

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 c819ae7 (BUG: Mark translatable strings in DICOM/Core) through this pull request:

This issue happened because the build was originally tested on a platform having a case insensitive filesystem.

The expected header filename with .h should have been qcoreapplication.h.

For sake of consistency, let's always use the upper case convention without the .h

$ fd qcoreapplication
include/QtCore/5.15.2/QtCore/private/qcoreapplication_p.h
include/QtCore/QCoreApplication
include/QtCore/qcoreapplication.h

This commit fixes a regression introduced in c819ae7 (BUG: Mark translatable
strings in DICOM/Core)
jcfr referenced this pull request in Slicer/Slicer Mar 22, 2023
Updated CTK to make more strings in DICOM browser translatable.

Change log:

Revision: ca5e014451e693b1f9e42250f5ce2ecfd0e7bd0f
Author: Mouhamed DIOP <mouhamed.diop@esp.sn>
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 <mouhamed.diop@esp.sn>
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 <pieper@isomics.com>
Date: 2023-03-15 9:10:21 AM
Message:
BUG: fix spelling error

Thanks @mohamed Alalli BILAL for finding this 👍
@jcfr jcfr merged commit 04618a4 into commontk:master Mar 22, 2023
@jcfr jcfr deleted the fix-ctkDICOM-build-updating-Qt-includes branch March 22, 2023 15:36
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