Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit ad7d44b

Browse files
authored
Merge pull request #388 from aminya/llvm
fix: do not add LLVM libraries to dyld by default
2 parents f37a612 + 7e66da1 commit ad7d44b

File tree

5 files changed

+7
-7
lines changed

5 files changed

+7
-7
lines changed

dist/legacy/lib.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/legacy/lib.js.map

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/modern/lib.mjs

+1-1
Large diffs are not rendered by default.

dist/modern/lib.mjs.map

+1-1
Large diffs are not rendered by default.

src/llvm/llvm.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -88,8 +88,8 @@ export async function activateLLVM(directory: string, version: string) {
8888
addEnv("LLVM_PATH", directory, rcOptions),
8989

9090
// Setup LLVM as the compiler
91-
addEnv("LD_LIBRARY_PATH", `${ld}${delimiter}${directory}/lib`, rcOptions),
92-
addEnv("DYLD_LIBRARY_PATH", `${dyld}${delimiter}${directory}/lib`, rcOptions),
91+
addEnv("LLVM_LD_LIBRARY_PATH", `${ld}${delimiter}${directory}/lib`, rcOptions),
92+
addEnv("LLVM_DYLD_LIBRARY_PATH", `${dyld}${delimiter}${directory}/lib`, rcOptions),
9393

9494
// compiler flags
9595
addEnv("LLVM_LDFLAGS", `-L${quoteIfHasSpace(`${directory}/lib`)}`, rcOptions),
@@ -102,7 +102,7 @@ export async function activateLLVM(directory: string, version: string) {
102102
? addEnv("LLVM_CPATH", `${directory}/lib/clang/${llvmMajor}/include`, rcOptions)
103103
: Promise.resolve(),
104104

105-
addEnv("LIBRARY_PATH", `${directory}/lib`, rcOptions),
105+
addEnv("LLVM_LIBRARY_PATH", `${directory}/lib`, rcOptions),
106106

107107
// os sdks
108108
setupMacOSSDK(),

0 commit comments

Comments
 (0)