Skip to content

Commit aa2f5be

Browse files
committed
fix: executor properly prevent dupe options
1 parent b8e2c11 commit aa2f5be

File tree

1 file changed

+8
-16
lines changed

1 file changed

+8
-16
lines changed

packages/nx/src/executors/build/executor.ts

Lines changed: 8 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -107,8 +107,7 @@ export default async function runExecutor(options: BuildBuilderSchema, context:
107107
nsOptions.push(options.platform);
108108
}
109109
if (options.device && !options.emulator) {
110-
nsOptions.push('--device');
111-
nsOptions.push(options.device);
110+
nsOptions.push(`--device=${options.device}`);
112111
}
113112
if (options.emulator) {
114113
nsOptions.push('--emulator');
@@ -135,34 +134,27 @@ export default async function runExecutor(options: BuildBuilderSchema, context:
135134
nsOptions.push('--aab');
136135
}
137136
if (options.keyStorePath) {
138-
nsOptions.push('--key-store-path');
139-
nsOptions.push(options.keyStorePath);
137+
nsOptions.push(`--key-store-path=${options.keyStorePath}`);
140138
}
141139
if (options.keyStorePassword) {
142-
nsOptions.push('--key-store-password');
143-
nsOptions.push(options.keyStorePassword);
140+
nsOptions.push(`--key-store-password=${options.keyStorePassword}`);
144141
}
145142
if (options.keyStoreAlias) {
146-
nsOptions.push('--key-store-alias');
147-
nsOptions.push(options.keyStoreAlias);
143+
nsOptions.push(`--key-store-alias=${options.keyStoreAlias}`);
148144
}
149145
if (options.keyStoreAliasPassword) {
150-
nsOptions.push('--key-store-alias-password');
151-
nsOptions.push(options.keyStoreAliasPassword);
146+
nsOptions.push(`--key-store-alias-password=${options.keyStoreAliasPassword}`);
152147
}
153148
if (options.provision) {
154-
nsOptions.push('--provision');
155-
nsOptions.push(options.provision);
149+
nsOptions.push(`--provision=${options.provision}`);
156150
}
157151
if (options.copyTo) {
158-
nsOptions.push('--copy-to');
159-
nsOptions.push(options.copyTo);
152+
nsOptions.push(`--copy-to=${options.copyTo}`);
160153
}
161154

162155
if (nsCliFileReplacements.length) {
163156
// console.log('nsCliFileReplacements:', nsCliFileReplacements);
164-
nsOptions.push('--env.replace');
165-
nsOptions.push(nsCliFileReplacements.join(','));
157+
nsOptions.push(`--env.replace=${nsCliFileReplacements.join(',')}`);
166158
}
167159
// always add --force (unless explicity set to false) for now since within Nx we use @nativescript/webpack at root only and the {N} cli shows a blocking error if not within the app
168160
if (options?.force !== false) {

0 commit comments

Comments
 (0)