-
Notifications
You must be signed in to change notification settings - Fork 37.4k
doc, windows: CompanyName "Bitcoin" => "Bitcoin Core project" #32719
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
Conversation
Matches /share/setup.nsi.in:14
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. Code Coverage & BenchmarksFor details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32719. ReviewsSee the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update. |
Makes sense to have the same metadata for the installer and the executables. lgtm ACK 239fc4d |
utACK 239fc4d |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK 239fc4d
Not tested, did a code review. PR makes a (small) change to align the "company name", in several RC files, to the name used in the installer.
Imo this is a sensible change to keep those names aligned. ✅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 239fc4d.
My Guix build:
|
Matches /share/setup.nsi.in:14 Github-Pull: bitcoin#32719 Rebased-From: 239fc4d
Backported to 29.x in #32589. |
Guix builds (on x86_64) [untrusted test-only build, possibly unsafe, not for production use] |
Matches /share/setup.nsi.in:14 Github-Pull: bitcoin#32719 Rebased-From: 239fc4d
0922f6b doc: update release notes for 29.x (fanquake) 5697605 contrib: tracing: Correctly read msg type in p2p_monitor.py (David Gumberg) 4c7ed36 test: Fix list index out of range error in feature_bip68_sequence.py (zaidmstrr) 3e23b47 doc: fix transifex 404s (fanquake) 616baf3 doc: taproot became always active in v24.0 (Sjors Provoost) ef6111b depends: capnp 1.2.0 (fanquake) 8246c6a test: wallet, coverage for crash on dup block disconnection during unclean shutdown (Martin Zumsande) a18085a wallet: fix crash on double block disconnection (furszy) 7264459 build: patch cmake min version on freetype (josibake) 27c5330 doc, windows: CompanyName "Bitcoin" => "Bitcoin Core project" (Hodlinator) 1b51d75 depends: fix SHA256SUM command on OpenBSD (use GNU mode output) (Sebastian Falbesoner) aea8a39 doc: make `-DWITH_ZMQ=ON` explicit on `build-unix.md` (Luis Schwab) 23e76ef guix: warn and abort when SOURCE_DATE_EPOCH is set (will) 876a7b2 doc: add missing packages for BSDs (cmake, gmake, curl) to depends/README.md (Sebastian Falbesoner) 78688c8 rpc, doc: update `listdescriptors` RCP help (rkrux) c899334 rpc: Note in fundrawtransaction doc, fee rate is for package (benthecarman) 247ee59 doc: update tor docs to use bitcoind binary from path (ismaelsadeeq) 4a1143b depends: use "mkdir -p" when installing xproto (fanquake) 646fa1d test: fix sync function in rpc_psbt.py (Martin Zumsande) 17b31fc doc: Add missing top-level description to pruneblockchain RPC (nervana21) e34b6fb guix: accomodate migration to codeberg (fanquake) 142153e cmake: Add missed `SSE41_CXXFLAGS` (Hennadii Stepanov) Pull request description: Backports - #31757 - #32333 - #32439 - #32551 (just 800b7cc) - #32568 - #32607 - #32630 - #32678 - #32679 - #32690 (just 8713e80) - #32693 - #32696 - #32708 - #32711 - #32719 - #32760 - #32765 - #32771 - #32776 - #32777 Closes #32625. ACKs for top commit: instagibbs: ACK 0922f6b willcl-ark: ACK 0922f6b Tree-SHA512: 0389e5d85fa897fdbefd37635f6ec822ca5ab48a57c4d40fdd4d1be2465c676f514b0db4d72c962ee15e0090b27ff17701e167d660eaa25f855d06bbb1fe0e6e
Brings Windows executables in line with /share/setup.nsi.in:14 used by the installer.
Discovered while reviewing tangential PR: #32634 (comment)