Updated variable check to be case insensitive

Author: michalChrobot

.yamato/project-builders/project-builders.yml

@@ -62,21 +62,21 @@ build_{{ netcodeProject[0] }}_project:
     # Build the project using Unity Editor. This will call the given static BuilderScripts method.
     # Ideally, it would be nice to parametrize the BuilderScripts (for example to pass scripting backend as parameter) but -executeMethod only calls static methods without parameters so for now we will have multiple configurations
     # Notice that for Android platform even if mono is selected, il2cpp will be used since mono is not supported for Android builds.
-    - IF "%PLATFORM_WIN64_MAC_ANDROID%"=="win64" (
-        IF "%SCRIPTING_BACKEND_IL2CPP_MONO%"=="il2cpp" (
+    - IF /I "%PLATFORM_WIN64_MAC_ANDROID%"=="win64" (
+        IF /I "%SCRIPTING_BACKEND_IL2CPP_MONO%"=="il2cpp" (
           C:/TestingEditor/Unity.exe -projectPath C:/ClonedProject/{{ netcodeProject[1].projectPath }} -buildTarget win64 -executeMethod BuilderScripts.BuildWinIl2cpp -batchmode -logFile ./artifacts/UnityLog.txt -automated -crash-report-folder ./artifacts/CrashArtifacts -quit
         ) ELSE (
           C:/TestingEditor/Unity.exe -projectPath C:/ClonedProject/{{ netcodeProject[1].projectPath }} -buildTarget win64 -executeMethod BuilderScripts.BuildWinMono -batchmode -logFile ./artifacts/UnityLog.txt -automated -crash-report-folder ./artifacts/CrashArtifacts -quit
         )
       )
-      ELSE IF "%PLATFORM_WIN64_MAC_ANDROID%"=="mac" (
-          IF "%SCRIPTING_BACKEND_IL2CPP_MONO%"=="il2cpp" (
+      ELSE IF /I "%PLATFORM_WIN64_MAC_ANDROID%"=="mac" (
+          IF /I "%SCRIPTING_BACKEND_IL2CPP_MONO%"=="il2cpp" (
             C:/TestingEditor/Unity.exe -projectPath C:/ClonedProject/{{ netcodeProject[1].projectPath }} -buildTarget osx -executeMethod BuilderScripts.BuildMacIl2cpp -batchmode -logFile ./artifacts/UnityLog.txt -automated -crash-report-folder ./artifacts/CrashArtifacts -quit
           ) ELSE (
             C:/TestingEditor/Unity.exe -projectPath C:/ClonedProject/{{ netcodeProject[1].projectPath }} -buildTarget osx -executeMethod BuilderScripts.BuildMacMono -batchmode -logFile ./artifacts/UnityLog.txt -automated -crash-report-folder ./artifacts/CrashArtifacts -quit
           )
       )
-      ELSE IF "%PLATFORM_WIN64_MAC_ANDROID%"=="android" (
+      ELSE IF /I "%PLATFORM_WIN64_MAC_ANDROID%"=="android" (
           C:/TestingEditor/Unity.exe -projectPath C:/ClonedProject/{{ netcodeProject[1].projectPath }} -buildTarget android -executeMethod BuilderScripts.BuildAndroidIl2cpp -batchmode -logFile ./artifacts/UnityLog.txt -automated -crash-report-folder ./artifacts/CrashArtifacts -quit
       )
 

Tools/scripts/BuildAutomation/disable-enable-burst.py

@@ -52,7 +52,7 @@ def create_config(settings_path):
 # Note that this jobs uses environment variables to pass parameters to the script.
 def resolve_target():
     # Get the platform value from the environment variable
-    platform_key = os.environ.get('PLATFORM_WIN64_MAC_ANDROID')
+    platform_key = os.environ.get('PLATFORM_WIN64_MAC_ANDROID').lower()
 
     resolved_target = platform_key
     if 'win64' == platform_key:
@@ -95,7 +95,7 @@ def main():
     parse_args()
     config_names = get_or_create_burst_AOT_config()
 
-    platform_key = os.environ.get('PLATFORM_WIN64_MAC_ANDROID')
+    platform_key = os.environ.get('PLATFORM_WIN64_MAC_ANDROID').lower()
     print(f"Burst compilation script: Unity project path is {args.project_path}")
     print(f"Burst compilation script: Target platform is {platform_key}")
 

Tools/scripts/BuildAutomation/validate_params.py

@@ -1,4 +1,4 @@
-import os
+import os
 import sys
 
 # --- Configuration ---
@@ -17,23 +17,20 @@ def main():
     """
     all_params_valid = True
 
-    # Iterate through the dictionary of rules.
     for var_name, allowed_values in VALIDATION_RULES.items():
-        # Get the variable's value from the environment.
-        actual_value = os.environ.get(var_name, '')
+        actual_value = os.environ.get(var_name, '').lower()
+        allowed_values_lower = {v.lower() for v in allowed_values}
 
-        # Check if the actual value is in the set of allowed values.
-        if actual_value not in allowed_values:
+        if actual_value not in allowed_values_lower:
             print(
                 f"ERROR: Invalid {var_name}: '{actual_value}'. "
                 f"Allowed values are: {list(allowed_values)}",
                 file=sys.stderr
             )
             all_params_valid = False
 
-    # --- Validation for Invalid Combinations ---
-    platform = os.environ.get('PLATFORM_WIN64_MAC_ANDROID')
-    scripting_backend = os.environ.get('SCRIPTING_BACKEND_IL2CPP_MONO')
+    platform = os.environ.get('PLATFORM_WIN64_MAC_ANDROID', '').lower()
+    scripting_backend = os.environ.get('SCRIPTING_BACKEND_IL2CPP_MONO', '').lower()
 
     if platform == 'mac' and scripting_backend == 'il2cpp':
         print(
@@ -51,13 +48,11 @@ def main():
         )
         all_params_valid = False
 
-    # --- Final Result ---
     if not all_params_valid:
         print("\nOne or more parameters failed validation. Halting build.", file=sys.stderr)
-        # Exit with a non-zero code to fail the Yamato job.
         sys.exit(1)
 
     print("All parameters are valid. Proceeding with the build.")
 
 if __name__ == "__main__":
-    main()
+    main()