Merge branch 'main' of github.com:Unity-Technologies/UnityRenderStreaming into feat/stability-project-develop-upgrade-render-streaming

.yamato/promotion.yml

@@ -3,7 +3,7 @@
 test_platforms:
   - name: win
     type: Unity::VM
-    image: package-ci/win10:v1.21.0-1103459
+    image: package-ci/win10:v4
     flavor: b1.large
 ---
 
@@ -33,7 +33,7 @@ promote_dry_run:
   name: Promote Dry Run to Production
   agent:
     type: Unity::VM
-    image: package-ci/win10:v1.21.0-1103459
+    image: package-ci/win10:v4
     flavor: b1.large
   variables:
     UPMCI_PROMOTION: 1
@@ -62,7 +62,7 @@ promote:
   name: Promote to Production
   agent:
     type: Unity::VM
-    image: package-ci/win10:v1.21.0-1103459
+    image: package-ci/win10:v4
     flavor: b1.large
   variables:
     UPMCI_PROMOTION: 1

.yamato/upm-ci-renderstreaming-packages.yml

@@ -362,7 +362,7 @@ publish_dry_run_{{ packagename }}:
   name: Publish Dry Run {{ package_displayname }}
   agent:
     type: Unity::VM
-    image: package-ci/win10:v1.21.0-1103459
+    image: package-ci/win10:v4
     flavor: b1.large
   commands:
     - npm install upm-ci-utils@{{ upm.package_version }} -g --registry {{ upm.registry_url }}
@@ -389,7 +389,7 @@ publish_{{ packagename }}:
   name: Publish {{ package_displayname }}
   agent:
     type: Unity::VM
-    image: package-ci/win10:v1.21.0-1103459
+    image: package-ci/win10:v4
     flavor: b1.large
   commands:
     - npm install upm-ci-utils@{{ upm.package_version }} -g --registry {{ upm.registry_url }}

.yamato/upm-ci-template.yml

@@ -11,7 +11,7 @@ prepack_{{ project.name }}_{{ editor.version }}:
   name: Pre-Pack {{ project.packagename }} {{ editor.version }} - Primed Artifacts
   agent:
     type: Unity::VM
-    image: package-ci/win10:v1.21.0-1103459
+    image: package-ci/win10:v4
     flavor: b1.large
   commands:
     - pip install unity-downloader-cli --index-url {{ intra_pypi_url }} --upgrade
@@ -114,7 +114,7 @@ publish_{{ project.name }}:
   name: Publish {{ project.packagename }}
   agent:
     type: Unity::VM
-    image: package-ci/win10:v1.21.0-1103459
+    image: package-ci/win10:v4
     flavor: b1.large
   commands:
     - npm install upm-ci-utils@{{ upm.package_version }} -g --registry {{ upm.registry_url }}
@@ -143,7 +143,7 @@ publish_dryrun_{{ project.name }}:
   name: Publish Dry Run {{ project.packagename }}
   agent:
     type: Unity::VM
-    image: package-ci/win10:v1.21.0-1103459
+    image: package-ci/win10:v4
     flavor: b1.large
   commands:
     - npm install upm-ci-utils@{{ upm.package_version }} -g --registry {{ upm.registry_url }}

.yamato/upm-ci-webapp.yml

@@ -1,7 +1,7 @@
 platforms:
   - name: win
     type: Unity::VM
-    image: package-ci/win10:v1.21.0-1103459
+    image: package-ci/win10:v4
     flavor: b1.xlarge
     pack_command: pack_webapp.cmd
     test_command: test_webapp.cmd

com.unity.renderstreaming/Runtime/Scripts/CommandLineParser.cs

@@ -46,18 +46,18 @@ namespace Unity.RenderStreaming
             public string ArgumentName { get; }
 
             /// <summary>
-            /// 
+            ///
             /// </summary>
             public bool Defined => m_defined;
 
             /// <summary>
-            /// 
+            ///
             /// </summary>
             public readonly bool Required;
 
 
             /// <summary>
-            /// 
+            ///
             /// </summary>
             public T Value => m_value;
 
@@ -212,10 +212,10 @@ namespace Unity.RenderStreaming
                 list.Add(arguments[startIndex + 1]);
                 i = startIndex + 2;
             }
-            if (list.Count == 0 && required)
+            if (list.Count == 0)
             {
                 argumentValue = null;
-                return false;
+                return !required;
             }
             argumentValue = list.ToArray();
             return true;

com.unity.renderstreaming/Tests/Runtime/CommandLineParserTest.cs

@@ -19,6 +19,7 @@ namespace Unity.RenderStreaming.RuntimeTest
             Assert.That(CommandLineParser.TryParse(arguments), Is.True);
             Assert.That((string)CommandLineParser.SignalingUrl, Is.Null);
             Assert.That((int?)CommandLineParser.PollingInterval, Is.Null);
+            Assert.That((string[])CommandLineParser.IceServerUrls, Is.Null);
         }
 
         [Test]