Skip to content

Commit 3526010

Browse files
committed
fix: use double quote for LDFLAGS/CPPFLAGS
1 parent e9ad632 commit 3526010

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

dist/setup_cpp.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/setup_cpp.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/llvm/llvm.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -310,8 +310,8 @@ export async function activateLLVM(directory: string, versionGiven: string) {
310310
addEnv("DYLD_LIBRARY_PATH", `${lib}${path.delimiter}${dyld}`),
311311

312312
// compiler flags
313-
addEnv("LDFLAGS", `-L'${directory}/lib'`),
314-
addEnv("CPPFLAGS", `-I'${directory}/include'`),
313+
addEnv("LDFLAGS", `-L"${directory}/lib"`),
314+
addEnv("CPPFLAGS", `-I"${directory}/include"`),
315315

316316
// compiler paths
317317
addEnv("CC", addBinExtension(`${directory}/bin/clang`)),

src/utils/env/addEnv.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,8 @@ async function addEnvSystem(name: string, valGiven: string | undefined) {
5454
switch (process.platform) {
5555
case "win32": {
5656
// We do not use `execa.sync(`setx PATH "${path};%PATH%"`)` because of its character limit
57-
await execPowershell(`[Environment]::SetEnvironmentVariable("${name}", "${val}", "User")`)
58-
info(`${name}="${val}" was set in the environment.`)
57+
await execPowershell(`[Environment]::SetEnvironmentVariable('${name}', '${val}', "User")`)
58+
info(`${name}='${val}' was set in the environment.`)
5959
return
6060
}
6161
case "linux":

0 commit comments

Comments
 (0)