Skip to content
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

Merged
merged 1 commit into from May 30, 2023

Conversation

alex
Copy link
Contributor

@alex alex commented May 26, 2023

No description provided.

@alex
Copy link
Contributor Author

alex commented May 26, 2023

One note: this involves reading the entire archive into memory. If we want to avoid that, we can add an mmap crate to do it that way.

@drahnr
Copy link
Collaborator

drahnr commented May 27, 2023

Nice, I'd prefer the usage of mmap where possible.

Cargo.toml Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

Codecov Report

Patch coverage: 15.78% and project coverage change: -0.10 ⚠️

Comparison is base (b2faf16) 29.42% compared to head (d4b6de2) 29.33%.

❗ Current head d4b6de2 differs from pull request most recent head ca4db69. Consider uploading reports for the commit ca4db69 to get more accurate results

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     
Impacted Files Coverage Δ
src/util.rs 23.45% <15.78%> (+0.09%) ⬆️

... and 12 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@alex
Copy link
Contributor Author

alex commented May 27, 2023

Updated:

  • Use memmap
  • Fix clippy warnings
  • Remove unrelated style changes

@alex
Copy link
Contributor Author

alex commented May 27, 2023

Failures don't look connected to this PR.

@Xuanwo
Copy link
Collaborator

Xuanwo commented May 27, 2023

Failures don't look connected to this PR.

Reruned (freebsd has known issues which not fixed yet, please ignore)

Copy link
Collaborator

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

src/util.rs Show resolved Hide resolved
@sylvestre sylvestre merged commit 57ac5ea into mozilla:main May 30, 2023
37 of 38 checks passed
@alex alex deleted the object branch May 30, 2023 10:38
@alex
Copy link
Contributor Author

alex commented May 31, 2023

Thanks for the review + merge!

Is there a plan for when the next sccache release will be?

@sylvestre
Copy link
Collaborator

voila @alex :)
https://github.com/mozilla/sccache/releases/tag/v0.5.1

I am curious why you need this PR to be in a release?

@alex
Copy link
Contributor Author

alex commented Jun 1, 2023 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.

None yet

5 participants