Skip to content

ci(macos): salt mcpp sandbox cache (std.pcm deployment-target mismatch)#318

Merged
Sunrisepeak merged 1 commit into
mainfrom
fix/macos-ci-bmi-cache
Jun 4, 2026
Merged

ci(macos): salt mcpp sandbox cache (std.pcm deployment-target mismatch)#318
Sunrisepeak merged 1 commit into
mainfrom
fix/macos-ci-bmi-cache

Conversation

@Sunrisepeak
Copy link
Copy Markdown
Member

macos CI on main is red since mcpp 0.0.49 became the index latest: the workflow pins MACOSX_DEPLOYMENT_TARGET=11.0, but the restored ~/.mcpp BMI cache holds a std.pcm built for arm64-apple-macosx15 under the same fingerprint — mcpp ≤ 0.0.49 doesn't include the deployment target in the BMI fingerprint, so the stale module is reused and every TU dies with:

error: AST file '...std.pcm' was compiled for the target 'arm64-apple-macosx15.0.0'
       but the current translation unit is being compiled for target 'arm64-apple-macosx11.0.0'

Salting the cache key (+ restore-keys) rebuilds a coherent all-11.0 store. Permanent fix ships in mcpp 0.0.50 (mcpp-community/mcpp#116: deployment target folded into the fingerprint + [build] macos_deployment_target manifest field), after which xlings switches to the manifest field with CI assertions.

…match)

The workflow pins MACOSX_DEPLOYMENT_TARGET=11.0, but the cached
~/.mcpp BMI store contains a std.pcm built for arm64-apple-macosx15
under the same fingerprint — mcpp <= 0.0.49 does not include the
deployment target in the BMI fingerprint, so the stale module is
reused and every TU fails with a config mismatch. Salt the cache so a
coherent (all-11.0) store is rebuilt. mcpp 0.0.50 folds the deployment
target into the fingerprint, fixing this class permanently
(mcpp-community/mcpp#116).
@Sunrisepeak Sunrisepeak merged commit cac284e into main Jun 4, 2026
3 checks passed
@Sunrisepeak Sunrisepeak deleted the fix/macos-ci-bmi-cache branch June 4, 2026 19:38
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.

1 participant