depends: fix major regression after d546191dc. #5586

pull theuni wants to merge 1 commits into bitcoin:master from theuni:fix-depends-major-bug changing 1 files +2 −2
  1. theuni commented at 6:18 PM on December 31, 2014: member

    This is included in #5582, but I'm PRing separately because it is a badly needed fix. It also needs backport to 0.10. Unfortunately, this will force all dependencies to be rebuilt. I'll accept lashings as penance...

    Will rebase #5882 after merge.

    Broken hash logic caused all depends on some platforms (osx at least) to end up with the same build-id. Without this fix, nothing will be rebuilt when recipes or dependencies change. The 'cut' was put in the wrong place, so rather than hashing the hashes, it hashed the filename which was always "-" (stdin). Probably a stupid c/p mistake.

  2. depends: fix major regression after d546191dc.
    Broken hash logic caused all depends on some platforms (osx at least) to end up
    with the same build-id. Without this fix, nothing will be rebuilt when recipes
    or dependencies change.
    d57b303e1e
  3. laanwj added this to the milestone 0.10.0 on Jan 1, 2015
  4. laanwj added the label Build system on Jan 1, 2015
  5. laanwj merged this on Jan 2, 2015
  6. laanwj closed this on Jan 2, 2015

  7. laanwj referenced this in commit 48ea5e0ea0 on Jan 2, 2015
  8. laanwj referenced this in commit 9241f7a7c1 on Jan 2, 2015
  9. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-13 18:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me