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

Univalue: Pull subtree #7147

Merged
merged 9,722 commits into from Dec 2, 2015
Merged

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Dec 1, 2015

@sipa @gmaxwell is this desired?

Closes #7102

jonasschnelli and others added 30 commits November 13, 2015 16:27
48edf57 Update key.cpp to new secp256k1 API (Pieter Wuille)
1d84107 Squashed 'src/secp256k1/' changes from 22f60a6..2bfb82b (Pieter Wuille)
a78e6ea Fix bug in mempool_tests unit test (Alex Morcos)
a264c32 http: speed up shutdown (Wladimir J. van der Laan)
Useful now that BIP113 is enforced for transactions entering the
mempool. Was previously only (indirectly) available by calling
getblocktemplate, a relatively expensive RPC call.
Previously if you called fundrawtransaction and set includeWatching to
false it'd act as through you set it to true.
Previously this case failed deep in Cwallet::CreateTransaction() with
the error message "Transaction amounts must be positive"
Completely static RPC call that doesn't change or even look at mutable
state anywhere.
10953a7 Better error message for fundrawtransaction w/ empty vout (Peter Todd)
61e1eb2 Actually use includeWatching value in fundrawtransaction (Peter Todd)
b3ae384 Remove LOCK(cs_main) from decodescript (Peter Todd)
a5bc8de add jonasschnellis key to git-verify-commits trusted-keys (Jonas Schnelli)
160c72a update jonasschnellis gpg key (Jonas Schnelli)
e495ed5 add documentation for exluding whitelistes peer from maxuploadtarget (Jonas Schnelli)
5760749 [docs] rename reducetraffic.md to reduce-traffic.md (Jonas Schnelli)
d61fcff don't enforce maxuploadtargets disconnect for whitelisted peers (Jonas Schnelli)
bbf49da Fix comment for blocksonly parameter interactions (Patick Strateman)
6a4982f Fix fRelayTxs comment (Patick Strateman)
59441a0 Display DEFAULT_WHITELISTALWAYSRELAY in help text (Patick Strateman)
71a2683 Use DEFAULT_BLOCKSONLY and DEFAULT_WHITELISTALWAYSRELAY constants (Patick Strateman)
762b13b Add help text for blocksonly and whitelistalwaysrelay (Patick Strateman)
3a96497 Add whitelistalwaysrelay option (Patick Strateman)
420fa81 Do not process tx inv's in blocksonly mode (Patick Strateman)
4044f07 Add blocksonly mode (Patick Strateman)
@jonasschnelli
Copy link
Contributor

Agree with @sipa: Let's wait for @laanwj open PR there.

b171c69 [doc] Update OS X build notes for new qt5 configure (Michael Ford)
@laanwj laanwj added this to the 0.12.0 milestone Dec 2, 2015
@laanwj
Copy link
Member

laanwj commented Dec 2, 2015

Not sure what is blocking jgarzik/univalue#14.

Need to move this along though for the 0.12 branch. Maybe we could create a univalue fork under bitcoin.

dd5862c Flush coins cache also after transaction processing (Pieter Wuille)
bde953e Uncache input txn in utxo cache if a tx is not accepted to mempool (Matt Corallo)
97bf377 Add CCoinsViewCache::HaveCoinsInCache to check if a tx is cached (Matt Corallo)
677aa3d Discard txn cache entries that were loaded for removed mempool txn (Matt Corallo)
b2e74bd Get the set of now-uncacheable-txn from CTxMemPool::TrimToSize (Matt Corallo)
74d0f90 Add method to remove a tx from CCoinsViewCache if it is unchanged (Matt Corallo)
8f0d79e test: Disable scheduler test manythreads (Wladimir J. van der Laan)
@jonasschnelli
Copy link
Contributor

I think a fork of UniValue under the github bitcoin identity would make sense.

@laanwj
Copy link
Member

laanwj commented Dec 2, 2015

@maflcko maflcko force-pushed the MarcoFalke-2015-univalueSync branch from fafb6b6 to fac8548 Compare December 2, 2015 10:34
@maflcko
Copy link
Member Author

maflcko commented Dec 2, 2015

Updated. Make sure to fetch https://github.com/bitcoin/univalue (and not jgarzik) when doing

$ contrib/devtools/git-subtree-check.sh src/univalue

@maflcko maflcko force-pushed the MarcoFalke-2015-univalueSync branch from fac8548 to fa82dce Compare December 2, 2015 11:15
@fanquake
Copy link
Member

fanquake commented Dec 2, 2015

ACK

On Wednesday, December 2, 2015, Wladimir J. van der Laan <
notifications@github.com> wrote:

src/univalue in HEAD was last updated to upstream commit 9218595ed19ccc1a7958c2c7284fd4acb9ce4fdc (tree 5d50d2814919fe88dda5f33df70310c2df080319)
src/univalue in HEAD currently refers to tree 5d50d2814919fe88dda5f33df70310c2df080319
GOOD


Reply to this email directly or view it on GitHub
#7147 (comment).

@laanwj
Copy link
Member

laanwj commented Dec 2, 2015

Can you re-do this, I've added a commit to jgarzik/univalue#14 and updated the repository

@maflcko maflcko force-pushed the MarcoFalke-2015-univalueSync branch from f01d913 to fad4ea8 Compare December 2, 2015 11:30
@laanwj
Copy link
Member

laanwj commented Dec 2, 2015

Thanks, should pass Travis now

src/univalue in HEAD was last updated to upstream commit 2740c4f71242086a7eb3dc32f812546ba9fad913 (tree 6abb4a6467fde2b738dfa8acf32fef55257130c2)
src/univalue in HEAD currently refers to tree 6abb4a6467fde2b738dfa8acf32fef55257130c2
GOOD

ACK

@maflcko
Copy link
Member Author

maflcko commented Dec 2, 2015

@laanwj You have run subtree-check on master, not my PR ;)

@laanwj
Copy link
Member

laanwj commented Dec 2, 2015

@MarcoFalke I was running it on master but without your PR merged, worst of both worlds. But I've since updated the post and it should be ok now?

@maflcko
Copy link
Member Author

maflcko commented Dec 2, 2015

Looks good.

@laanwj laanwj merged commit fad4ea8 into bitcoin:master Dec 2, 2015
laanwj added a commit that referenced this pull request Dec 2, 2015
9827091 Squashed 'src/univalue/' changes from 5839ac3..2740c4f (MarcoFalke)
@maflcko maflcko deleted the MarcoFalke-2015-univalueSync branch December 2, 2015 12:31
@sipa
Copy link
Member

sipa commented Dec 2, 2015 via email

@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

upgrade univalue