8000 doc, windows: CompanyName "Bitcoin" => "Bitcoin Core project" by hodlinator · Pull Request #32719 · bitcoin/bitcoin · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged
merged 1 commit into from
Jun 11, 2025

Conversation

hodlinator
Copy link
Contributor

Brings Windows executables in line with /share/setup.nsi.in:14 used by the installer.

Discovered while reviewing tangential PR: #32634 (comment)

@DrahtBot
Copy link
Contributor
DrahtBot commented Jun 10, 2025

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32719.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK maflcko, Sjors, janb84, hebasto

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

@hodlinator
Copy link
Contributor Author

Tooltip shown when hovering mouse cursor above bitcoin-qt.exe:

Before:
Before

After:
After

@maflcko
Copy link
Member
maflcko commented Jun 11, 2025

Makes sense to have the same metadata for the installer and the executables.

lgtm ACK 239fc4d

@Sjors
Copy link
Member
Sjors commented Jun 11, 2025

utACK 239fc4d

Copy link
Contributor
@janb84 janb84 left a 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. ✅

Copy link
Member
@hebasto hebasto left a comment

Choose a reason for hiding this comment

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

ACK 239fc4d.

@hebasto
Copy link
Member
hebasto commented Jun 11, 2025

My Guix build:

aarch64
458b3244f108716a6439c79c99f50da02eeec8e980e995455727fc134630e471  guix-build-239fc4d62e73/output/dist-archive/bitcoin-239fc4d62e73.tar.gz
19e34684107ea85baf7949d341dda007849860ff4341bc5139c1dd61a37a9c48  guix-build-239fc4d62e73/output/x86_64-w64-mingw32/SHA256SUMS.part
4515f2169e10825fa02a34aebd7c928ff366ee2481b4332180cd410fe5e56031  guix-build-239fc4d62e73/output/x86_64-w64-mingw32/bitcoin-239fc4d62e73-win64-codesigning.tar.gz
20a47574d6875275812c08cbbe108696f8761e62ba04f19b011c892a78d4a61c  guix-build-239fc4d62e73/output/x86_64-w64-mingw32/bitcoin-239fc4d62e73-win64-debug.zip
4fd6f3b1e79d3aad6e4ac6feb4169aa84924bb02de8ec754f7fa7c689ab09b98  guix-build-239fc4d62e73/output/x86_64-w64-mingw32/bitcoin-239fc4d62e73-win64-setup-unsigned.exe
781edc16d9974df92ff25ea23b6cf78abd6b0a2f1f7d11fa12f3ad96810d042a  guix-build-239fc4d62e73/output/x86_64-w64-mingw32/bitcoin-239fc4d62e73-win64-unsigned.zip

@hebasto hebasto merged commit c8abd97 into bitcoin:master Jun 11, 2025
19 checks passed
fanquake pushed a commit to fanquake/bitcoin that referenced this pull request Jun 11, 2025
Matches /share/setup.nsi.in:14

Github-Pull: bitcoin#32719
Rebased-From: 239fc4d
@fanquake fanquake mentioned this pull request Jun 11, 2025
@fanquake
Copy link
Member

Backported to 29.x in #32589.

@DrahtBot
Copy link
Contributor

Guix builds (on x86_64) [untrusted test-only build, possibly unsafe, not for production use]

File commit 4b8ac9e
(master)
commit b7689e0
(pull/32719/merge)
*-aarch64-linux-gnu-debug.tar.gz 95843d791689a0d6... 6640674e73a619bb...
*-aarch64-linux-gnu.tar.gz 5902b6b4f6e8acb5... bf7ace6eb4aeb23d...
*-arm-linux-gnueabihf-debug.tar.gz ae866e8a69a6a131... 2a8b58a16e39f46b...
*-arm-linux-gnueabihf.tar.gz d0ad05f39e0f3961... d3b469d10bcac3a9...
*-arm64-apple-darwin-codesigning.tar.gz bff03356722dedf7... 0e0f416c4968f747...
*-arm64-apple-darwin-unsigned.tar.gz 35903897cc214374... 9dcd40bf5f43abfb...
*-arm64-apple-darwin-unsigned.zip 184d537bbed13561... 6b69ac29cfdcd478...
*-powerpc64-linux-gnu-debug.tar.gz 7a8eaa57cfa61411... 80a4e9c6b2a28550...
*-powerpc64-linux-gnu.tar.gz 4d82633d664ec7e5... 1274f3a59229c887...
*-riscv64-linux-gnu-debug.tar.gz d60a68e9bbdca8e1... 454dfe3adb75fd4c...
*-riscv64-linux-gnu.tar.gz 428d2aabf5bb0127... 4111d9715884d984...
*-x86_64-apple-darwin-codesigning.tar.gz 12bdcf4eab32ebab... 49998fac5a57b532...
*-x86_64-apple-darwin-unsigned.tar.gz c3c1757df2637822... 04efb1fa9fed635e...
*-x86_64-apple-darwin-unsigned.zip b9babe4d47e333a9... 5e9b9fd36264be32...
*-x86_64-linux-gnu-debug.tar.gz 16ee30da81277705... 29f926a9cdc68aec...
*-x86_64-linux-gnu.tar.gz 65cd1bf1fc356db2... f7ad3e7a8a5febb4...
*.tar.gz 2ae252bdf1632a85... 57bdb8549739b81f...
SHA256SUMS.part 028db81f12d961c7... 7cd6dcc2c53e1a11...
guix_build.log bec9fa8833ef378d... 57f0866619e770ef...
guix_build.log.diff fb02fb170dd049f6...

@hodlinator hodlinator deleted the 2025/06/CompanyName branch June 12, 2025 19:55
fanquake pushed a commit to fanquake/bitcoin that referenced this pull request Jun 19, 2025
Matches /share/setup.nsi.in:14

Github-Pull: bitcoin#32719
Rebased-From: 239fc4d
fanquake added a commit that referenced this pull request Jun 25, 2025
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants
0