Skip to content

Commit 8423367

Browse files
committed
Use perl instead of sed for genrule objcopy substitution
Signed-off-by: Matt Leon <mattleon@google.com>
1 parent 4a176a3 commit 8423367

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

bazel/external/wasmtime.BUILD

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ genrule(
4141
],
4242
cmd = """
4343
for symbol in $$(nm -P $(<) 2>/dev/null | grep -E ^_?wasm_ | cut -d" " -f1); do
44-
echo $$symbol | sed -r 's/^(_?)(wasm_[a-z_]+)$$/\\1\\2 \\1wasmtime_\\2/' >>prefixed
44+
echo $$symbol | perl -p -e 's!^(_?)(wasm_[a-z_0-9.:-]+)$$!\\1\\2 \\1wasmtime_\\2!' >>prefixed
4545
done
4646
# This should be OBJCOPY, but bazel-zig-cc doesn't define it.
4747
$(location @llvm_toolchain_llvm//:objcopy) --redefine-syms=prefixed $(<) $@
@@ -69,7 +69,7 @@ genrule(
6969
for enabled_feature in $$(echo "{}"); do
7070
perl -pi -e "s/#cmakedefine WASMTIME_FEATURE_$$enabled_feature/#define WASMTIME_FEATURE_$$enabled_feature 1/" $$TMPDIR/working_file
7171
done
72-
perl -pi -e 's/#cmakedefine \\(.*\\)/\\/\\/ \\1 is not defined./' $$TMPDIR/working_file
72+
perl -pi -e 's?#cmakedefine (.*)?// \\1 is not defined.?' $$TMPDIR/working_file
7373
cp $$TMPDIR/working_file $@
7474
""".format(" ".join([f.upper() for f in features])),
7575
)

0 commit comments

Comments
 (0)