Skip to content

gdcm/3.2.1 package update #65869

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.

Sign up for GitHub

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

Jump to bottom
Merged
OddBloke merged 3 commits into from
Sep 16, 2025
Merged

gdcm/3.2.1 package update #65869

OddBloke merged 3 commits into from
Sep 16, 2025

Conversation

@octo-sts
Copy link
Contributor

@octo-sts octo-sts bot commented Sep 11, 2025

@octo-sts octo-sts bot added request-version-update request for a newer version of a package automated pr gdcm labels Sep 11, 2025
@octo-sts
Copy link
Contributor Author

octo-sts bot commented Sep 11, 2025 •
edited
Loading

🩹 Build Failed: Patch Application Failed

Reversed (or previously applied) patch detected! Assume -R? [n] Apply anyway? [n] Skipping patch. Hunk #1 ignored at 16. 1 out of 1 hunk ignored -- saving rejects to file Source/MediaStorageAndFileFormat/gdcmImageChangePhotometricInterpretation.h.rej

Build Details

Category Details
Build System melange
Failure Point patch step - applying numeric-limits.patch to gdcmImageChangePhotometricInterpretation.h

Root Cause Analysis 🔍

The patch 'numeric-limits.patch' appears to have already been applied to the source code or is being applied in reverse. The patch system detected this and refused to apply it, causing the build pipeline to fail with exit status 1.

Was this comment helpful? Please use 👍 or 👎 reactions on this comment.

@octo-sts octo-sts bot added ai/skip-comment Stop AI from commenting on PR bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. labels Sep 11, 2025
@powersj
Copy link
Member

powersj commented Sep 12, 2025

Good news: Build passes now :)
Bad news: Tests fail :(

test_gdcm.cpp:1:10: fatal error: gdcmVersion.h: No such file or directory"

@OddBloke OddBloke self-assigned this Sep 15, 2025
@OddBloke OddBloke force-pushed the wolfictl-4e08487d-bb08-4134-adf4-81ad7aaeebbf branch from b77bda4 to 6dbf842 Compare September 15, 2025 15:57
@OddBloke OddBloke requested a review from a team September 15, 2025 16:05
@OddBloke OddBloke enabled auto-merge September 15, 2025 16:05
aborrero
aborrero approved these changes Sep 16, 2025
View reviewed changes
Copy link
Member

@aborrero aborrero left a comment

Choose a reason for hiding this comment

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

LGTM.

@OddBloke OddBloke merged commit 9d9292a into main Sep 16, 2025
25 checks passed
@OddBloke OddBloke deleted the wolfictl-4e08487d-bb08-4134-adf4-81ad7aaeebbf branch September 16, 2025 18:36
@octo-sts octo-sts bot mentioned this pull request Dec 30, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aborrero aborrero aborrero approved these changes

Assignees

@OddBloke OddBloke

Labels

ai/skip-comment Stop AI from commenting on PR automated pr bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. gdcm manual/review-needed request-version-update request for a newer version of a package

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@powersj @aborrero @OddBloke @wolfi-bot