Skip to content

Commit 21b4be7

Browse files
committed
Fix review comment
1 parent ff19a00 commit 21b4be7

File tree

1 file changed

+3
-9
lines changed

1 file changed

+3
-9
lines changed

src/toolchain/BuildFlags.ts

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -202,14 +202,15 @@ export class BuildFlags {
202202
if (!configuration.disableSandbox) {
203203
return args;
204204
}
205+
const disableSandboxFlags = ["--disable-sandbox", "-Xswiftc", "-disable-sandbox"];
205206
switch (args[0]) {
206207
case "package": {
207-
return [args[0], ...BuildFlags.disableSandboxFlags(), ...args.slice(1)];
208+
return [args[0], ...disableSandboxFlags, ...args.slice(1)];
208209
}
209210
case "build":
210211
case "run":
211212
case "test": {
212-
return [...args, ...BuildFlags.disableSandboxFlags()];
213+
return [...args, ...disableSandboxFlags];
213214
}
214215
default:
215216
// Do nothing for other commands
@@ -223,13 +224,6 @@ export class BuildFlags {
223224
);
224225
}
225226

226-
/**
227-
* Get flags for disabling sandboxing when running SwiftPM
228-
*/
229-
static disableSandboxFlags(): string[] {
230-
return ["--disable-sandbox", "-Xswiftc", "-disable-sandbox"];
231-
}
232-
233227
/**
234228
* Filter argument list
235229
* @param args argument list

0 commit comments

Comments
 (0)