Skip to content

refactor: lake: log process output as info on errors#13151

Merged
tydeu merged 1 commit intoleanprover:masterfrom
tydeu:lake/proc-error-logInfo
Mar 27, 2026
Merged

refactor: lake: log process output as info on errors#13151
tydeu merged 1 commit intoleanprover:masterfrom
tydeu:lake/proc-error-logInfo

Conversation

@tydeu
Copy link
Copy Markdown
Member

@tydeu tydeu commented Mar 27, 2026

This PR changes Lake.proc to always log process output as info if the process exits with a nonzero return code. This way it behaves the same as captureProc on errors.

@tydeu tydeu added changelog-lake Lake lake-ci Run all Lake tests labels Mar 27, 2026
@tydeu tydeu changed the title refactor: lake: log process output on errors as info refactor: lake: log process output as info on errors Mar 27, 2026
@github-actions github-actions bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label Mar 27, 2026
@leanprover-bot
Copy link
Copy Markdown
Collaborator

Reference manual CI status:

  • ❗ Reference manual CI can not be attempted yet, as the nightly-testing-2026-03-25 tag does not exist there yet. We will retry when you push more commits. If you rebase your branch onto nightly-with-manual, reference manual CI should run now. You can force reference manual CI using the force-manual-ci label. (2026-03-27 06:42:27)

mathlib-nightly-testing bot pushed a commit to leanprover-community/batteries that referenced this pull request Mar 27, 2026
@github-actions github-actions bot added the mathlib4-nightly-available A branch for this PR exists at leanprover-community/mathlib4-nightly-testing:lean-pr-testing-NNNN label Mar 27, 2026
mathlib-nightly-testing bot pushed a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Mar 27, 2026
@mathlib-lean-pr-testing mathlib-lean-pr-testing bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Mar 27, 2026
@mathlib-lean-pr-testing
Copy link
Copy Markdown

Mathlib CI status (docs):

mathlib-nightly-testing bot pushed a commit to leanprover-community/batteries that referenced this pull request Mar 27, 2026
mathlib-nightly-testing bot pushed a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Mar 27, 2026
@mathlib-lean-pr-testing
Copy link
Copy Markdown

Mathlib CI status (docs):

@tydeu tydeu added this pull request to the merge queue Mar 27, 2026
Merged via the queue into leanprover:master with commit d0d135d Mar 27, 2026
48 of 54 checks passed
@tydeu tydeu deleted the lake/proc-error-logInfo branch March 29, 2026 18:30
volodeyka pushed a commit that referenced this pull request Apr 16, 2026
This PR changes `Lake.proc` to always log process output as `info` if
the process exits with a nonzero return code. This way it behaves the
same as `captureProc` on errors.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

builds-mathlib CI has verified that Mathlib builds against this PR changelog-lake Lake lake-ci Run all Lake tests mathlib4-nightly-available A branch for this PR exists at leanprover-community/mathlib4-nightly-testing:lean-pr-testing-NNNN toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants