Skip to content

Conversation

@theuni
Copy link
Member

@theuni theuni commented May 14, 2015

I'm pushing this up before bed in case the breakage gets worse as Europe wakes up and starts PR'ing. I'd prefer to check the build log of this PR itself before merge, but I think all should be OK as-is.

In some cases (Travis), sources and build caches may be moved around in-between builds, and we can't necessarily trust that everything is still intact. In particular, this should properly detect and recover from Travis corruption that came up this week.

This introduces pre-build checks that verify against stashed checksums.

Note that this will cause all sources to be re-downloaded, since (moved) cached sources weren't completely trustworthy before this.

In some cases (Travis), sources and build caches may be moved around in-between
builds, and we can't necessarily trust that everything is still intact.

This introduces pre-build checks that verify against stashed checksums.

Note that this will cause all sources to be re-downloaded, since cached sources
weren't trustworthy before this.
@laanwj laanwj added the Tests label May 14, 2015
@laanwj
Copy link
Member

laanwj commented May 14, 2015

utACK

@laanwj laanwj merged commit 235b3a7 into bitcoin:master May 14, 2015
laanwj added a commit that referenced this pull request May 14, 2015
235b3a7 depends: sanity-check sources and cached builds (Cory Fields)
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants