Skip to content

Conversation

@codablock
Copy link

@codablock codablock commented Dec 27, 2017

Continuation of #1770

This is the final batch to get to Bitcoin 0.13. After that, I'll continue with Bitcoin 0.14 PRs.

This PR includes a temporary hack to fix a compilation error. This was needed because of the out-of-order backport of the main.cpp split, which in combination with backporting of Bitcoin PR bitcoin#8179 results in compilation errors. I don't remember exactly why I did not simply forward declare the needed stuff, but I remember that it resulted in more problems.

The hack is really just temporary and the fix for it is already in my local branch. When the time for it comes (all the other required PRs are backported), the hack will vanish.

This PR currently also includes #1819, which I'll remove after it is merged into develop.

laanwj and others added 30 commits December 28, 2017 11:44
01a9904 [trivial] Ignore split-debug.sh (fanquake)
fa61756 [gitian] set correct PATH for wrappers (MarcoFalke)
0e209f9 [trivial] Sync ax_pthread with upstream draft (fanquake)
1c2a1ba Add address label to request payment QR Code (QT) (Francesco 'makevoid' Canessa)
…ry and website to the About Dialog

fa58e5e [doc] Add website links to about dialog (MarcoFalke)
…sults

bc0a895 Do not set extra flags for unfiltered DNS seed results (Pieter Wuille)
…t parsing

6fa950a [RPC] Fix createrawtx sequence number unsigned int parsing (Jonas Schnelli)
This was probably missed while backporting Bitcoin PR bitcoin#7892
…cestors

29fac19 Add unit tests for ancestor feerate mining (Suhas Daftuar)
c82a4e9 Use ancestor-feerate based transaction selection for mining (Suhas Daftuar)
…lent return

facb6c0 [qa] mininode: fail on send_message instead of silent return (MarcoFalke)
faa91b1 [wallet] tests: Don't use floating point (MarcoFalke)
7734479 readme: Omit phrasing; 'new' (Will Binns)
9e3ec74 Clarify documentation for running a tor node (Nathaniel Mahieu)
2759597 Only pass -lQt5PlatformSupport if >=Qt5.6 (Jonas Schnelli)
59d063d Use runtime linking of QT libdbus, use custom/temp. SDK URL (Jonas Schnelli)
6194d9a Fix bitcoin_qt.m4 and fix-xcb-include-order.patch (Jonas Schnelli)
f6eb4e2 [depends] OpenSSL 1.0.1k - update config_opts (fanquake)
f25209a depends: bump OSX toolchain (Cory Fields)
…for each node

fad1845 [qa] test_framework: Use different rpc_auth_pair for each node (MarcoFalke)
595b22e Stop treating importaddress'ed scripts as change (Pieter Wuille)
1e9aab0 Remove sipa's old revoked key from verify-commits (Peter Todd)
966151e Add README for verify-commits (Peter Todd)
11164ec Remove keys that are no longer used for merging (Peter Todd)
22421fa Remove pointless warning (Peter Todd)
9523e8a Make verify-commits path-independent (Matt Corallo)
f7d4a25 Make verify-commits POSIX-compliant (Matt Corallo)
…accepted blocks.

54326a6 Increase maximum orphan size to 100,000 bytes. (Gregory Maxwell)
8c99d1b Treat orphans as implicit inv for parents, discard when parents rejected. (Gregory Maxwell)
11cc143 Adds an expiration time for orphan tx. (Gregory Maxwell)
db0ffe8 This eliminates the primary leak that causes the orphan map to  always grow to its maximum size. (Gregory Maxwell)
1b0bcc5 Track orphan by prev COutPoint rather than prev hash (Pieter Wuille)
Can be removed when we catch up with backporting. You'll notice it's time
for this when you get conflicts while merging the affected backported PR.
ad0752e Stop trimming when mapTx is empty (Pieter Wuille)
fa58f94 [qa] pull-tester: Start longest test first (MarcoFalke)
fa3b379 [qa] pull-tester: Fix assertion and check for run_parallel (MarcoFalke)
fa32465 [qa] fundrawtransaction: Create get_unspent() (MarcoFalke)
fa8ce3b [qa] assert 'changePosition out of bounds' (MarcoFalke)
e5a680d [Doc] Update OS X build notes for 10.11 SDK (fanquake)
3775ff9 Enable mempool consistency checks in unit tests (Pieter Wuille)
a1c92c2 trivial: capitalize BIP32 in option help (Wladimir J. van der Laan)
d241487 [doc] Add OS X ZMQ requirement to QA readme (fanquake)
d7828ab check that transactionView->selectionModel()->selectedRows(0) exists (fsb4000)
f70bcfc [trivial] Add aarch64 to depends .gitignore (fanquake)
409f833 RPC: Hide softfork if timeout is 0 (jl2012)
2129fce The bit field is shown only when status is "started" (Pavel Janík)
a7897c0 qt: Remove client name from debug window (Wladimir J. van der Laan)
laanwj and others added 16 commits December 28, 2017 11:44
975a41d windows: Add testnet icon for testnet link (Wladimir J. van der Laan)
0ce8e99 windows: Add testnet link to installer (Wladimir J. van der Laan)
36f1b9d Tests: Increase sync_blocks() timeouts in pruning.py (Suhas Daftuar)
20f3cd7 wallet: Revert input selection post-pruning (Wladimir J. van der Laan)
4f44cb6 qt: Network-specific example address (Wladimir J. van der Laan)
fa6ad56 [travis] Update SDK_URL (MarcoFalke)
cf2ef78 build: require boost for bench (Cory Fields)
ab0c35a [Doc] Update bips.md for CSV softfork. (fanquake)
099d4b0 gitian: use a wrapped gcc/g++ to avoid the need for a system change (Cory Fields)
b556bed build: fix Windows builds without pkg-config (Cory Fields)
0c928cb build: Fix Qt5PlatformSupport check without pkg-config (Cory Fields)
1ba3db6 bash-completion: Adapt for 0.12 and 0.13 (Christian von Roques)
e91cf4b Add test for handling of unconnecting headers (Suhas Daftuar)
96fa953 Improve handling of unconnecting headers (Suhas Daftuar)
c1d61fb Add warning if -blockminsize is used. (Suhas Daftuar)
27362dd Remove -blockminsize option (Suhas Daftuar)
d2e46e1 Remove addScoreTxs() (Suhas Daftuar)
6dd4bc2 Exclude witness transactions in addPackageTxs() pre-segwit activation (Suhas Daftuar)
f15c2cd CreateNewBlock: add support for size-accounting to addPackageTxs (Suhas Daftuar)
…8295

Also remove nBlockMinSize check from addPackageTxs. Removed by SegWit
related commits in Bitcoin.
d6dc1bc Fix 0.12 release notes on block relaying (Krzysztof Jurewicz)
This also overwrites/dashifies icons/bitcoin_testnet.ico which was
introduced in Bitcoin bitcoin#8285.

The icons were previously located in the drkblue theme directory while the
path used in dash.qrc was poining to the non-theme icons directory. Also,
the icons were never ported to the other themes. This commit moves them one
level up until someone actually ports these to the other themes (if ever
needed).
Dash allows free transactions by default, mixing up the expected ordering
of transactions in TestPackageSelection
@codablock codablock force-pushed the pr_backport_bitcoin_0.13-6 branch from aab9727 to f61a754 Compare December 28, 2017 10:45
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.

👍

Just a few small nits, otherwise looks good, will test.

====================

* Update translations, see [translation_process.md](https://github.com/dashpay/dash/blob/master/doc/translation_process.md#syncing-with-transifex)
* Update translations (ping wumpus on IRC) see [translation_process.md](https://github.com/dashpay/dash/blob/master/doc/translation_process.md#synchronising-translations).
Copy link

Choose a reason for hiding this comment

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

(ping wumpus on IRC) should not be added

@@ -1 +1,2 @@
contrib/dashd.bash-completion dashd
contrib/dash-cli.bash-completion dash-cli
Copy link

Choose a reason for hiding this comment

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

needs new line at eof

@UdjinM6 UdjinM6 added this to the 12.3 milestone Dec 28, 2017
@codablock codablock force-pushed the pr_backport_bitcoin_0.13-6 branch from 3df3df0 to 9c2e970 Compare December 28, 2017 14:53
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.

slightly tested, ACK

@UdjinM6 UdjinM6 merged commit 1274cd3 into dashpay:develop Dec 28, 2017
@UdjinM6 UdjinM6 mentioned this pull request Dec 28, 2017
@UdjinM6 UdjinM6 mentioned this pull request Jan 5, 2018
9 tasks
@codablock codablock deleted the pr_backport_bitcoin_0.13-6 branch September 14, 2018 12:50
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.

5 participants