Skip to content

fix(ci): disable incremental compilation for sccache#306

Merged
namastex888 merged 2 commits into
mainfrom
fix/sccache-incremental
Dec 10, 2025
Merged

fix(ci): disable incremental compilation for sccache#306
namastex888 merged 2 commits into
mainfrom
fix/sccache-incremental

Conversation

@namastex888

@namastex888 namastex888 commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

Summary

  • Fixes sccache build failure by disabling Cargo incremental compilation
  • Fixes forge-core version mismatch by syncing workspace version before tagging
  • Adds file ownership cleanup for Linux container builds

Issues Fixed

1. sccache CARGO_INCREMENTAL conflict

  • RC.11 builds failed with: sccache: incremental compilation is prohibited
  • Root cause: forge-builder Docker container has CARGO_INCREMENTAL=1 set
  • Fix: Added CARGO_INCREMENTAL=0 to sccache configuration

2. forge-core version mismatch

  • Crates showed v0.8.7-rc.4 even from v0.8.7-rc.11 tag
  • Root cause: release.yml created tags without updating workspace version first
  • Fix: Now triggers sync-version-from-forge workflow before creating tag

3. Linux container file ownership

  • CI failed with permission errors on git checkout
  • Root cause: Docker container runs as root, creates files runner can't delete
  • Fix: Added chown -R 1000:1000 cleanup step at end of build

Test plan

  • Merge with rc label to trigger RC.12
  • Verify all platform builds succeed with sccache enabled
  • Verify forge-core tag v0.8.7-rc.12 has matching workspace version

@namastex888 namastex888 added the rc Release candidate - triggers RC automation label Dec 9, 2025
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

…r ownership

Two fixes for CI issues:

1. sccache incompatibility: Add CARGO_INCREMENTAL=0 because sccache doesn't
   work with Cargo's incremental compilation. The forge-builder Docker
   container has CARGO_INCREMENTAL=1 set, which caused builds to fail.

2. Container ownership: Add cleanup step to fix file ownership at end of
   Linux container builds. Docker containers run as root, creating files
   that the runner user can't clean up, causing subsequent jobs (e2e, test)
   to fail with EACCES permission errors.
@namastex888
namastex888 force-pushed the fix/sccache-incremental branch from 26cbece to dc4a621 Compare December 9, 2025 23:56
Previously, release.yml created tags in forge-core pointing at main HEAD
without updating the workspace version first. This caused version mismatch
where crates showed old version (e.g., 0.8.7-rc.4) even from newer tags
(e.g., v0.8.7-rc.11).

Now triggers sync-version-from-forge workflow in forge-core which:
1. Updates [workspace.package] version using cargo set-version
2. Commits the change
3. Creates annotated tag pointing at the versioned commit
@namastex888
namastex888 merged commit f182a3a into main Dec 10, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rc Release candidate - triggers RC automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant