Page MenuHomePhabricator

Revert "Cleanup unused contrib scripts"
ClosedPublic

Authored by jasonbcox on Nov 12 2018, 21:08.

Details

Summary

This reverts commit b86f0e99002157f8e415a5d2198d78f77612f336.
Reverts https://reviews.bitcoinabc.org/D1504

Test Plan

N/A. Need to fix these scripts up a bit first

Diff Detail

Repository
rABC Bitcoin ABC
Branch
rev
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3867
Build 5807: Bitcoin ABC Buildbot (legacy)
Build 5806: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Nov 12 2018, 21:52

May be supersceded by T477

contrib/gitian/gitian-build.sh
34

Do we know what gitian.sigs and bitcoin-detached-sigs are?

189

We should add downloading this during the setup phase possibly.

236

This seems really old. What is this for?

238
242

These are out of date now.

305

Why is this indented so much more?

356

What's going on with this formatting?

contrib/verifybinaries/verify.sh
25

We don't have this infrastructure setup

73

--decrypt???

jasonbcox added inline comments.
contrib/gitian/gitian-build.sh
189

Agreed. It's in our instructions to fix manually already.

242

Good catch. I'll fix this in a future diff

305

The indenting in all fucked in this file. It's a mix between tabs and spaces. Some lines even have tabs and spaces on the same line. No idea how that's even valid python...

356

See above

contrib/gitian/gitian-build.sh
305

It's not valid python, but it is valid shell...

Can you please convert the tabs to spaces?

This revision is now accepted and ready to land.Nov 26 2018, 19:56
This revision was automatically updated to reflect the committed changes.