Skip to content

Conversation

@fanquake
Copy link
Member

@fanquake fanquake commented Mar 6, 2022

The time machines should be updated in lockstep.

The time machines should be updated in lockstep.
Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

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

ACK 29862bd, I have reviewed the code and it looks OK, I agree it can be merged.

@gruve-p
Copy link
Contributor

gruve-p commented Mar 6, 2022

ACK 29862bd

@achow101
Copy link
Member

achow101 commented Mar 6, 2022

ACK 29862bd

@achow101 achow101 merged commit 4774b75 into bitcoin:master Mar 6, 2022
@fanquake fanquake deleted the sync_guix_time_machine branch March 6, 2022 14:47
fanquake added a commit to fanquake/bitcoin that referenced this pull request Mar 6, 2022
The time machines should be updated in lockstep.

Github-Pull: bitcoin#24484
Rebased-From: 29862bd
fanquake added a commit that referenced this pull request Mar 7, 2022
0dcbd2e guix: use same commit for codesigning time-machine (fanquake)

Pull request description:

  The time machines should be updated in lockstep. Backports #24484.

  Github-Pull: #24484
  Rebased-From: 29862bd

ACKs for top commit:
  hebasto:
    ACK 0dcbd2e

Tree-SHA512: 831f2d1a0bd92bc6acbe20b0c2e2557f9cc8056034c6975ec6be2cae2eafe584d618b83519623caa81adc434121174e4c56a2df3cc8912352146ac610901bd0f
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Mar 7, 2022
29862bd guix: use same commit for codesigning time-machine (fanquake)

Pull request description:

  The time machines should be updated in lockstep.

ACKs for top commit:
  gruve-p:
    ACK bitcoin@29862bd
  achow101:
    ACK 29862bd
  hebasto:
    ACK 29862bd, I have reviewed the code and it looks OK, I agree it can be merged.

Tree-SHA512: 1cd6e449bba62041f6fde1463c490488b8d60dd86714818f27e92725008af850d3ce3fbde0c302406044cc73094cccc9e9bf4179256e9783627b18c5e66f8b49
@fanquake
Copy link
Member Author

Backported in #24485.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants