Skip to content

cache GNU tar's tarball - #413

Open
oech3 wants to merge 1 commit into
uutils:mainfrom
oech3:gnucache
Open

oech3 wants to merge 1 commit into
uutils:mainfrom
oech3:gnucache

Conversation

@oech3

@oech3 oech3 commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@oech3
oech3 force-pushed the gnucache branch 4 times, most recently from 33c9068 to f78dc68 Compare October 22, 2026 14:39
@oech3
oech3 marked this pull request as ready for review October 22, 2026 14:50
@oech3
oech3 requested a review from kaladron as a code owner October 22, 2026 14:50
@oech3

This comment was marked as resolved.

@codspeed

codspeed Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

1 warning

⚠️ 5 benchmarks are dominated by system calls

System calls cannot be simulated consistently and are left out of the measure. The reported value understates the real cost. Switch to the Walltime instrument to measure them.

Learn more about system calls

See affected benchmarks

✅ 7 untouched benchmarks


Comparing oech3:gnucache (467d86b) with main (b453849)

Open in CodSpeed

@oech3
oech3 force-pushed the gnucache branch 2 times, most recently from 6e5f185 to 59418c3 Compare October 22, 2026 14:56
@oech3

This comment was marked as resolved.

@oech3
oech3 force-pushed the gnucache branch 3 times, most recently from 6e5d097 to 6225ce3 Compare October 22, 2026 15:15
@oech3

oech3 commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor Author

I was needed to put cache on /tmp. I know it is not an ideal path, but I cannot fix it.

@codecov

codecov Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.17%. Comparing base (791ae26) to head (467d86b).
⚠️ Report is 100 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #413      +/-   ##
==========================================
+ Coverage   96.84%   98.17%   +1.32%     
==========================================
  Files          11       15       +4     
  Lines        1492     1970     +478     
  Branches       29       39      +10     
==========================================
+ Hits         1445     1934     +489     
+ Misses         46       35      -11     
  Partials        1        1              
Flag Coverage Δ
macos_latest 98.17% <ø> (+1.32%) ⬆️
ubuntu_latest 98.17% <ø> (+1.32%) ⬆️
windows_latest 0.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread .github/workflows/GnuTests.yml Outdated
Comment thread .github/workflows/GnuTests.yml Outdated
Comment thread .github/workflows/GnuTests.yml Outdated
Comment thread util/build-gnu.sh Outdated
@oech3
oech3 force-pushed the gnucache branch 2 times, most recently from ed83c60 to 467d86b Compare October 23, 2026 04:10
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.

2 participants