diff options
author | Andrew Clayton <a.clayton@nginx.com> | 2024-02-22 02:24:36 +0000 |
---|---|---|
committer | Andrew Clayton <a.clayton@nginx.com> | 2024-02-22 03:17:08 +0000 |
commit | d54af163c46bffd1bea0c2b544e412b5d628ec80 (patch) | |
tree | 9c3ec22452e9c81943ac7f860cb907fb86d85cd9 /auto/pcre | |
parent | 7b13c30604952059c337b8c6afa588fdb5b8895a (diff) | |
download | unit-d54af163c46bffd1bea0c2b544e412b5d628ec80.tar.gz unit-d54af163c46bffd1bea0c2b544e412b5d628ec80.tar.bz2 |
Wasm-wc: Use the cargo build output as the make target dependency
cargo build creates the language module under
src/wasm-wasi-component/target/release/libwasm_wasi_component.so and not
build/lib/unit/modules/wasm_wasi_component.unit.so which is what we were
using as a target dependency in the Makefile which doesn't exist so this
resulted in the following
$ make wasm-wasi-component-install
cargo build --release --manifest-path src/wasm-wasi-component/Cargo.toml
Finished release [optimized] target(s) in 0.17s
install -d /opt/unit/modules
install -p src/wasm-wasi-component/target/release/libwasm_wasi_component.so \
/opt/unit/modules/wasm_wasi_component.unit.so
I.e it wanted to rebuild the module, after this patch we get the more
correct
$ make wasm-wasi-component-install
install -d /opt/unit/modules
install -p src/wasm-wasi-component/target/release/libwasm_wasi_component.so \
/opt/unit/modules/wasm_wasi_component.unit.so
This is all a little ugly because we're fighting against cargo wanting
to do its own thing and this wasm-wasi-component language module build
process is likely going to get some re-working anyway, so this will do
for now.
Reported-by: Konstantin Pavlov <thresh@nginx.com>
Signed-off-by: Andrew Clayton <a.clayton@nginx.com>
Diffstat (limited to 'auto/pcre')
0 files changed, 0 insertions, 0 deletions