Repository navigation
Open
Labels
priority:p2Normal - queue itNormal - queue itscope:repoConfined to this repositoryConfined to this repositorystatus:readyFully specified and ready to be picked upFully specified and ready to be picked uptech-debtKnown shortcut, drift, or hygiene owed - includes cleanupKnown shortcut, drift, or hygiene owed - includes cleanup
Description
Follow-up to #29 / #31 (Wave-3 Chapel rehabilitation).
The Wave-3 rehab in #31 cleared the errors on chpl 2.8.0 (FileSystem.stat → OS.POSIX.stat; string-throws propagation; forall ref-shadow). The unstable-API warnings were intentionally left out of scope. This issue tracks them so they don't drift.
Inventory (from the audit sub-agent run during the #31 work):
_padsymbol names unstable — 6 instances insrc/chapel/FFIBridge.chpl(lines 185, 251, 253, 284, 348, 351). Fix: rename_pad→pad_a/pad_betc. Pure mechanical edit; no semantic change.dmappedkeyword unstable — 3 instances insrc/chapel/ManifestLoader.chpl(lines 85, 114, 195). Fix: replace{range} dmapped new blockDist({range})withblockDist.createDomain({range}). New factory function is the chpl 2.x idiom.DynamicItersmodule unstable —src/chapel/DocudactylHPC.chpl:29. Probably can't be fixed without a stable replacement; document the dependency and live with the warning OR pinpragma "no-doc"if available.string.c_str()unstable — ~12 sites acrossDocudactylHPC.chpl+FaultHandler.chpl. No clean replacement in 2.8.0; either suppress or accept.c_addrOfon cross-locale arrays unstable —ManifestLoader.chpl:198(broadcast mode forall). Audit confirmed the usage is safe (forall i in entryDom do docEntries[i] = localEntries[i]); the warning is spurious. Suppress with a pragma or refactor to avoid the cross-localec_addrOfcall.
These warnings don't fail the build but they pollute CI output and signal API drift risk. Suggested approach: one PR for the _pad + dmapped mechanical fixes (clearly safe), then a second PR judging the remaining cases (DynamicIters / string.c_str() / c_addrOf) which may need pragmas or design discussion.
Not urgent — file so the inventory survives.
Metadata
Metadata
Assignees
Labels
priority:p2Normal - queue itNormal - queue itscope:repoConfined to this repositoryConfined to this repositorystatus:readyFully specified and ready to be picked upFully specified and ready to be picked uptech-debtKnown shortcut, drift, or hygiene owed - includes cleanupKnown shortcut, drift, or hygiene owed - includes cleanup
Activity
Fix HPC compatibility and harden CI with regression coverage (#83)