Add 4th component to version in loader.rc
authorCharles Giessen <charles@lunarg.com>
Wed, 29 Mar 2023 20:10:04 +0000 (14:10 -0600)
committerCharles Giessen <charles@lunarg.com>
Tue, 16 May 2023 23:03:04 +0000 (17:03 -0600)
FILVERSION is supposed to be a 4 component field. The previous commit to make loader.rc
be generated did not account for that and so requires this fix to make sure it has four
components. Thankfully, this issue is not present in SDK released loaders thanks to the
SDK specifying its own version string that has 4 components.

loader/loader.rc
scripts/generate_source.py

index 1d2280dd1431efd3f8451be1d97bf41ac8777faf..daa488fbb7996aaa1def6e4dd0fcc930c0fab2b8 100644 (file)
@@ -22,7 +22,7 @@
 #include "winres.h"
 
 // All set through CMake
-#define VER_FILE_VERSION 1, 3, 245
+#define VER_FILE_VERSION 1, 3, 245, 0
 #define VER_FILE_DESCRIPTION_STR "1.3.245.Dev Build"
 #define VER_FILE_VERSION_STR "Vulkan Loader - Dev Build"
 #define VER_COPYRIGHT_STR "Copyright (C) 2015-2023"
index 2f67253be074643ae51939efa48c1b154041265d..2411e84e821f5b487bc684cf8a84376d860532e7 100755 (executable)
@@ -120,7 +120,7 @@ def main(argv):
 
         with open(common_codegen.repo_relative('loader/loader.rc.in'), "r") as rc_file:
             rc_file_contents = rc_file.read()
-        rc_ver = ', '.join(args.generated_version.split('.'))
+        rc_ver = ', '.join(args.generated_version.split('.') + ['0'])
         rc_file_contents = rc_file_contents.replace('${LOADER_VER_FILE_VERSION}', f'{rc_ver}')
         rc_file_contents = rc_file_contents.replace('${LOADER_VER_FILE_DESCRIPTION_STR}', f'"{args.generated_version}.Dev Build"')
         rc_file_contents = rc_file_contents.replace('${LOADER_VER_FILE_VERSION_STR}', f'"Vulkan Loader - Dev Build"')