Opened 5 months ago

Closed 5 months ago

#68969 closed defect (fixed)

binaryen 111.0 fails to compile due to -Werror warnings

Reported by: kpreid (Kevin Reid) Owned by: MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: binaryen

Description

:info:build In file included from /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_macports_release_tarballs_ports_lang_binaryen/binaryen/work/binaryen-111/src/wasm/wasm-debug.cpp:23:
:info:build In file included from /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_macports_release_tarballs_ports_lang_binaryen/binaryen/work/binaryen-111/third_party/llvm-project/include/llvm/include/llvm/DebugInfo/DWARFContext.h:12:
:info:build In file included from /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_macports_release_tarballs_ports_lang_binaryen/binaryen/work/binaryen-111/third_party/llvm-project/include/llvm/ADT/MapVector.h:19:
:info:build /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_macports_release_tarballs_ports_lang_binaryen/binaryen/work/binaryen-111/third_party/llvm-project/include/llvm/ADT/DenseMap.h:128:16: error: variable 'NumEntries' set but not used [-Werror,-Wunused-but-set-variable]
:info:build       unsigned NumEntries = getNumEntries();
:info:build                ^
:info:build /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_macports_release_tarballs_ports_lang_binaryen/binaryen/work/binaryen-111/src/wasm/wasm-binary.cpp:860:13: error: variable 'emitted' set but not used [-Werror,-Wunused-but-set-variable]
:info:build       Index emitted = 0;
:info:build             ^
:info:build 1 error generated.
:info:build make[2]: *** [src/wasm/CMakeFiles/wasm.dir/wasm-debug.cpp.o] Error 1

Previous ticket #66294 said

Yes, ports should not use -Werror precisely for this kind of reason.

and since this has come up twice now, perhaps that patch should be made.

Change History (2)

comment:1 Changed 5 months ago by jmroot (Joshua Root)

Owner: set to MarcusCalhoun-Lopez
Status: newassigned

comment:2 Changed 5 months ago by amaksoft (Andrei Makeev)

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.