Skip to content

Commit 332655c

Browse files
hebastohebasto
committed
build: Rename PACKAGE_* variables to CLIENT_*
The use of `PACKAGE_NAME` for the project's variable name is problematic, as this name is commonly used in CMake's interface variables. If third-party CMake code handles with scopes improperly, our `PACKAGE_NAME` variable could end up with an unexpected value. This change avoids such conflicts by renaming all `PACKAGE_*` variables to `CLIENT_*`.
1 parent e6e29e3 commit 332655c

File tree

8 files changed

+38
-38
lines changed

8 files changed

+38
-38
lines changed

‎CMakeLists.txt‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ endif()
1616
#=============================
1717
# Project / Package metadata
1818
#=============================
19-
set(PACKAGE_NAME "Bitcoin Core")
19+
set(CLIENT_NAME "Bitcoin Core")
2020
set(CLIENT_VERSION_MAJOR 28)
2121
set(CLIENT_VERSION_MINOR 99)
2222
set(CLIENT_VERSION_BUILD 0)
@@ -47,14 +47,14 @@ project(BitcoinCore
4747
LANGUAGES NONE
4848
)
4949

50-
set(PACKAGE_VERSION ${PROJECT_VERSION})
50+
set(CLIENT_VERSION_STRING ${PROJECT_VERSION})
5151
if(CLIENT_VERSION_RC GREATER 0)
52-
string(APPEND PACKAGE_VERSION "rc${CLIENT_VERSION_RC}")
52+
string(APPEND CLIENT_VERSION_STRING "rc${CLIENT_VERSION_RC}")
5353
endif()
5454

5555
set(COPYRIGHT_HOLDERS "The %s developers")
56-
set(COPYRIGHT_HOLDERS_FINAL "The ${PACKAGE_NAME} developers")
57-
set(PACKAGE_BUGREPORT "https://github.com/bitcoin/bitcoin/issues")
56+
set(COPYRIGHT_HOLDERS_FINAL "The ${CLIENT_NAME} developers")
57+
set(CLIENT_BUGREPORT "https://github.com/bitcoin/bitcoin/issues")
5858

5959
#=============================
6060
# Language setup

‎cmake/bitcoin-build-config.h.in‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
#define COPYRIGHT_HOLDERS_FINAL "@COPYRIGHT_HOLDERS_FINAL@"
2525

2626
/* Replacement for %s in copyright holders string */
27-
#define COPYRIGHT_HOLDERS_SUBSTITUTION "@PACKAGE_NAME@"
27+
#define COPYRIGHT_HOLDERS_SUBSTITUTION "@CLIENT_NAME@"
2828

2929
/* Copyright year */
3030
#define COPYRIGHT_YEAR @COPYRIGHT_YEAR@
@@ -124,16 +124,16 @@
124124
#cmakedefine HAVE_VM_VM_PARAM_H 1
125125

126126
/* Define to the address where bug reports for this package should be sent. */
127-
#define PACKAGE_BUGREPORT "@PACKAGE_BUGREPORT@"
127+
#define PACKAGE_BUGREPORT "@CLIENT_BUGREPORT@"
128128

129129
/* Define to the full name of this package. */
130-
#define PACKAGE_NAME "@PACKAGE_NAME@"
130+
#define PACKAGE_NAME "@CLIENT_NAME@"
131131

132132
/* Define to the home page for this package. */
133133
#define PACKAGE_URL "@PROJECT_HOMEPAGE_URL@"
134134

135135
/* Define to the version of this package. */
136-
#define PACKAGE_VERSION "@PACKAGE_VERSION@"
136+
#define PACKAGE_VERSION "@CLIENT_VERSION_STRING@"
137137

138138
/* Define to 1 if strerror_r returns char *. */
139139
#cmakedefine STRERROR_R_CHAR_P 1

‎cmake/module/GenerateSetupNsi.cmake‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55
function(generate_setup_nsi)
66
set(abs_top_srcdir ${PROJECT_SOURCE_DIR})
77
set(abs_top_builddir ${PROJECT_BINARY_DIR})
8-
set(PACKAGE_URL ${PROJECT_HOMEPAGE_URL})
9-
set(PACKAGE_TARNAME "bitcoin")
8+
set(CLIENT_URL ${PROJECT_HOMEPAGE_URL})
9+
set(CLIENT_TARNAME "bitcoin")
1010
set(BITCOIN_GUI_NAME "bitcoin-qt")
1111
set(BITCOIN_DAEMON_NAME "bitcoind")
1212
set(BITCOIN_CLI_NAME "bitcoin-cli")

‎cmake/module/Maintenance.cmake‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ function(add_macos_deploy_target)
9898
file(CONFIGURE OUTPUT ${macos_app}/Contents/Resources/empty.lproj CONTENT "")
9999
configure_file(${PROJECT_SOURCE_DIR}/src/qt/res/icons/bitcoin.icns ${macos_app}/Contents/Resources/bitcoin.icns NO_SOURCE_PERMISSIONS COPYONLY)
100100
file(CONFIGURE OUTPUT ${macos_app}/Contents/Resources/Base.lproj/InfoPlist.strings
101-
CONTENT "{ CFBundleDisplayName = \"@PACKAGE_NAME@\"; CFBundleName = \"@PACKAGE_NAME@\"; }"
101+
CONTENT "{ CFBundleDisplayName = \"@CLIENT_NAME@\"; CFBundleName = \"@CLIENT_NAME@\"; }"
102102
)
103103

104104
add_custom_command(
@@ -109,7 +109,7 @@ function(add_macos_deploy_target)
109109
VERBATIM
110110
)
111111

112-
string(REPLACE " " "-" osx_volname ${PACKAGE_NAME})
112+
string(REPLACE " " "-" osx_volname ${CLIENT_NAME})
113113
if(CMAKE_HOST_APPLE)
114114
add_custom_command(
115115
OUTPUT ${PROJECT_BINARY_DIR}/${osx_volname}.zip

‎doc/Doxyfile.in‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ PROJECT_NAME = "Bitcoin Core"
3838
# could be handy for archiving the generated documentation or if some version
3939
# control system is used.
4040

41-
PROJECT_NUMBER = @PACKAGE_VERSION@
41+
PROJECT_NUMBER = @CLIENT_VERSION_STRING@
4242

4343
# Using the PROJECT_BRIEF tag one can provide an optional one line description
4444
# for a project that appears at the top of each page and should give viewer a

‎libbitcoinkernel.pc.in‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,9 @@ exec_prefix=${prefix}
33
libdir=${prefix}/@CMAKE_INSTALL_LIBDIR@
44
includedir=${prefix}/@CMAKE_INSTALL_INCLUDEDIR@
55

6-
Name: @PACKAGE_NAME@ kernel library
6+
Name: @CLIENT_NAME@ kernel library
77
Description: Experimental library for the Bitcoin Core validation engine.
8-
Version: @PACKAGE_VERSION@
8+
Version: @CLIENT_VERSION_STRING@
99
Libs: -L${libdir} -lbitcoinkernel
1010
Libs.private: -L${libdir} @LIBS_PRIVATE@
1111
Cflags: -I${includedir}

‎share/setup.nsi.in‎

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
Name "@PACKAGE_NAME@ (64-bit)"
1+
Name "@CLIENT_NAME@ (64-bit)"
22

33
RequestExecutionLevel highest
44
SetCompressor /SOLID lzma
@@ -11,8 +11,8 @@ Unicode true
1111

1212
# General Symbol Definitions
1313
!define REGKEY "SOFTWARE\$(^Name)"
14-
!define COMPANY "@PACKAGE_NAME@ project"
15-
!define URL @PACKAGE_URL@
14+
!define COMPANY "@CLIENT_NAME@ project"
15+
!define URL @CLIENT_URL@
1616

1717
# MUI Symbol Definitions
1818
!define MUI_ICON "@abs_top_srcdir@/share/pixmaps/bitcoin.ico"
@@ -24,7 +24,7 @@ Unicode true
2424
!define MUI_STARTMENUPAGE_REGISTRY_ROOT HKLM
2525
!define MUI_STARTMENUPAGE_REGISTRY_KEY ${REGKEY}
2626
!define MUI_STARTMENUPAGE_REGISTRY_VALUENAME StartMenuGroup
27-
!define MUI_STARTMENUPAGE_DEFAULTFOLDER "@PACKAGE_NAME@"
27+
!define MUI_STARTMENUPAGE_DEFAULTFOLDER "@CLIENT_NAME@"
2828
!define MUI_FINISHPAGE_RUN "$WINDIR\explorer.exe"
2929
!define MUI_FINISHPAGE_RUN_PARAMETERS $INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@
3030
!define MUI_UNICON "${NSISDIR}\Contrib\Graphics\Icons\modern-uninstall.ico"
@@ -58,12 +58,12 @@ XPStyle on
5858
BrandingText " "
5959
ShowInstDetails show
6060
VIProductVersion @CLIENT_VERSION_MAJOR@.@CLIENT_VERSION_MINOR@.@[email protected]
61-
VIAddVersionKey ProductName "@PACKAGE_NAME@"
62-
VIAddVersionKey ProductVersion "@PACKAGE_VERSION@"
61+
VIAddVersionKey ProductName "@CLIENT_NAME@"
62+
VIAddVersionKey ProductVersion "@CLIENT_VERSION_STRING@"
6363
VIAddVersionKey CompanyName "${COMPANY}"
6464
VIAddVersionKey CompanyWebsite "${URL}"
65-
VIAddVersionKey FileVersion "@PACKAGE_VERSION@"
66-
VIAddVersionKey FileDescription "Installer for @PACKAGE_NAME@"
65+
VIAddVersionKey FileVersion "@CLIENT_VERSION_STRING@"
66+
VIAddVersionKey FileDescription "Installer for @CLIENT_NAME@"
6767
VIAddVersionKey LegalCopyright "Copyright (C) 2009-@COPYRIGHT_YEAR@ @COPYRIGHT_HOLDERS_FINAL@"
6868
InstallDirRegKey HKCU "${REGKEY}" Path
6969
ShowUninstDetails show
@@ -95,23 +95,23 @@ Section -post SEC0001
9595
!insertmacro MUI_STARTMENU_WRITE_BEGIN Application
9696
CreateDirectory $SMPROGRAMS\$StartMenuGroup
9797
CreateShortcut "$SMPROGRAMS\$StartMenuGroup\$(^Name).lnk" $INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@
98-
CreateShortcut "$SMPROGRAMS\$StartMenuGroup\@PACKAGE_NAME@ (testnet).lnk" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" "-testnet" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" 1
99-
CreateShortcut "$SMPROGRAMS\$StartMenuGroup\@PACKAGE_NAME@ (test signet).lnk" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" "-signet" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" 2
100-
CreateShortcut "$SMPROGRAMS\$StartMenuGroup\@PACKAGE_NAME@ (testnet4).lnk" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" "-testnet4" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" 3
98+
CreateShortcut "$SMPROGRAMS\$StartMenuGroup\@CLIENT_NAME@ (testnet).lnk" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" "-testnet" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" 1
99+
CreateShortcut "$SMPROGRAMS\$StartMenuGroup\@CLIENT_NAME@ (test signet).lnk" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" "-signet" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" 2
100+
CreateShortcut "$SMPROGRAMS\$StartMenuGroup\@CLIENT_NAME@ (testnet4).lnk" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" "-testnet4" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" 3
101101
CreateShortcut "$SMPROGRAMS\$StartMenuGroup\Uninstall $(^Name).lnk" $INSTDIR\uninstall.exe
102102
!insertmacro MUI_STARTMENU_WRITE_END
103103
WriteRegStr HKCU "SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\$(^Name)" DisplayName "$(^Name)"
104-
WriteRegStr HKCU "SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\$(^Name)" DisplayVersion "@PACKAGE_VERSION@"
104+
WriteRegStr HKCU "SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\$(^Name)" DisplayVersion "@CLIENT_VERSION_STRING@"
105105
WriteRegStr HKCU "SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\$(^Name)" Publisher "${COMPANY}"
106106
WriteRegStr HKCU "SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\$(^Name)" URLInfoAbout "${URL}"
107107
WriteRegStr HKCU "SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\$(^Name)" DisplayIcon $INSTDIR\bitcoin-qt.exe
108108
WriteRegStr HKCU "SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\$(^Name)" UninstallString $INSTDIR\uninstall.exe
109109
WriteRegDWORD HKCU "SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\$(^Name)" NoModify 1
110110
WriteRegDWORD HKCU "SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\$(^Name)" NoRepair 1
111-
WriteRegStr HKCR "@PACKAGE_TARNAME@" "URL Protocol" ""
112-
WriteRegStr HKCR "@PACKAGE_TARNAME@" "" "URL:Bitcoin"
113-
WriteRegStr HKCR "@PACKAGE_TARNAME@\DefaultIcon" "" $INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@
114-
WriteRegStr HKCR "@PACKAGE_TARNAME@\shell\open\command" "" '"$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" "%1"'
111+
WriteRegStr HKCR "@CLIENT_TARNAME@" "URL Protocol" ""
112+
WriteRegStr HKCR "@CLIENT_TARNAME@" "" "URL:Bitcoin"
113+
WriteRegStr HKCR "@CLIENT_TARNAME@\DefaultIcon" "" $INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@
114+
WriteRegStr HKCR "@CLIENT_TARNAME@\shell\open\command" "" '"$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" "%1"'
115115
SectionEnd
116116

117117
# Macro for selecting uninstaller sections
@@ -142,9 +142,9 @@ Section -un.post UNSEC0001
142142
DeleteRegKey HKCU "SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\$(^Name)"
143143
Delete /REBOOTOK "$SMPROGRAMS\$StartMenuGroup\Uninstall $(^Name).lnk"
144144
Delete /REBOOTOK "$SMPROGRAMS\$StartMenuGroup\$(^Name).lnk"
145-
Delete /REBOOTOK "$SMPROGRAMS\$StartMenuGroup\@PACKAGE_NAME@ (testnet).lnk"
146-
Delete /REBOOTOK "$SMPROGRAMS\$StartMenuGroup\@PACKAGE_NAME@ (testnet4).lnk"
147-
Delete /REBOOTOK "$SMPROGRAMS\$StartMenuGroup\@PACKAGE_NAME@ (test signet).lnk"
145+
Delete /REBOOTOK "$SMPROGRAMS\$StartMenuGroup\@CLIENT_NAME@ (testnet).lnk"
146+
Delete /REBOOTOK "$SMPROGRAMS\$StartMenuGroup\@CLIENT_NAME@ (testnet4).lnk"
147+
Delete /REBOOTOK "$SMPROGRAMS\$StartMenuGroup\@CLIENT_NAME@ (test signet).lnk"
148148
Delete /REBOOTOK "$SMSTARTUP\Bitcoin.lnk"
149149
Delete /REBOOTOK $INSTDIR\uninstall.exe
150150
Delete /REBOOTOK $INSTDIR\debug.log
@@ -153,7 +153,7 @@ Section -un.post UNSEC0001
153153
DeleteRegValue HKCU "${REGKEY}" Path
154154
DeleteRegKey /IfEmpty HKCU "${REGKEY}\Components"
155155
DeleteRegKey /IfEmpty HKCU "${REGKEY}"
156-
DeleteRegKey HKCR "@PACKAGE_TARNAME@"
156+
DeleteRegKey HKCR "@CLIENT_TARNAME@"
157157
RmDir /REBOOTOK $SMPROGRAMS\$StartMenuGroup
158158
RmDir /REBOOTOK $INSTDIR
159159
Push $R0

‎test/config.ini.in‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@
66
# test/*/test_runner.py and test/util/rpcauth-test.py
77

88
[environment]
9-
PACKAGE_NAME=@PACKAGE_NAME@
10-
PACKAGE_BUGREPORT=@PACKAGE_BUGREPORT@
9+
PACKAGE_NAME=@CLIENT_NAME@
10+
PACKAGE_BUGREPORT=@CLIENT_BUGREPORT@
1111
SRCDIR=@abs_top_srcdir@
1212
BUILDDIR=@abs_top_builddir@
1313
EXEEXT=@EXEEXT@

0 commit comments

Comments
 (0)