Skip to content

BUG: Fix GDCM posix_memalign undefined with mingw - RELEASE #5774

Closed
blowekamp wants to merge 1 commit intoInsightSoftwareConsortium:release-5.4from
blowekamp:release-5.4
Closed

BUG: Fix GDCM posix_memalign undefined with mingw - RELEASE #5774
blowekamp wants to merge 1 commit intoInsightSoftwareConsortium:release-5.4from
blowekamp:release-5.4

Conversation

@blowekamp
Copy link
Member

Address compilation error, by switching to use _aligned_malloc.

PR Checklist

  • No API changes were made (or the changes have been approved)
  • No major design changes were made (or the changes have been approved)
  • Added test (or behavior not changed)
  • Updated API documentation (or API not changed)
  • Added license to new files (if any)
  • Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
  • Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

Address compilation error, by switching to use _aligned_malloc.
@github-actions github-actions bot added type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots area:ThirdParty Issues affecting the ThirdParty module labels Feb 6, 2026
@dzenanz
Copy link
Member

dzenanz commented Feb 6, 2026

Should this PR also go to main and upstream?

@blowekamp
Copy link
Member Author

This is a minimal change to patch release.

I want to look into using std::aligned_alloc for upstream GDCM.

@blowekamp
Copy link
Member Author

blowekamp commented Feb 9, 2026

This is a minimal change to patch release.

I want to look into using std::aligned_alloc for upstream GDCM.

Unfortunately, this a C not C++ file so the STD function cannot be used.

@blowekamp blowekamp marked this pull request as ready for review February 9, 2026 16:38
@blowekamp
Copy link
Member Author

Replaced by #5792 due to incorrect remote branch name.

@blowekamp blowekamp closed this Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ThirdParty Issues affecting the ThirdParty module type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants