aboutsummaryrefslogtreecommitdiff
path: root/NorthstarDedicatedTest
diff options
context:
space:
mode:
authorGeckoEidechse <40122905+GeckoEidechse@users.noreply.github.com>2022-04-27 22:56:13 +0200
committerGitHub <noreply@github.com>2022-04-27 22:56:13 +0200
commit6c8112a6c368dd36d21fd94689e9682bc3b012a5 (patch)
tree1bcec2a9e0a1a8fd3a87f3212283f22164d58248 /NorthstarDedicatedTest
parent3a1985d046d16861d833a4049b03b03413b2d650 (diff)
downloadNorthstarLauncher-6c8112a6c368dd36d21fd94689e9682bc3b012a5.tar.gz
NorthstarLauncher-6c8112a6c368dd36d21fd94689e9682bc3b012a5.zip
Hardcode version number (#152)
* Hardcode version number This way, we don't get the wrong version number when launching Titanfall2.exe via `-northstar` The hardcoded value should be overwritten by CI on release Fixes #129 * Set Northstar version in separate header file * Make `northstar_version` array a constant * Fix formatting
Diffstat (limited to 'NorthstarDedicatedTest')
-rw-r--r--NorthstarDedicatedTest/NorthstarDedicatedTest.vcxproj1
-rw-r--r--NorthstarDedicatedTest/NorthstarDedicatedTest.vcxproj.filters3
-rw-r--r--NorthstarDedicatedTest/ns_version.h7
-rw-r--r--NorthstarDedicatedTest/version.cpp67
4 files changed, 25 insertions, 53 deletions
diff --git a/NorthstarDedicatedTest/NorthstarDedicatedTest.vcxproj b/NorthstarDedicatedTest/NorthstarDedicatedTest.vcxproj
index 13f907e1..ea9ccbb3 100644
--- a/NorthstarDedicatedTest/NorthstarDedicatedTest.vcxproj
+++ b/NorthstarDedicatedTest/NorthstarDedicatedTest.vcxproj
@@ -118,6 +118,7 @@
<ClInclude Include="clientruihooks.h" />
<ClInclude Include="clientvideooverrides.h" />
<ClInclude Include="localchatwriter.h" />
+ <ClInclude Include="ns_version.h" />
<ClInclude Include="plugins.h" />
<ClInclude Include="plugin_abi.h" />
<ClInclude Include="serverchathooks.h" />
diff --git a/NorthstarDedicatedTest/NorthstarDedicatedTest.vcxproj.filters b/NorthstarDedicatedTest/NorthstarDedicatedTest.vcxproj.filters
index e9c52d1d..da8f064e 100644
--- a/NorthstarDedicatedTest/NorthstarDedicatedTest.vcxproj.filters
+++ b/NorthstarDedicatedTest/NorthstarDedicatedTest.vcxproj.filters
@@ -1509,6 +1509,9 @@
<ClInclude Include="clientruihooks.h">
<Filter>Header Files\Client</Filter>
</ClInclude>
+ <ClInclude Include="ns_version.h">
+ <Filter>Header Files</Filter>
+ </ClInclude>
</ItemGroup>
<ItemGroup>
<ClCompile Include="dllmain.cpp">
diff --git a/NorthstarDedicatedTest/ns_version.h b/NorthstarDedicatedTest/ns_version.h
new file mode 100644
index 00000000..d30594fb
--- /dev/null
+++ b/NorthstarDedicatedTest/ns_version.h
@@ -0,0 +1,7 @@
+#pragma once
+#ifndef NORTHSTAR_VERSION
+// Turning off clang-format here so it doesn't mess with style as it needs to be this way for regex-ing with CI
+// clang-format off
+#define NORTHSTAR_VERSION 0,0,0,1
+// clang-format on
+#endif
diff --git a/NorthstarDedicatedTest/version.cpp b/NorthstarDedicatedTest/version.cpp
index a0bcee20..3755e4c4 100644
--- a/NorthstarDedicatedTest/version.cpp
+++ b/NorthstarDedicatedTest/version.cpp
@@ -1,65 +1,26 @@
#include "version.h"
#include "pch.h"
+#include "ns_version.h"
char version[16];
char NSUserAgent[32];
void InitialiseVersion()
{
- HRSRC hResInfo;
- DWORD dwSize;
- HGLOBAL hResData;
- LPVOID pRes, pResCopy;
- UINT uLen = 0;
- VS_FIXEDFILEINFO* lpFfi = NULL;
- HINSTANCE hInst = ::GetModuleHandle(NULL);
+ constexpr int northstar_version[4] {NORTHSTAR_VERSION};
- hResInfo = FindResourceW(hInst, MAKEINTRESOURCE(1), RT_VERSION);
- if (hResInfo != NULL)
+ // We actually use the rightmost integer do determine whether or not we're a debug/dev build
+ // If it is set to 1, we are a dev build
+ // On github CI, we set this 1 to a 0 automatically as we replace the 0,0,0,1 with the real version number
+ if (northstar_version[3] == 1)
{
- dwSize = SizeofResource(hInst, hResInfo);
- hResData = LoadResource(hInst, hResInfo);
- if (hResData != NULL)
- {
- pRes = LockResource(hResData);
- pResCopy = LocalAlloc(LMEM_FIXED, dwSize);
- if (pResCopy != 0)
- {
- CopyMemory(pResCopy, pRes, dwSize);
- VerQueryValueW(pResCopy, L"\\", (LPVOID*)&lpFfi, &uLen);
-
- DWORD dwFileVersionMS = lpFfi->dwFileVersionMS;
- DWORD dwFileVersionLS = lpFfi->dwFileVersionLS;
-
- DWORD dwLeftMost = HIWORD(dwFileVersionMS);
- DWORD dwSecondLeft = LOWORD(dwFileVersionMS);
- DWORD dwSecondRight = HIWORD(dwFileVersionLS);
- DWORD dwRightMost = LOWORD(dwFileVersionLS);
-
- // We actually use the rightmost integer do determine whether or not we're a debug/dev build
- // If it is set to 1 (as in resources.rc), we are a dev build
- // On github CI, we set this 1 to a 0 automatically as we replace the 0.0.0.1 with the real version number
- if (dwRightMost == 1)
- {
- sprintf(version, "%d.%d.%d.%d+dev", dwLeftMost, dwSecondLeft, dwSecondRight, dwRightMost);
- sprintf(NSUserAgent, "R2Northstar/%d.%d.%d+dev", dwLeftMost, dwSecondLeft, dwSecondRight);
- }
- else
- {
- sprintf(version, "%d.%d.%d.%d", dwLeftMost, dwSecondLeft, dwSecondRight, dwRightMost);
- sprintf(NSUserAgent, "R2Northstar/%d.%d.%d", dwLeftMost, dwSecondLeft, dwSecondRight);
- }
- UnlockResource(hResData);
- FreeResource(hResData);
- LocalFree(pResCopy);
- return;
- }
- UnlockResource(hResData);
- FreeResource(hResData);
- LocalFree(pResCopy);
- }
+ sprintf(version, "%d.%d.%d.%d+dev", northstar_version[0], northstar_version[1], northstar_version[2], northstar_version[3]);
+ sprintf(NSUserAgent, "R2Northstar/%d.%d.%d+dev", northstar_version[0], northstar_version[1], northstar_version[2]);
+ }
+ else
+ {
+ sprintf(version, "%d.%d.%d.%d", northstar_version[0], northstar_version[1], northstar_version[2], northstar_version[3]);
+ sprintf(NSUserAgent, "R2Northstar/%d.%d.%d", northstar_version[0], northstar_version[1], northstar_version[2]);
}
- // Could not locate version info for whatever reason
- spdlog::error("Failed to load version info:\n{}", std::system_category().message(GetLastError()));
- sprintf(NSUserAgent, "R2Northstar/0.0.0");
+ return;
}