Skip to content

CI: Disable incremental build for faster CI#10462

Merged
sylvestre merged 1 commit intouutils:mainfrom
oech3:sccinc
Jan 25, 2026
Merged

CI: Disable incremental build for faster CI#10462
sylvestre merged 1 commit intouutils:mainfrom
oech3:sccinc

Conversation

@oech3
Copy link
Contributor

@oech3 oech3 commented Jan 24, 2026

benchmark (closes #10459 ): 18m 28s -> 15m 40s
l10n: 28m 23s -> 16m 1s
CICD: 44m 52s -> 31m 35s
code-quality: 6m 53s -> 7m 26s (bad)

@github-actions
Copy link

GNU testsuite comparison:

Note: The gnu test tests/tail/tail-n0f is now being skipped but was previously passing.

@oech3 oech3 marked this pull request as ready for review January 24, 2026 07:18
@oech3
Copy link
Contributor Author

oech3 commented Jan 24, 2026

@xtqqczze @ChrisDryden I'm not familiar with this. Would you check that am I missing something?

@oech3

This comment was marked as resolved.

@oech3 oech3 changed the title CI: Disable incremental build if sccache is used CI: Disable incremental build for faster CI Jan 24, 2026
@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/tty/tty-eof (passes in this run but fails in the 'main' branch)

@oech3
Copy link
Contributor Author

oech3 commented Jan 24, 2026

?! 2nd result is gabadge.

@oech3
Copy link
Contributor Author

oech3 commented Jan 25, 2026

I guess it was caused by traffic jam...? Faster than main now (i.e. main is slow

@sylvestre sylvestre merged commit bf12e9c into uutils:main Jan 25, 2026
175 of 176 checks passed
@oech3 oech3 deleted the sccinc branch January 25, 2026 21:44
@xtqqczze
Copy link
Contributor

@oech3 Are you definitely sure incremental compilation was enabled?

The dtolnay/rust-toolchain action that we use seems to be configured to disable it:

https://github.com/dtolnay/rust-toolchain/blob/f7ccc83f9ed1e5b9c81d8a67d7ad1a747e22a561/action.yml#L110

@oech3
Copy link
Contributor Author

oech3 commented Jan 25, 2026

Hmm ... I think it reduced size of uploaded caches.

@oech3
Copy link
Contributor Author

oech3 commented Jan 25, 2026

I might misunderstood by CIs using VM, rustup, or preinstalled rust .

@xtqqczze
Copy link
Contributor

Well at least it doesn't hurt to be explicit about CARGO_INCREMENTAL.

@oech3
Copy link
Contributor Author

oech3 commented Jan 26, 2026 via email

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.

benchmark.yml: Missing RUSTC_WRAPPER with sccache action

3 participants