Skip to content

Commit 49cf0e7

Browse files
committedMay 31, 2019
Rename sysroot to wasi-sysroot.
This renames /opt/wasi/share/sysroot to /opt/wasi/share/wasi-sysroot. If the user selects an alternate prefix, this allows wasi-sysroot to coexist with other packages with less risk of namespace collision. For example, if the user uses a prefix of /usr/local, this avoids using /usr/local/share/sysroot, which is fairly generic, and uses /usr/local/share/wasi-sysroot, which more clearly indicates its purpose.
1 parent 0c16443 commit 49cf0e7

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed
 

‎Makefile

+7-7
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ build/llvm.BUILT:
2525
-DLLVM_EXTERNAL_CLANG_SOURCE_DIR=$(LLVM_PROJ_DIR)/clang \
2626
-DLLVM_EXTERNAL_LLD_SOURCE_DIR=$(LLVM_PROJ_DIR)/lld \
2727
-DLLVM_ENABLE_PROJECTS="lld;clang" \
28-
-DDEFAULT_SYSROOT=$(PREFIX)/share/sysroot \
28+
-DDEFAULT_SYSROOT=$(PREFIX)/share/wasi-sysroot \
2929
$(LLVM_PROJ_DIR)/llvm
3030
cd build/llvm; $(MAKE) -j 8 \
3131
install-clang \
@@ -43,7 +43,7 @@ build/llvm.BUILT:
4343
build/wasi-libc.BUILT: build/llvm.BUILT
4444
make -C $(ROOT_DIR)/src/wasi-libc \
4545
WASM_CC=$(PREFIX)/bin/clang \
46-
SYSROOT=$(PREFIX)/share/sysroot
46+
SYSROOT=$(PREFIX)/share/wasi-sysroot
4747
touch build/wasi-libc.BUILT
4848

4949
build/compiler-rt.BUILT: build/llvm.BUILT
@@ -94,7 +94,7 @@ build/libcxx.BUILT: build/llvm.BUILT build/compiler-rt.BUILT build/wasi-libc.BUI
9494
$(LLVM_PROJ_DIR)/libcxx
9595
cd build/libcxx; make -j 8 install
9696
# libc++abi.a doesn't do a multiarch install, so fix it up.
97-
mv $(PREFIX)/share/sysroot/lib/libc++.a $(PREFIX)/share/sysroot/lib/wasm32-wasi/
97+
mv $(PREFIX)/share/wasi-sysroot/lib/libc++.a $(PREFIX)/share/wasi-sysroot/lib/wasm32-wasi/
9898
touch build/libcxx.BUILT
9999

100100
build/libcxxabi.BUILT: build/libcxx.BUILT build/llvm.BUILT
@@ -115,18 +115,18 @@ build/libcxxabi.BUILT: build/libcxx.BUILT build/llvm.BUILT
115115
-DLLVM_COMPILER_CHECKED=ON \
116116
-DCMAKE_BUILD_TYPE=RelWithDebugInfo \
117117
-DLIBCXXABI_LIBCXX_PATH=$(LLVM_PROJ_DIR)/libcxx \
118-
-DLIBCXXABI_LIBCXX_INCLUDES=$(PREFIX)/share/sysroot/include/c++/v1 \
118+
-DLIBCXXABI_LIBCXX_INCLUDES=$(PREFIX)/share/wasi-sysroot/include/c++/v1 \
119119
-DLLVM_CONFIG_PATH=$(ROOT_DIR)/build/llvm/bin/llvm-config \
120120
-DCMAKE_TOOLCHAIN_FILE=$(ROOT_DIR)/wasi-sdk.cmake \
121121
-DWASI_SDK_PREFIX=$(PREFIX) \
122-
-DCMAKE_C_FLAGS="$(DEBUG_PREFIX_MAP) -I$(PREFIX)/share/sysroot/include" \
123-
-DCMAKE_CXX_FLAGS="$(DEBUG_PREFIX_MAP) -I$(PREFIX)/share/sysroot/include/c++/v1" \
122+
-DCMAKE_C_FLAGS="$(DEBUG_PREFIX_MAP) -I$(PREFIX)/share/wasi-sysroot/include" \
123+
-DCMAKE_CXX_FLAGS="$(DEBUG_PREFIX_MAP) -I$(PREFIX)/share/wasi-sysroot/include/c++/v1" \
124124
-DUNIX:BOOL=ON \
125125
--debug-trycompile \
126126
$(LLVM_PROJ_DIR)/libcxxabi
127127
cd build/libcxxabi; make -j 8 install
128128
# libc++abi.a doesn't do a multiarch install, so fix it up.
129-
mv $(PREFIX)/share/sysroot/lib/libc++abi.a $(PREFIX)/share/sysroot/lib/wasm32-wasi/
129+
mv $(PREFIX)/share/wasi-sysroot/lib/libc++abi.a $(PREFIX)/share/wasi-sysroot/lib/wasm32-wasi/
130130
touch build/libcxxabi.BUILT
131131

132132
build/config.BUILT:

‎wasi-sdk.cmake

+2-2
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ set(CMAKE_C_FLAGS "-v" CACHE STRING "wasi-sdk build")
1818
set(CMAKE_CXX_FLAGS "-v -std=c++11" CACHE STRING "wasi-sdk build")
1919
set(CMAKE_EXE_LINKER_FLAGS "-Wl,--no-threads" CACHE STRING "wasi-sdk build")
2020

21-
set(CMAKE_SYSROOT ${WASI_SDK_PREFIX}/share/sysroot CACHE STRING "wasi-sdk build")
22-
set(CMAKE_STAGING_PREFIX ${WASI_SDK_PREFIX}/share/sysroot CACHE STRING "wasi-sdk build")
21+
set(CMAKE_SYSROOT ${WASI_SDK_PREFIX}/share/wasi-sysroot CACHE STRING "wasi-sdk build")
22+
set(CMAKE_STAGING_PREFIX ${WASI_SDK_PREFIX}/share/wasi-sysroot CACHE STRING "wasi-sdk build")
2323

2424
# Don't look in the sysroot for executables to run during the build
2525
set(CMAKE_FIND_ROOT_PATH_MODE_PROGRAM NEVER)

0 commit comments

Comments
 (0)