Skip to content

Add gitian build descriptor so that bitcoin can be built deterministically #55

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Add gitian build descriptor so that bitcoin can be built deterministically #55

wants to merge 2 commits into from

Conversation

devrandom
Copy link

@jgarzik
Copy link
Contributor

jgarzik commented Feb 16, 2011

Does this really need its own, single-file directory?

@devrandom
Copy link
Author

I'm thinking there should be a standard directory structure, similar to the debian packaging conventions, that would make it easy and predictable to build a package given its source tree.

Since bitcoin is the first project that is interested in adding this to the source tree, you are free to set the standard by choosing where to put it.

@jgarzik
Copy link
Contributor

jgarzik commented Feb 17, 2011

From bitcoin's perspective, we don't want to end up with 10 files for 10 different systems (gitian, debian, fedora, ...) each in their own directory. Having bitcoin/contrib/gitian.yml seems preferable to bitcoin/gitian/just_this_one_file.yml.

@devrandom
Copy link
Author

Sounds good to me.

@jgarzik
Copy link
Contributor

jgarzik commented Feb 20, 2011

updated pull looks OK to me, but I'd like to poll gavin's opinion on contrib/ top-level dir as a place to throw things like this

@jgarzik
Copy link
Contributor

jgarzik commented Mar 10, 2011

pulled

rdponticelli pushed a commit to Criptomonedas/bitcoin that referenced this pull request Nov 26, 2014
@sipa
Merge pull request bitcoin#55
41863ce 
8881212 Make tests and bench just use asm directly instead of library (Pieter Wuille)
dexX7 pushed a commit to dexX7/bitcoin that referenced this pull request Mar 2, 2015
@zathras-crypto
Merge pull request bitcoin#55 from dexX7/mcore-0.0.9-zathras-ui-rebra…
bb71cdb 
…nding

Rebranding of "Bitcoin Core" to "Omni Core"
dexX7 added a commit to dexX7/bitcoin that referenced this pull request May 27, 2015
@dexX7
Merge pull request bitcoin#55
650aa5e 
e931ce6 Use RPC checks for transaction construction (dexX7)
3b4e061 Use parsing function for RPC input (dexX7)
3517a25 Add requirement checks for RPC input (dexX7)
f5963c6 Add parsing functions for RPC input (dexX7)
jtimon added a commit to jtimon/bitcoin that referenced this pull request Nov 25, 2015
@jtimon
Merge pull request bitcoin#55 from apoelstra/scriptdest
2f7f3bf 
Add commandline flag to set genesis block script destination
ptschip pushed a commit to ptschip/bitcoin that referenced this pull request Jul 25, 2016
@gandrewstone
Merge pull request bitcoin#55 from ptschip/12.1bu_pref
fcbb299 
re-enable the preferrential thinblock timer
CryptAxe pushed a commit to CryptAxe/bitcoin that referenced this pull request Nov 29, 2017
@psztorc
Merge pull request bitcoin#55 from CryptAxe/splitCommit
1ab1737 
Split up BMM & SCDB hashMerkleRoot commitments
classesjack pushed a commit to classesjack/bitcoin that referenced this pull request Jan 2, 2018
@Earlz
Merge pull request bitcoin#55 from qtumproject/neil/fix-issue-54
d7c79fe 
Clear selected coins vector (issue 54)
effectsToCause added a commit to vericoin/vericoin that referenced this pull request Jun 22, 2018
@effectsToCause
Merge pull request bitcoin#55 from vericoin/master
86a6bb5 
timestamps
effectsToCause added a commit to vericoin/vericoin that referenced this pull request Jun 22, 2018
@effectsToCause
Merge pull request bitcoin#52 from effectsToCause/master
aea9776 
Merge pull request bitcoin#55 from vericoin/master
effectsToCause added a commit to vericoin/vericoin that referenced this pull request Jun 22, 2018
@effectsToCause
Merge pull request bitcoin#55 from FuzzyHobbit/patch-7
824dc31 
Use largest finite value for MAX_MONEY
rajarshimaitra pushed a commit to rajarshimaitra/bitcoin that referenced this pull request Aug 5, 2021
@aantonop
Merge pull request bitcoin#55 from lnbook/Enegnei-patch-1
0d01954 
More grammar, spelling, syntax changes
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
Feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@devrandom @jgarzik