Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feedback on the gitian tutorial #14

Open
justinmoon opened this issue Dec 24, 2020 · 5 comments
Open

Feedback on the gitian tutorial #14

justinmoon opened this issue Dec 24, 2020 · 5 comments

Comments

@justinmoon
Copy link
Contributor

justinmoon commented Dec 24, 2020

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
Copy link
Contributor Author

And maybe mention including a GPG pubkey in your first PR

@justinmoon
Copy link
Contributor Author

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
Copy link
Contributor Author

justinmoon commented Dec 26, 2020

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

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

@jonatack
Copy link
Owner

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

@justinmoon
Copy link
Contributor Author

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.

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

No branches or pull requests

2 participants