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

only bundle libraries that are needed #823

Merged
merged 2 commits into from
Jun 9, 2023

Conversation

KristofferC
Copy link
Member

No description provided.

@KristofferC KristofferC force-pushed the kc/only_bundle_used_libraries branch from 7cb3247 to 6f7a883 Compare June 7, 2023 18:03
@KristofferC KristofferC force-pushed the kc/only_bundle_used_libraries branch from 6f7a883 to 94501fc Compare June 7, 2023 18:21
@codecov
Copy link

codecov bot commented Jun 7, 2023

Codecov Report

Merging #823 (863d970) into master (dff7b2c) will increase coverage by 1.43%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #823      +/-   ##
==========================================
+ Coverage   82.59%   84.03%   +1.43%     
==========================================
  Files           3        3              
  Lines         655      714      +59     
==========================================
+ Hits          541      600      +59     
  Misses        114      114              
Impacted Files Coverage Δ
src/PackageCompiler.jl 93.54% <100.00%> (+0.74%) ⬆️

@KristofferC KristofferC force-pushed the kc/only_bundle_used_libraries branch 2 times, most recently from f1243f7 to 1fedc58 Compare June 8, 2023 14:53
@KristofferC KristofferC force-pushed the kc/only_bundle_used_libraries branch from 1fedc58 to 863d970 Compare June 8, 2023 15:17
@KristofferC KristofferC changed the title WIP: only bundle libraries that are needed only bundle libraries that are needed Jun 8, 2023
@KristofferC
Copy link
Member Author

I think this should be ready to go.

@sloede
Copy link
Collaborator

sloede commented Jun 8, 2023

Out of curiosity, what will this PR being to PC.jl? From the title I assume it will improve somehow which libraries are bundled with a sysimage, but I can't tell from looking at the code 😬

@KristofferC
Copy link
Member Author

KristofferC commented Jun 8, 2023

It will:

@sloede
Copy link
Collaborator

sloede commented Jun 8, 2023

It will:

Great, thanks!

@KristofferC KristofferC merged commit b4045fa into master Jun 9, 2023
@KristofferC KristofferC deleted the kc/only_bundle_used_libraries branch June 9, 2023 06:36
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