Skip to content

Mpi char buffer test with new images#4667

Draft
scottwittenburg wants to merge 3 commits intoornladios:masterfrom
scottwittenburg:mpi-char-buffer-test-with-new-images
Draft

Mpi char buffer test with new images#4667
scottwittenburg wants to merge 3 commits intoornladios:masterfrom
scottwittenburg:mpi-char-buffer-test-with-new-images

Conversation

@scottwittenburg
Copy link
Copy Markdown
Contributor

DO NOT MERGE!

Test whether the error in #4664 is fixed by the new ci images in #4666.

* Update to use Ubuntu 22.04 instead of 20.04
* Replace gcc8 with gcc12
* Replace clang 6 and 10 with clang 11 and 14
* Update to 2025 version of spack and E4S mirror
* Remove custom blosc2 build in favor of spack-built version
* Remove unneeded adios2 patch, update the xrootd patch
The largest value "digit" could assume is 333, when NVARS is the largest
possible int.

../source/testing/adios2/performance/manyvars/PerfManyVars.c:94:39: warning: '%d'
directive output may be truncated writing between 1 and 10 bytes into a region of
size between 3 and 12 [-Wformat-truncation=]
     94 |     snprintf(fmt, sizeof(fmt), "v%%%d.%dd", digit, digit);
        |                                       ^~
  ../source/testing/adios2/performance/manyvars/PerfManyVars.c:94:32: note:
  directive argument in the range [1, 2147483647]
     94 |     snprintf(fmt, sizeof(fmt), "v%%%d.%dd", digit, digit);
        |                                ^~~~~~~~~~~
  ../source/testing/adios2/performance/manyvars/PerfManyVars.c:94:5: note:
  'snprintf' output between 7 and 25 bytes into a destination of size 16
     94 |     snprintf(fmt, sizeof(fmt), "v%%%d.%dd", digit, digit);
        |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
@github-actions
Copy link
Copy Markdown

This PR is stale because it has been open 6 months with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions Bot added the Stale label Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant