bitcoin-development icon indicating copy to clipboard operation
bitcoin-development copied to clipboard

Feedback on the gitian tutorial

Open justinmoon opened this issue 4 years ago • 5 comments
trafficstars

Super helpful.

Maybe in this section you could have people create another branch off of master? The previous section you make a branch and if you just copy the commands and push you'll end up with 2 commits on one PR. Seems like people usually do multiple PR -- one with "signed" stuff and another "unsigned". Edit: Made a PR for this.

And for some reason this command:

wget -O osslsigncode-2.0.tar.gz -P inputs https://github.com/mtrojnar/osslsigncode/archive/2.0.tar.gz

downloaded the file as gitian-builder/osslsigncode-2.0.tar.gz. The -P inputs didn't seem to have any effect (should download it as gitian-builder/inputs/osslsigncode-2.0.tar.gz). This caused an error during bin/gbuild -i --commit signature=v${VERSION} ../bitcoin/contrib/gitian-descriptors/gitian-win-signer.yml.

justinmoon avatar Dec 24 '20 01:12 justinmoon

And maybe mention including a GPG pubkey in your first PR

justinmoon avatar Dec 24 '20 03:12 justinmoon

Also, it seems like the bitcoin/contrib/gitian-build.py script can simplify some of this stuff. For instance, this one command will build and sign both the signed and unsigned executables.

./gitian-build.py  --build --sign $SIGNER $VERSION

justinmoon avatar Dec 26 '20 05:12 justinmoon

gitian-build.py makes two nice little commits as well ... on master though ...

Screenshot from 2020-12-26 00-11-53

justinmoon avatar Dec 26 '20 06:12 justinmoon

Thanks for the PR @justinmoon. Happy to review and merge any other proposals you have.

jonatack avatar Dec 28 '20 20:12 jonatack

The downside of gitian-build.py is that since it does everything, when if fails you need to start all over. So not sure if it actually helps.

justinmoon avatar Dec 29 '20 19:12 justinmoon