Skip to content

COMP: Add GDCM system version check for SetSecondaryCaptureImagePlaneModule#4601

Merged
thewtex merged 1 commit intoInsightSoftwareConsortium:masterfrom
thewtex:gdcm-version-check
Apr 21, 2024
Merged

COMP: Add GDCM system version check for SetSecondaryCaptureImagePlaneModule#4601
thewtex merged 1 commit intoInsightSoftwareConsortium:masterfrom
thewtex:gdcm-version-check

Conversation

@thewtex
Copy link
Copy Markdown
Member

@thewtex thewtex commented Apr 20, 2024

This has not been released in GDCM, yet. Avoid build errors for a system
GDCM. There will be a difference in behavior, which will be flagged with
failing tests.

Co-authored-by: Mihail Isakov mihail.isakov@gmail.com

@github-actions github-actions Bot added type:Compiler Compiler support or related warnings area:IO Issues affecting the IO module labels Apr 20, 2024
@thewtex thewtex requested review from issakomi and malaterre April 20, 2024 22:27
@thewtex
Copy link
Copy Markdown
Member Author

thewtex commented Apr 20, 2024

Verified with system GDCM on Debian Bullseye.

@issakomi
Copy link
Copy Markdown
Member

Thank you! BTW, there is another call on line 284 (in 'Debug' mode).

…Module

This has not been released in GDCM, yet. Avoid build errors for a system
GDCM. There will be a difference in behavior, which will be flagged with
failing tests.

Co-authored-by: Mihail Isakov <mihail.isakov@gmail.com>
@thewtex thewtex force-pushed the gdcm-version-check branch from 06a4dce to f4e9b0b Compare April 20, 2024 23:03
@thewtex thewtex added this to the ITK 5.4.0 milestone Apr 20, 2024
@thewtex thewtex merged commit 596ed5f into InsightSoftwareConsortium:master Apr 21, 2024
@thewtex thewtex deleted the gdcm-version-check branch April 21, 2024 19:14
issakomi added a commit to issakomi/ITK that referenced this pull request Apr 24, 2024
…aneModule

Follow up InsightSoftwareConsortium#4601
GDCM master has currenlty 3.1.0. Updated check to include versions 3.1+.
issakomi added a commit to issakomi/ITK that referenced this pull request Apr 24, 2024
…aneModule

Follow up InsightSoftwareConsortium#4601
GDCM master is currently at version 3.1.0. Updated check to include versions 3.1+.
issakomi added a commit to issakomi/ITK that referenced this pull request Apr 24, 2024
…aneModule

Follow up InsightSoftwareConsortium#4601
GDCM master is currently at version 3.1.0. Updated check to include versions 3.1+.
bourdaisj pushed a commit to bourdaisj/ITK that referenced this pull request May 15, 2024
…aneModule

Follow up InsightSoftwareConsortium#4601
GDCM master is currently at version 3.1.0. Updated check to include versions 3.1+.
hjmjohnson pushed a commit to hjmjohnson/ITK that referenced this pull request May 6, 2026
…ersion-check

COMP: Add GDCM system version check for SetSecondaryCaptureImagePlaneModule
hjmjohnson pushed a commit to hjmjohnson/ITK that referenced this pull request May 6, 2026
…aneModule

Follow up InsightSoftwareConsortium#4601
GDCM master is currently at version 3.1.0. Updated check to include versions 3.1+.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:IO Issues affecting the IO module type:Compiler Compiler support or related warnings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants