-
Notifications
You must be signed in to change notification settings - Fork 566
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use object for parsing archives, and handle macOS fat archives #1776
Conversation
One note: this involves reading the entire archive into memory. If we want to avoid that, we can add an |
Nice, I'd prefer the usage of |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #1776 +/- ##
==========================================
- Coverage 29.42% 29.33% -0.10%
==========================================
Files 49 49
Lines 17626 17620 -6
Branches 8511 8506 -5
==========================================
- Hits 5187 5168 -19
+ Misses 7339 7334 -5
- Partials 5100 5118 +18
☔ View full report in Codecov by Sentry. |
Updated:
|
Failures don't look connected to this PR. |
Reruned (freebsd has known issues which not fixed yet, please ignore) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot!
Thanks for the review + merge! Is there a plan for when the next sccache release will be? |
voila @alex :) I am curious why you need this PR to be in a release? |
Thank you!
…On Thu, Jun 1, 2023 at 6:08 AM Sylvestre Ledru ***@***.***> wrote:
voila @alex <https://github.com/alex> :)
https://github.com/mozilla/sccache/releases/tag/v0.5.1
I am curious why you need this PR to be in a release?
—
Reply to this email directly, view it on GitHub
<#1776 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAAGBETX26I4AN6G2ED5HTXJBSZBANCNFSM6AAAAAAYQXJLLE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
All that is necessary for evil to succeed is for good people to do nothing.
|
No description provided.