Skip to content

Commit 41f67c7

Browse files
committed
Merge #121: Sync with the main repo
Pull request description: Sync with the main repo up to bitcoin/bitcoin@7190de9. **Note 1** The bitcoin/bitcoin@7190de9 was chosen to sync to make this PR manageable to review. The next commit in the main repo introduces a sophisticated conflicts which needs some special treatment. **Note 2** To make next syncs easier, a few groups of commits were rearranged when they were applied back. _Group A_ before: - 63e623c _"build, qml: Add qtdeclarative module"_ - 47c2e63 _"build, qml: Add qtquickcontrols2 module"_ - 1de8556 _"build, qt: Apply workaround for Qt build system bug for riscv64"_ - 6c58ae5 _"qml, doc: Add import QtQuick.Layouts"_ - 641f2a5 _"doc: Drop unneeded runtime dependencies"_ after: - 3fc6ec8 _"depends, qml: Add Qt 5.15.3 Declarative module"_ - c983645 _"depends, qml: Add Qt 5.15.3 Quick Controls 2 module"_ - 7902e08 _"build, qml: Update bitcoin_qt.m4 with QML paths and plugins"_ - 46af006 _"qml: Import static plugins"_ - d51ac6c _"doc, qml: Update runtime dependencies"_ - ae5a391 _"qml: Add import QtQuick.Layouts"_ _Group B_ before: - b8413e5 _"qml: Add qInstallMessageHandler"_ - 055e5c7 _"qml: Drop erroneous `#include <QStringLiteral>`"_ after: - d0a2ce6 _"qml: Add qInstallMessageHandler"_ _Group C_ before: - 6cb9fed _"ci, qml: Add required Qt Quick dev packages"_ - ea4f68a _"ci: Replace bionic with hirsute for system Qt 5.15.2"_ after: - a92f553 _"ci, qml: Add required Qt Quick dev packages"_ **Note 3** Commits: - 719e1f9 _"ci: temporarily disable failing ci task"_ - 651dd65 _"build, qml: Fix compiling with GCC 11"_ have been effectively dropped as they are no longer required. **Note 4** A new commit 229c56b _"ci: Fix Win64 task"_ were added. **Note 5** Errors in the ["previous releases..."](https://cirrus-ci.com/task/4752772586274816) CI task were solved by clearing CI cache. --- #### Guix builds: ``` $ find guix-build-$(git rev-parse --short=12 HEAD)/output/ -type f -print0 | env LC_ALL=C sort -z | xargs -r0 sha256sum 787ea22c1cba508827ce1f68d3b5645d0a7c0cabe2d04536c8fae6b9cf119c16 guix-build-229c56b3d90c/output/aarch64-linux-gnu/SHA256SUMS.part 6ee44b279593d8647c96d43a5855b5fec9f736e219d99d5e7478382c8e785161 guix-build-229c56b3d90c/output/aarch64-linux-gnu/bitcoin-229c56b3d90c-aarch64-linux-gnu-debug.tar.gz fc5142bbee75be4e3d50714ee578943a6e61943ee67c037ade78a763b7ad8fde guix-build-229c56b3d90c/output/aarch64-linux-gnu/bitcoin-229c56b3d90c-aarch64-linux-gnu.tar.gz 134f91d9b6aca45121087bbe12468b712b7e4171d92cc475800d5baaff7f2261 guix-build-229c56b3d90c/output/arm-linux-gnueabihf/SHA256SUMS.part 7220fa47fb8dc9f77ad988e6e57da3e9a5448ce8e2dda810162e90c719c2f844 guix-build-229c56b3d90c/output/arm-linux-gnueabihf/bitcoin-229c56b3d90c-arm-linux-gnueabihf-debug.tar.gz a2dde664a055ae34eca1b39a1b6f57f93499dad7c9fa23e10e9e69b6eb9de54e guix-build-229c56b3d90c/output/arm-linux-gnueabihf/bitcoin-229c56b3d90c-arm-linux-gnueabihf.tar.gz a44d56f3720e064fe189dea3db4bc3a5bb09641bc2d19da64fe14ea8e61ce6aa guix-build-229c56b3d90c/output/arm64-apple-darwin/SHA256SUMS.part b10309a84595bd991dcca098fad17e79c234f55ae3773fcccd535d19556c4938 guix-build-229c56b3d90c/output/arm64-apple-darwin/bitcoin-229c56b3d90c-arm64-apple-darwin-unsigned.dmg 655c8562a8246b66bfae8e37a96c15d2b424751b2b2ed6a1c47ce49bd3e4a927 guix-build-229c56b3d90c/output/arm64-apple-darwin/bitcoin-229c56b3d90c-arm64-apple-darwin-unsigned.tar.gz f9de01b39b0b0b81fe26f347e13d2496ada04a5d35ab2e1bd8d477dc2713f9d4 guix-build-229c56b3d90c/output/arm64-apple-darwin/bitcoin-229c56b3d90c-arm64-apple-darwin.tar.gz 54b65495a9ea44c924807babe2c33fbad38e826047860d9338c37509d395c822 guix-build-229c56b3d90c/output/dist-archive/bitcoin-229c56b3d90c.tar.gz c08b72fe9e74aac842ab99cad909dd8ef87a1e9fda377cec166dfaab7d5de13c guix-build-229c56b3d90c/output/powerpc64-linux-gnu/SHA256SUMS.part c1c69f324bc016874973a21c79c90d861243bace655ca54fe921ed7edf948129 guix-build-229c56b3d90c/output/powerpc64-linux-gnu/bitcoin-229c56b3d90c-powerpc64-linux-gnu-debug.tar.gz 3e809e29d35855b331a32f2ff33df4896bd522fe57b754803cd5dffdf98d94fb guix-build-229c56b3d90c/output/powerpc64-linux-gnu/bitcoin-229c56b3d90c-powerpc64-linux-gnu.tar.gz 7ac1aac7a2ddf1010a11afb878d6d0c241c0c1a3f5a15f7089581fffd24ce406 guix-build-229c56b3d90c/output/powerpc64le-linux-gnu/SHA256SUMS.part 73017251c9c00d72f7d5276f7ec8431682f49e2e59f9d8639505d8a6acbb8b10 guix-build-229c56b3d90c/output/powerpc64le-linux-gnu/bitcoin-229c56b3d90c-powerpc64le-linux-gnu-debug.tar.gz 81bab40a6d5f71f6d9f5a0a9d6f6aedfa0194738cc87214bf72a299e10a77f87 guix-build-229c56b3d90c/output/powerpc64le-linux-gnu/bitcoin-229c56b3d90c-powerpc64le-linux-gnu.tar.gz 7f3263abc0de641cd5cab29e4257ee4d740a995876c2adbc8e2163e9c9889fac guix-build-229c56b3d90c/output/riscv64-linux-gnu/SHA256SUMS.part 5b07bb4ee2bec25e6a503d78d4cbc847eb6da2a521b03a15c9e05529fb522621 guix-build-229c56b3d90c/output/riscv64-linux-gnu/bitcoin-229c56b3d90c-riscv64-linux-gnu-debug.tar.gz 309899e2d7f9fa96de84b5c77c65cba31e4cde8e7ed2d90f100f78459c5b73f7 guix-build-229c56b3d90c/output/riscv64-linux-gnu/bitcoin-229c56b3d90c-riscv64-linux-gnu.tar.gz f2f9b9f3e0934d2f9bdee7667812f0eefa3fd56bcbf3879ae7992084e38b3a01 guix-build-229c56b3d90c/output/x86_64-apple-darwin/SHA256SUMS.part 02a927161f15e1c766c4cfb007e280eb77ba651e818c74a63d0c780498ff9fce guix-build-229c56b3d90c/output/x86_64-apple-darwin/bitcoin-229c56b3d90c-x86_64-apple-darwin-unsigned.dmg 8bb2bfe96eafdbf50bdebdf73f355f9b56d410ea1d317a195b03c8872a74398b guix-build-229c56b3d90c/output/x86_64-apple-darwin/bitcoin-229c56b3d90c-x86_64-apple-darwin-unsigned.tar.gz d547fd75d8b1c8275ef1007221ee8ee98a7ef5cd8a341bd9049c6502ace477f1 guix-build-229c56b3d90c/output/x86_64-apple-darwin/bitcoin-229c56b3d90c-x86_64-apple-darwin.tar.gz 7e142a98140612b5fcc8a298348d7e452c2d1b58fe6d356c3a66baf9397db5fe guix-build-229c56b3d90c/output/x86_64-linux-gnu/SHA256SUMS.part 113bb237f595110d712194aeb480fed2db2f94e31ebfe087ba638f73ce2efe71 guix-build-229c56b3d90c/output/x86_64-linux-gnu/bitcoin-229c56b3d90c-x86_64-linux-gnu-debug.tar.gz 69e963b6a184751be8bad3df280e0d15c437da2c5a66cfc23b7cbf3ea09f7727 guix-build-229c56b3d90c/output/x86_64-linux-gnu/bitcoin-229c56b3d90c-x86_64-linux-gnu.tar.gz 5aa4a8ef55a511475975006e820f6f6e47ba5d961c26562f6673a3edd567146b guix-build-229c56b3d90c/output/x86_64-w64-mingw32/SHA256SUMS.part 9cd6abdbe502e1b96b5a98e5aff5bd17882024153dcf037188c931f96dab86b9 guix-build-229c56b3d90c/output/x86_64-w64-mingw32/bitcoin-229c56b3d90c-win64-debug.zip 0aa6fbf091fa49615c9f392ab9da2a41788b57740da28f712f1c27acaad9952d guix-build-229c56b3d90c/output/x86_64-w64-mingw32/bitcoin-229c56b3d90c-win64-setup-unsigned.exe a1ddd861d6a006c1083b44dd29adc8ca5f0881f8a89187ffc812d70414fe9b3d guix-build-229c56b3d90c/output/x86_64-w64-mingw32/bitcoin-229c56b3d90c-win64-unsigned.tar.gz 3d76a1103a4242cb5a31f8b4a4b2f297948ef4e6cb99967e8adf923d0c7f3b6d guix-build-229c56b3d90c/output/x86_64-w64-mingw32/bitcoin-229c56b3d90c-win64.zip ``` [![Windows](https://img.shields.io/badge/OS-Windows-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/win64/insecure_win_gui.zip?branch=pull/121) [![macOS](https://img.shields.io/badge/OS-macOS-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos/insecure_mac_gui.zip?branch=pull/121) [![Android](https://img.shields.io/badge/OS-Android-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/android/insecure_android_apk.zip?branch=pull/121) ACKs for top commit: jarolrod: ACK 229c56b Tree-SHA512: aed1914d5fba9b6e69c762852d4f471a2378977e69179026eaf550a317c9bac68940f84debdc5ab957c527d74faaa4ea867182b06ea0f85ad322852be3e8698a
2 parents 3629649 + 229c56b commit 41f67c7

File tree

832 files changed

+133046
-123300
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

832 files changed

+133046
-123300
lines changed

.cirrus.yml

+53-29
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ env: # Global defaults
22
PACKAGE_MANAGER_INSTALL: "apt-get update && apt-get install -y"
33
MAKEJOBS: "-j10"
44
TEST_RUNNER_PORT_MIN: "14000" # Must be larger than 12321, which is used for the http cache. See https://cirrus-ci.org/guide/writing-tasks/#http-cache
5+
CI_FAILFAST_TEST_LEAVE_DANGLING: "1" # Cirrus CI does not care about dangling process and setting this variable avoids killing the CI script itself on error
56
CCACHE_SIZE: "200M"
67
CCACHE_DIR: "/tmp/ccache_dir"
78
CCACHE_NOHASHDIR: "1" # Debug info might contain a stale path if the build dir changes, but this is fine
@@ -71,6 +72,19 @@ task:
7172
env:
7273
<< : *CIRRUS_EPHEMERAL_WORKER_TEMPLATE_ENV
7374

75+
task:
76+
name: 'tidy [jammy]'
77+
<< : *GLOBAL_TASK_TEMPLATE
78+
container:
79+
image: ubuntu:jammy
80+
cpu: 2
81+
memory: 5G
82+
# For faster CI feedback, immediately schedule the linters
83+
<< : *CREDITS_TEMPLATE
84+
env:
85+
<< : *CIRRUS_EPHEMERAL_WORKER_TEMPLATE_ENV
86+
FILE_ENV: "./ci/test/00_setup_env_native_tidy.sh"
87+
7488
task:
7589
name: "Win64 native [msvc]"
7690
<< : *FILTER_TEMPLATE
@@ -82,14 +96,17 @@ task:
8296
env:
8397
PATH: 'C:\jom;C:\Python39;C:\Python39\Scripts;C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\MSBuild\Current\Bin;%PATH%'
8498
PYTHONUTF8: 1
85-
CI_VCPKG_TAG: '2021.05.12'
99+
CI_VCPKG_TAG: '2022.04.12'
86100
VCPKG_DOWNLOADS: 'C:\Users\ContainerAdministrator\AppData\Local\vcpkg\downloads'
87101
VCPKG_DEFAULT_BINARY_CACHE: 'C:\Users\ContainerAdministrator\AppData\Local\vcpkg\archives'
88-
QT_DOWNLOAD_URL: 'https://download.qt.io/official_releases/qt/5.15/5.15.2/single/qt-everywhere-src-5.15.2.zip'
89-
QT_LOCAL_PATH: 'C:\qt-everywhere-src-5.15.2.zip'
90-
QT_SOURCE_DIR: 'C:\qt-everywhere-src-5.15.2'
102+
CCACHE_DIR: 'C:\Users\ContainerAdministrator\AppData\Local\ccache'
103+
WRAPPED_CL: 'C:\Users\ContainerAdministrator\AppData\Local\Temp\cirrus-ci-build\ci\test\wrapped-cl.bat'
104+
QT_DOWNLOAD_URL: 'https://download.qt.io/official_releases/qt/5.15/5.15.3/single/qt-everywhere-opensource-src-5.15.3.zip'
105+
QT_LOCAL_PATH: 'C:\qt-everywhere-opensource-src-5.15.3.zip'
106+
QT_SOURCE_DIR: 'C:\qt-everywhere-src-5.15.3'
91107
QTBASEDIR: 'C:\Qt_static'
92108
x64_NATIVE_TOOLS: '"C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\VC\Auxiliary\Build\vcvars64.bat"'
109+
QT_CONFIGURE_COMMAND: '..\configure -release -silent -opensource -confirm-license -opengl desktop -static -static-runtime -mp -qt-zlib -qt-pcre -qt-libpng -nomake examples -nomake tests -nomake tools -no-angle -no-dbus -no-gif -no-gtk -no-ico -no-icu -no-libjpeg -no-libudev -no-sql-sqlite -no-sql-odbc -no-sqlite -no-vulkan -skip qt3d -skip qtactiveqt -skip qtandroidextras -skip qtcharts -skip qtconnectivity -skip qtdatavis3d -skip qtdeclarative -skip doc -skip qtdoc -skip qtgamepad -skip qtgraphicaleffects -skip qtimageformats -skip qtlocation -skip qtlottie -skip qtmacextras -skip qtmultimedia -skip qtnetworkauth -skip qtpurchasing -skip qtquick3d -skip qtquickcontrols -skip qtquickcontrols2 -skip qtquicktimeline -skip qtremoteobjects -skip qtscript -skip qtscxml -skip qtsensors -skip qtserialbus -skip qtserialport -skip qtspeech -skip qtsvg -skip qtvirtualkeyboard -skip qtwayland -skip qtwebchannel -skip qtwebengine -skip qtwebglplugin -skip qtwebsockets -skip qtwebview -skip qtx11extras -skip qtxmlpatterns -no-openssl -no-feature-bearermanagement -no-feature-printdialog -no-feature-printer -no-feature-printpreviewdialog -no-feature-printpreviewwidget -no-feature-sql -no-feature-sqlmodel -no-feature-textbrowser -no-feature-textmarkdownwriter -no-feature-textodfwriter -no-feature-xml'
93110
IgnoreWarnIntDirInTempDetected: 'true'
94111
merge_script:
95112
- git config --global user.email "[email protected]"
@@ -104,7 +121,7 @@ task:
104121
folder: "%QTBASEDIR%"
105122
reupload_on_changes: false
106123
fingerprint_script:
107-
- echo %QT_DOWNLOAD_URL%
124+
- echo %QT_DOWNLOAD_URL% %QT_CONFIGURE_COMMAND%
108125
- msbuild -version
109126
populate_script:
110127
- curl -L -o C:\jom.zip http://download.qt.io/official_releases/jom/jom.zip
@@ -116,7 +133,7 @@ task:
116133
- cd %QT_SOURCE_DIR%
117134
- mkdir build
118135
- cd build
119-
- ..\configure -release -silent -opensource -confirm-license -opengl desktop -static -static-runtime -mp -qt-zlib -qt-pcre -qt-libpng -nomake examples -nomake tests -nomake tools -no-angle -no-dbus -no-gif -no-gtk -no-ico -no-icu -no-libjpeg -no-libudev -no-sql-sqlite -no-sql-odbc -no-sqlite -no-vulkan -skip qt3d -skip qtactiveqt -skip qtandroidextras -skip qtcharts -skip qtconnectivity -skip qtdatavis3d -skip qtdeclarative -skip doc -skip qtdoc -skip qtgamepad -skip qtgraphicaleffects -skip qtimageformats -skip qtlocation -skip qtlottie -skip qtmacextras -skip qtmultimedia -skip qtnetworkauth -skip qtpurchasing -skip qtquick3d -skip qtquickcontrols -skip qtquickcontrols2 -skip qtquicktimeline -skip qtremoteobjects -skip qtscript -skip qtscxml -skip qtsensors -skip qtserialbus -skip qtserialport -skip qtspeech -skip qtsvg -skip qtvirtualkeyboard -skip qtwayland -skip qtwebchannel -skip qtwebengine -skip qtwebglplugin -skip qtwebsockets -skip qtwebview -skip qtx11extras -skip qtxmlpatterns -no-openssl -no-feature-bearermanagement -no-feature-printdialog -no-feature-printer -no-feature-printpreviewdialog -no-feature-printpreviewwidget -no-feature-sql -no-feature-sqlmodel -no-feature-textbrowser -no-feature-textmarkdownwriter -no-feature-textodfwriter -no-feature-xml -prefix %QTBASEDIR%
136+
- '%QT_CONFIGURE_COMMAND% -prefix %QTBASEDIR%'
120137
- jom
121138
- jom install
122139
vcpkg_tools_cache:
@@ -130,12 +147,17 @@ task:
130147
reupload_on_changes: true
131148
fingerprint_script:
132149
- echo %CI_VCPKG_TAG%
150+
- type build_msvc\vcpkg.json
133151
- msbuild -version
134152
populate_script:
135153
- mkdir %VCPKG_DEFAULT_BINARY_CACHE%
136-
install_python_script:
154+
ccache_cache:
155+
folder: '%CCACHE_DIR%'
156+
install_tools_script:
157+
- choco install --yes --no-progress ccache
137158
- choco install --yes --no-progress python3 --version=3.9.6
138159
- pip install zmq
160+
- ccache --version
139161
- python -VV
140162
install_vcpkg_script:
141163
- cd ..
@@ -147,9 +169,12 @@ task:
147169
- .\vcpkg integrate install
148170
- .\vcpkg version
149171
build_script:
172+
- '%x64_NATIVE_TOOLS%'
150173
- cd %CIRRUS_WORKING_DIR%
174+
- ccache --zero-stats
151175
- python build_msvc\msvc-autogen.py
152-
- msbuild build_msvc\bitcoin.sln -property:Configuration=Release -maxCpuCount -verbosity:minimal -noLogo
176+
- msbuild build_msvc\bitcoin.sln -property:CLToolExe=%WRAPPED_CL% -property:Configuration=Release -maxCpuCount -verbosity:minimal -noLogo
177+
- ccache --show-stats
153178
unit_tests_script:
154179
- src\test_bitcoin.exe -l test_suite
155180
- src\bench_bitcoin.exe > NUL
@@ -176,11 +201,11 @@ task:
176201
QEMU_USER_CMD: "" # Disable qemu and run the test natively
177202

178203
task:
179-
name: 'Win64 [unit tests, no gui tests, no boost::process, no functional tests] [focal]'
204+
name: 'Win64 [unit tests, no gui tests, no boost::process, no functional tests] [jammy]'
180205
<< : *GLOBAL_TASK_TEMPLATE
181206
alias: win64
182207
container:
183-
image: ubuntu:focal
208+
image: ubuntu:jammy
184209
env:
185210
<< : *CIRRUS_EPHEMERAL_WORKER_TEMPLATE_ENV
186211
FILE_ENV: "./ci/test/00_setup_env_win64.sh"
@@ -189,23 +214,23 @@ task:
189214
insecure_win_gui_artifacts:
190215
path: "insecure_win_gui.exe"
191216

192-
#task:
193-
# name: '32-bit + dash [gui] [CentOS 8]'
194-
# << : *GLOBAL_TASK_TEMPLATE
195-
# container:
196-
# image: centos:8
197-
# env:
198-
# << : *CIRRUS_EPHEMERAL_WORKER_TEMPLATE_ENV
199-
# PACKAGE_MANAGER_INSTALL: "yum install -y"
200-
# FILE_ENV: "./ci/test/00_setup_env_i686_centos.sh"
217+
task:
218+
name: '32-bit + dash [gui] [CentOS 8]'
219+
<< : *GLOBAL_TASK_TEMPLATE
220+
container:
221+
image: quay.io/centos/centos:stream8
222+
env:
223+
<< : *CIRRUS_EPHEMERAL_WORKER_TEMPLATE_ENV
224+
PACKAGE_MANAGER_INSTALL: "yum install -y"
225+
FILE_ENV: "./ci/test/00_setup_env_i686_centos.sh"
201226

202227
task:
203-
name: '[previous releases, uses qt5 dev package and some depends packages, DEBUG] [unsigned char] [hirsute]'
228+
name: '[previous releases, uses qt5 dev package and some depends packages, DEBUG] [unsigned char] [bullseye]'
204229
previous_releases_cache:
205230
folder: "releases"
206231
<< : *GLOBAL_TASK_TEMPLATE
207232
container:
208-
image: ubuntu:hirsute
233+
image: debian:bullseye
209234
env:
210235
<< : *CIRRUS_EPHEMERAL_WORKER_TEMPLATE_ENV
211236
FILE_ENV: "./ci/test/00_setup_env_native_qt5.sh"
@@ -242,11 +267,10 @@ task:
242267
MAKEJOBS: "-j4" # Avoid excessive memory use
243268

244269
task:
245-
name: '[fuzzer,address,undefined,integer, no depends] [focal]'
246-
only_if: $CIRRUS_BRANCH == $CIRRUS_DEFAULT_BRANCH || $CIRRUS_BASE_BRANCH == $CIRRUS_DEFAULT_BRANCH
270+
name: '[fuzzer,address,undefined,integer, no depends] [jammy]'
247271
<< : *GLOBAL_TASK_TEMPLATE
248272
container:
249-
image: ubuntu:focal
273+
image: ubuntu:jammy
250274
cpu: 4 # Increase CPU and memory to avoid timeout
251275
memory: 16G
252276
env:
@@ -265,13 +289,13 @@ task:
265289
FILE_ENV: "./ci/test/00_setup_env_i686_multiprocess.sh"
266290

267291
task:
268-
name: '[no wallet] [bionic]'
292+
name: '[no wallet, libbitcoinkernel] [bionic]'
269293
<< : *GLOBAL_TASK_TEMPLATE
270294
container:
271295
image: ubuntu:bionic
272296
env:
273297
<< : *CIRRUS_EPHEMERAL_WORKER_TEMPLATE_ENV
274-
FILE_ENV: "./ci/test/00_setup_env_native_nowallet.sh"
298+
FILE_ENV: "./ci/test/00_setup_env_native_nowallet_libbitcoinkernel.sh"
275299

276300
task:
277301
name: 'macOS 10.15 [gui, no tests] [focal]'
@@ -284,7 +308,7 @@ task:
284308
container:
285309
image: ubuntu:focal
286310
env:
287-
MACOS_SDK: "Xcode-12.1-12A7403-extracted-SDK-with-libcxx-headers"
311+
MACOS_SDK: "Xcode-12.2-12B45b-extracted-SDK-with-libcxx-headers"
288312
<< : *CIRRUS_EPHEMERAL_WORKER_TEMPLATE_ENV
289313
FILE_ENV: "./ci/test/00_setup_env_mac.sh"
290314
copy_artifacts_script:
@@ -299,9 +323,9 @@ task:
299323
brew_install_script:
300324
- brew install boost libevent qt@5 miniupnpc libnatpmp ccache zeromq qrencode libtool automake gnu-getopt
301325
<< : *GLOBAL_TASK_TEMPLATE
302-
osx_instance:
326+
macos_instance:
303327
# Use latest image, but hardcode version to avoid silent upgrades (and breaks)
304-
image: monterey-xcode-13.2 # https://cirrus-ci.org/guide/macOS
328+
image: monterey-xcode-13.3 # https://cirrus-ci.org/guide/macOS
305329
env:
306330
<< : *CIRRUS_EPHEMERAL_WORKER_TEMPLATE_ENV
307331
CI_USE_APT_INSTALL: "no"

.gitignore

+3
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ src/bitcoin-gui
99
src/bitcoin-node
1010
src/bitcoin-tx
1111
src/bitcoin-util
12+
src/bitcoin-chainstate
1213
src/bitcoin-wallet
1314
src/test/fuzz/fuzz
1415
src/test/test_bitcoin
@@ -150,3 +151,5 @@ osx_volname
150151
dist/
151152

152153
/guix-build-*
154+
155+
/ci/scratch/

.tx/config

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
[main]
22
host = https://www.transifex.com
33

4-
[bitcoin.qt-translation-022x]
4+
[bitcoin.qt-translation-023x]
55
file_filter = src/qt/locale/bitcoin_<lang>.xlf
66
source_file = src/qt/locale/bitcoin_en.xlf
77
source_lang = en

CONTRIBUTING.md

+19-10
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@ revolves around a meritocracy where contributors earn trust from the developer
1212
community over time. Nevertheless, some hierarchy is necessary for practical
1313
purposes. As such, there are repository "maintainers" who are responsible for
1414
merging pull requests, as well as a "lead maintainer" who is responsible for the
15-
release cycle as well as overall merging, moderation and appointment of
16-
maintainers.
15+
[release cycle](/doc/release-process.md) as well as overall merging, moderation
16+
and appointment of maintainers.
1717

1818
Getting Started
1919
---------------
@@ -153,7 +153,8 @@ the pull request affects. Valid areas as:
153153
- `test`, `qa` or `ci` for changes to the unit tests, QA tests or CI code
154154
- `util` or `lib` for changes to the utils or libraries
155155
- `wallet` for changes to the wallet code
156-
- `build` for changes to the GNU Autotools or reproducible builds
156+
- `build` for changes to the GNU Autotools or MSVC builds
157+
- `guix` for changes to the GUIX reproducible builds
157158

158159
Examples:
159160

@@ -189,9 +190,14 @@ in the body of the pull request to indicate tasks are pending.
189190

190191
At this stage, one should expect comments and review from other contributors. You
191192
can add more commits to your pull request by committing them locally and pushing
192-
to your fork until you have satisfied all feedback.
193+
to your fork.
193194

194-
Note: Code review is a burdensome but important part of the development process, and as such, certain types of pull requests are rejected. In general, if the **improvements** do not warrant the **review effort** required, the PR has a high chance of being rejected. It is up to the PR author to convince the reviewers that the changes warrant the review effort, and if reviewers are "Concept NACK'ing" the PR, the author may need to present arguments and/or do research backing their suggested changes.
195+
You are expected to reply to any review comments before your pull request is
196+
merged. You may update the code or reject the feedback if you do not agree with
197+
it, but you should express so in a reply. If there is outstanding feedback and
198+
you are not actively working on it, your pull request may be closed.
199+
200+
Please refer to the [peer review](#peer-review) section below for more details.
195201

196202
### Squashing Commits
197203

@@ -322,6 +328,14 @@ maintainers take into account the peer review when determining if there is
322328
consensus to merge a pull request (remember that discussions may have been
323329
spread out over GitHub, mailing list and IRC discussions).
324330

331+
Code review is a burdensome but important part of the development process, and
332+
as such, certain types of pull requests are rejected. In general, if the
333+
**improvements** do not warrant the **review effort** required, the PR has a
334+
high chance of being rejected. It is up to the PR author to convince the
335+
reviewers that the changes warrant the review effort, and if reviewers are
336+
"Concept NACK'ing" the PR, the author may need to present arguments and/or do
337+
research backing their suggested changes.
338+
325339
#### Conceptual Review
326340

327341
A review can be a conceptual review, where the reviewer leaves a comment
@@ -422,11 +436,6 @@ https://github.com/bitcoin/bitcoin/pull/16189).
422436
Also see the [backport.py script](
423437
https://github.com/bitcoin-core/bitcoin-maintainer-tools#backport).
424438

425-
Release Policy
426-
--------------
427-
428-
The project leader is the release manager for each Bitcoin Core release.
429-
430439
Copyright
431440
---------
432441

Makefile.am

+4-16
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ OSX_APP=Bitcoin-Qt.app
3838
OSX_VOLNAME = $(subst $(space),-,$(PACKAGE_NAME))
3939
OSX_DMG = $(OSX_VOLNAME).dmg
4040
OSX_TEMP_ISO = $(OSX_DMG:.dmg=).temp.iso
41-
OSX_BACKGROUND_IMAGE=$(top_srcdir)/contrib/macdeploy/background.tiff
4241
OSX_DEPLOY_SCRIPT=$(top_srcdir)/contrib/macdeploy/macdeployqtplus
4342
OSX_INSTALLER_ICONS=$(top_srcdir)/src/qt/res/icons/bitcoin.icns
4443
OSX_PLIST=$(top_builddir)/share/qt/Info.plist #not installed
@@ -48,7 +47,8 @@ DIST_CONTRIB = \
4847
$(top_srcdir)/test/sanitizer_suppressions/tsan \
4948
$(top_srcdir)/test/sanitizer_suppressions/ubsan \
5049
$(top_srcdir)/contrib/linearize/linearize-data.py \
51-
$(top_srcdir)/contrib/linearize/linearize-hashes.py
50+
$(top_srcdir)/contrib/linearize/linearize-hashes.py \
51+
$(top_srcdir)/contrib/signet/miner
5252

5353
DIST_SHARE = \
5454
$(top_srcdir)/share/genbuild.sh \
@@ -64,7 +64,6 @@ WINDOWS_PACKAGING = $(top_srcdir)/share/pixmaps/bitcoin.ico \
6464
$(top_srcdir)/doc/README_windows.txt
6565

6666
OSX_PACKAGING = $(OSX_DEPLOY_SCRIPT) $(OSX_INSTALLER_ICONS) \
67-
$(top_srcdir)/contrib/macdeploy/detached-sig-apply.sh \
6867
$(top_srcdir)/contrib/macdeploy/detached-sig-create.sh
6968

7069
COVERAGE_INFO = $(COV_TOOL_WRAPPER) baseline.info \
@@ -130,28 +129,17 @@ $(OSX_DMG): $(OSX_APP_BUILT) $(OSX_PACKAGING)
130129
deploydir: $(OSX_DMG)
131130
else !BUILD_DARWIN
132131
APP_DIST_DIR=$(top_builddir)/dist
133-
APP_DIST_EXTRAS=$(APP_DIST_DIR)/.background/background.tiff $(APP_DIST_DIR)/.DS_Store $(APP_DIST_DIR)/Applications
134132

135-
$(APP_DIST_DIR)/Applications:
136-
@rm -f $@
137-
@cd $(@D); $(LN_S) /Applications $(@F)
138-
139-
$(APP_DIST_EXTRAS): $(APP_DIST_DIR)/$(OSX_APP)/Contents/MacOS/Bitcoin-Qt
140-
141-
$(OSX_TEMP_ISO): $(APP_DIST_EXTRAS)
133+
$(OSX_TEMP_ISO): $(APP_DIST_DIR)/$(OSX_APP)/Contents/MacOS/Bitcoin-Qt
142134
$(XORRISOFS) -D -l -V "$(OSX_VOLNAME)" -no-pad -r -dir-mode 0755 -o $@ $(APP_DIST_DIR) -- $(if $(SOURCE_DATE_EPOCH),-volume_date all_file_dates =$(SOURCE_DATE_EPOCH))
143135

144136
$(OSX_DMG): $(OSX_TEMP_ISO)
145137
$(DMG) dmg "$<" "$@"
146138

147-
$(APP_DIST_DIR)/.background/background.tiff:
148-
$(MKDIR_P) $(@D)
149-
cp $(OSX_BACKGROUND_IMAGE) $@
150-
151139
$(APP_DIST_DIR)/$(OSX_APP)/Contents/MacOS/Bitcoin-Qt: $(OSX_APP_BUILT) $(OSX_PACKAGING)
152140
INSTALLNAMETOOL=$(INSTALLNAMETOOL) OTOOL=$(OTOOL) STRIP=$(STRIP) $(PYTHON) $(OSX_DEPLOY_SCRIPT) $(OSX_APP) $(OSX_VOLNAME) -translations-dir=$(QT_TRANSLATION_DIR)
153141

154-
deploydir: $(APP_DIST_EXTRAS)
142+
deploydir: $(APP_DIST_DIR)/$(OSX_APP)/Contents/MacOS/Bitcoin-Qt
155143
endif !BUILD_DARWIN
156144

157145
appbundle: $(OSX_APP_BUILT)

REVIEWERS

+1-2
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,11 @@
1818

1919

2020
# Maintainers
21+
# @achow101
2122
# @fanquake
2223
# @hebasto
23-
# @jonasschnelli
2424
# @laanwj
2525
# @marcofalke
26-
# @meshcollider
2726
# @sipa
2827

2928
# Docs

0 commit comments

Comments
 (0)