-
Notifications
You must be signed in to change notification settings - Fork 30
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
Do not download again sources that are already cached #59
Conversation
src_path = abspath(source.url) | ||
|
||
# And if this is a locally-sourced tarball, just verify | ||
verify(src_path, source.hash) | ||
verify(src_path, source.hash) || error("Verification failed") |
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.
To be honest, I have no idea why we have this branch. I'm leaving it as it is, but I don't know when we have sources that are local files
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.
Unsurprisingly, these are 2 of the only 3 lines missing in code coverage in this file 😄
cf9ae70
to
f3e96df
Compare
Ahhhhhh, of course. The old |
f3e96df
to
15bec36
Compare
@info "Cached file found in $(path)" | ||
else | ||
@info "Downloading $(url) to $(path)..." |
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.
I hope these logging messages are fine. I know this is a bit verbose, but they're very useful for debugging and making sure we're doing what we're supposed to.
Fix JuliaPackaging/BinaryBuilder.jl#928