Skip to content

fix(release): stable Linux source-scope probe receipt - #69

Merged
leoisadev1 merged 1 commit into
mainfrom
fix/linux-probe-stderr-repro
Oct 7, 2026
Merged

leoisadev1 merged 1 commit into
mainfrom
fix/linux-probe-stderr-repro

Conversation

@leoisadev1

@leoisadev1 leoisadev1 commented Oct 7, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Build release linux 37621664130 (@ e37711c / fix(release): stable LLVM source digest #66) passed AppImage/deb/rpm/linux tar.gz on both rebuilds, then failed compare on convt-0.2.0-source.tar.gz.
  • Sole differing member: third-party/linux-source-scope/remove-sdk-only-probe.json (same mtime/mode; content hash drift). Manifests failed because they embed the source archive hash.
  • Cause: probe receipt stored full cargo stderr after $SOURCE / $EMPTY_CARGO_HOME substitution. Parallel cargo output still drifts across rebuilds.
  • Fix: keep deterministic fields only (command, empty_cargo_home, exit_code).

Test plan

  • python3 -m py_compile scripts/release/linux-source-scope.py
  • Re-dispatch Build release linux for v0.2.0; expect dual-build compare green and linux-release-review artifact
  • Upload AppImage/deb/rpm/linux tar.gz to release v0.2.0 (keep Mac/Win)

CNV-45


Devin Review

Build release linux 37621664130 passed AppImage/deb/rpm/tar.gz twice, then
failed compare on convt-0.2.0-source.tar.gz: the only differing member was
third-party/linux-source-scope/remove-sdk-only-probe.json. That receipt
stored full cargo stderr after path substitution; parallel cargo output
still drifts across rebuilds, which cascades into release/update manifests.

Keep command + empty_cargo_home + exit_code only — the evidence the probe
exists for.
@leoisadev1
leoisadev1 merged commit e6cfaa6 into main Oct 7, 2026
8 of 9 checks passed
@leoisadev1
leoisadev1 deleted the fix/linux-probe-stderr-repro branch October 7, 2026 14:45

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@greptile-apps

greptile-apps Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Low risk] Adjusts release build probe logging.

This PR appears safe to merge; it removes variable receipt content without changing release checks.

What we checked:

  • Source checks remain in place: No. The script still rejects blocked SDK packages in the build graph, changed build graphs, and changed retained source files.

Summary

Removes Cargo stderr from remove-sdk-only-probe.json so changing output order and temporary paths no longer change the Linux source archive.

  • Keeps command, empty_cargo_home, and exit_code.
  • Leaves the probe and existing source checks unchanged.
  • No actionable issues found. The dual-build release comparison was not run during this review.
  • Reduced diagnostic detail is intentional: leoisadev1 explicitly described keeping only deterministic fields in the PR description.

Reviews (1) · Last reviewed commit: "fix(release): drop nondeterministic carg..." · Reviewed by Greptile

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