Skip to content

Commit

Permalink
pythonGH-113516: don't set LDSHARED when building for WASI
Browse files Browse the repository at this point in the history
  • Loading branch information
brettcannon committed Feb 14, 2024
1 parent 4684301 commit 98ad9c1
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Don't set `LDSHARED` when building for WASI.
1 change: 0 additions & 1 deletion Tools/wasm/wasi-env
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,6 @@ if command -v ccache >/dev/null 2>&1; then
CXX="ccache ${CXX}"
fi

LDSHARED="${WASI_SDK_PATH}/bin/wasm-ld"
AR="${WASI_SDK_PATH}/bin/llvm-ar"
RANLIB="${WASI_SDK_PATH}/bin/ranlib"

Expand Down
2 changes: 1 addition & 1 deletion Tools/wasm/wasi.py
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,7 @@ def wasi_sdk_env(context):
wasi_sdk_path = context.wasi_sdk_path
sysroot = wasi_sdk_path / "share" / "wasi-sysroot"
env = {"CC": "clang", "CPP": "clang-cpp", "CXX": "clang++",
"LDSHARED": "wasm-ld", "AR": "llvm-ar", "RANLIB": "ranlib"}
"AR": "llvm-ar", "RANLIB": "ranlib"}

for env_var, binary_name in list(env.items()):
env[env_var] = os.fsdecode(wasi_sdk_path / "bin" / binary_name)
Expand Down

0 comments on commit 98ad9c1

Please sign in to comment.