Merge pull request #672 from tidev/fix-platform-again

CHANGELOG.md

@@ -1,3 +1,9 @@
+
+7.1.2 (7/31/2024)
+-------------------
+ * fix: Force `--platform` to `ios`; the SDK converts `ios` to `iphone`, but
+   the CLI needs to reverse this to be able to look up the platform config
+
 7.1.1 (7/30/2024)
 -------------------
  * fix: Rename platform config `iphone` to `ios` to align with

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "titanium",
-	"version": "7.1.1",
+	"version": "7.1.2",
 	"author": "TiDev, Inc. <[email protected]>",
 	"description": "Command line interface for building Titanium SDK apps",
 	"type": "module",

src/cli.js

@@ -639,6 +639,13 @@ export class CLI {
 			return;
 		}
 
+		// when specifying `--platform ios`, the SDK's option callback converts
+		// it to `iphone`, however the platform config uses `ios` and we must
+		// convert it back
+		if (this.argv.platform === 'iphone') {
+			this.argv.platform = 'ios';
+		}
+
 		this.debugLogger.trace(`Processing --platform option: ${this.argv.platform || 'not specified'}`);
 		try {
 			if (!this.argv.platform) {
@@ -681,6 +688,7 @@ export class CLI {
 		const platformConf = this.command.conf.platforms[this.argv.platform];
 
 		this.argv.$platform = this.argv.platform;
+		// set the platform in Commander so we don't lose it when we re-parse the args
 		this.command.setOptionValue('platform', this.argv.platform);
 
 		// set platform context