On Thu, Aug 30, 2018 at 06:49:34PM -0400, Brian Callahan wrote: > > On 08/30/18 18:35, Brian Callahan wrote: > > Hi Fabian -- > > > > On 08/30/18 17:28, Fabian Raetz wrote: > > > On Thu, Aug 30, 2018 at 05:12:35PM -0400, Brian Callahan wrote: > > > > Hi Fabian -- > > > > > > > > On 08/30/18 16:42, Fabian Raetz wrote: > > > > > Hi all. > > > > > > > > > > i've been using the "WebAssembly Binary Toolkit" lately and > > > > > thought i create a > > > > > proper port for it. > > > > > > > > > > The port doesn't support running the tests as they require > > > > > some git submodules including > > > > > the gtest source. Sadly, my cmake skills are not good enough > > > > > to make the build system > > > > > use gtest from ports so i decided against it for the moment. > > > > > > > > > > In case you wanna compile a small WebAssembly programm > > > > > yourself, take a look at > > > > > https://jameshfisher.com/2017/10/13/webassembly-hello-world.html > > > > > > > > > > Cheers, > > > > > Fabian > > > > > > > > > > HOMEPAGE: https://github.com/WebAssembly/wabt > > > > > > > > > > DESCR: > > > > > WABT (we pronounce it "wabbit") is a suite of tools for > > > > > WebAssembly, including: > > > > > > > > > > wat2wasm: translate from WebAssembly text format to the > > > > > WebAssembly binary > > > > > format > > > > > wasm2wat: the inverse of wat2wasm, translate from the > > > > > binary format back > > > > > to the text format (also known as a .wat) > > > > > wasm-objdump: print information about a wasm binary. > > > > > Similiar to objdump. > > > > > wasm-interp: decode and run a WebAssembly binary file > > > > > using a stack-based > > > > > interpreter > > > > > wat-desugar: parse .wat text form as supported by the > > > > > spec interpreter > > > > > (s-expressions, flat syntax, or mixed) > > > > > and print "canonical" > > > > > flat format > > > > > wasm2c: convert a WebAssembly binary file to a C source and header > > > > > > > > > > > > > > Is this different from the wabt port I ok'd here? > > > > https://marc.info/?l=openbsd-ports&m=152950481400669&w=2 > > > There are some subtle differences. The port I sent is in the > > > category "devel" > > > instead of "lang". Also, I packaged version 1.0.5 where the local > > > patch is > > > already included. DESCR is also slightly different with regards to > > > formating. > > > > > > Otherwise both port are equal :) > > > > I changed the DESCR formatting to be different than both of them: > > --- > > WABT (we pronounce it "wabbit") is a suite of tools for WebAssembly, > > including: > > > > wat2wasm: > > translate from WebAssembly text format to the WebAssembly binary format > > > > wasm2wat: > > the inverse of wat2wasm, translate from the binary format back to the > > text format (also known as a .wat) > > > > wasm-objdump: > > print information about a wasm binary. Similiar to objdump. > > > > wasm-interp: > > decode and run a WebAssembly binary file using a stack-based > > interpreter > > > > wat-desugar: > > parse .wat text form as supported by the spec interpreter > > (s-expressions, flat syntax, or mixed) and print "canonical" flat > > format > > > > wasm2c: > > convert a WebAssembly binary file to a C source and header > > ---
Thanks for your review :) I attached a new tar which integrates your feedback. Some comments inline ... > > > > It reads better for my eyes but I guess these things eventually become a > > matter of opinion. > > > > On the more necessary side of things: > > * the build picks up a -Werror that has to go I patched the CMakeLists.txt. Is there a better way to this? > > * CMake will pick up re2c if you have it installed so either set > > -DRUN_RE2C=OFF in CONFIGURE_ARGS or add a BDEP on re2c. Added the BDEP because the option is on by default. > > * It's C++11, so needs a COMPILER line. Done. > > * It looks for, and finds, python. Do we need to add MODULES=lang/python? > > > > There's no MAINTAINER. Either of you want to do it? I would take it and added myself as MAINTAINER if that's ok for Raphael > > > > And now that I think about it, I think I like this living in lang/. devel/ > is overloaded, and we can at least justify this one living somewhere else. Done. > > ~Brian > > > ~Brian > > > > > > I'm still waiting for an OK to import that one. > > > > > > > > ~Brian > > >
wabt-1.0.5-v2.tar.gz
Description: Binary data
