Skip to content

Conversation

@charlesrocket
Copy link

@charlesrocket charlesrocket commented Sep 23, 2018

  • Backport python migration of gitian-build.sh

Build system is still unstable on Debian due poor bionic support on stable debootstrap

rename gitian-build script

fix release notes typo

0.12.3-backports
@charlesrocket
Copy link
Author

testing docker/etc fixes

@UdjinM6 UdjinM6 requested a review from codablock September 26, 2018 14:20
Copy link

@codablock codablock left a comment

Choose a reason for hiding this comment

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

utACK for the script itself. Added a few comments which are mostly for a new PR as I assume.
I'm testing this atm and will report back later

codablock
codablock previously approved these changes Oct 19, 2018
Copy link

@codablock codablock left a comment

Choose a reason for hiding this comment

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

tACK

codablock
codablock previously approved these changes Oct 19, 2018
Copy link

@codablock codablock left a comment

Choose a reason for hiding this comment

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

re-ACK

@codablock
Copy link

@charlesrocket btw, while looking into the commits I noticed that you're using a strange user-name (-k). Is this intended?

@charlesrocket
Copy link
Author

@codablock yes, but thats not a strange one yet 😄 - I think you can use unicode as well, but never really tried

@UdjinM6 UdjinM6 added this to the 12.4 milestone Oct 20, 2018
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

Looks good but I would like to see #2319 (comment) addressed.

checksum added
@charlesrocket
Copy link
Author

@UdjinM6 added SDK with checksum

UdjinM6
UdjinM6 previously approved these changes Oct 20, 2018
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK

Copy link

@codablock codablock left a comment

Choose a reason for hiding this comment

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

re-ACK 👍

@UdjinM6 UdjinM6 merged commit 3313bbd into dashpay:develop Oct 21, 2018
@charlesrocket charlesrocket deleted the gitian-builder-upd branch October 21, 2018 19:39
CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request May 4, 2021
…hpay#2319)

* backport python version of gitian-build from bitcoin

rename gitian-build script

fix release notes typo

0.12.3-backports

* change gitian host IP address

* docker/etc fixes

* use docker as default virtualization tech

* add checksum to depends download stage

* add SDK download

checksum added

* remove SDK check

* fix verification
CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request May 4, 2021
Backport bitcoin bitcoin#13623 Migrate gitian-build.sh to python (dashpay#2319)
* backport python version of gitian-build from bitcoin
@UdjinM6 UdjinM6 mentioned this pull request Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants