Skip to content

[cgns] Update to 4.5.2 - #52945

Draft
xiaozhuai (xiaozhuai) wants to merge 4 commits into
microsoft:masterfrom
xiaozhuai:dev-cgns
Draft

[cgns] Update to 4.5.2#52945
xiaozhuai (xiaozhuai) wants to merge 4 commits into
microsoft:masterfrom
xiaozhuai:dev-cgns

Conversation

@xiaozhuai

Copy link
Copy Markdown
Contributor

Update to 4.5.2

@xiaozhuai

Copy link
Copy Markdown
Contributor Author

Replace #51501

Comment thread ports/cgns/portfile.cmake
"legacy" CGNS_ENABLE_LEGACY
"tests" CGNS_ENABLE_TESTS
)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
if ("mpi" IN_LIST FEATURES)
list(APPEND FEATURE_OPTIONS
-DHDF5_NEED_MPI=ON
-DCGNS_ENABLE_PARALLEL=ON
)
endif()

Additional options are required by upstream.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

CI failed

@vicroms
Victor Romero (vicroms) marked this pull request as draft July 21, 2026 08:26
@xiaozhuai
xiaozhuai (xiaozhuai) marked this pull request as ready for review July 21, 2026 08:39
@dg0yt

Copy link
Copy Markdown
Contributor

May need to wait for #52532: Refresh seacas and vtk after 3 year without significant port updates.

@xiaozhuai
xiaozhuai (xiaozhuai) marked this pull request as draft July 21, 2026 10:22
@xiaozhuai

Copy link
Copy Markdown
Contributor Author

May need to wait for #52532: Refresh seacas and vtk after 3 year without significant port updates.

Sure~

@xiaozhuai
xiaozhuai (xiaozhuai) force-pushed the dev-cgns branch 3 times, most recently from 927d125 to 823805c Compare July 22, 2026 04:44
@xiaozhuai

xiaozhuai (xiaozhuai) commented Jul 22, 2026

Copy link
Copy Markdown
Contributor Author
  1. Remove the fortran feature from cgns because it has never worked properly.
  2. Cherry pick [seacas] Update to 2025-10-14 from [seacas] Update to 2025-10-14 [vtk] Update to 9.6.2 [f3d] Update to 3.5.0 #52532 to make the CI green.
  3. Fix upstream known issue Parallel build of CGNS -- problem reading file-per-rank and writing single file using parallel io CGNS/CGNS#835

@xiaozhuai

Copy link
Copy Markdown
Contributor Author

Now vtk fail to build, wait for #52532

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants