search

sandstorm-io / sandstorm

Sandstorm is a self-hostable web productivity suite. It's implemented as a security-hardened web app package manager.
https://sandstorm.io
Other
6.75k stars 706 forks source link

Powerbox docs: capnp eval not working #2961

Open dmitry-sher opened 7 years ago

dmitry-sher commented 7 years ago

Sandstorm version: 211 Capnp version: 0.5.3 OS: Ubuntu 16 x64 in VirtualBox (kernel: 4.8.0-040800-lowlatency)

I was going through tutorial here: https://docs.sandstorm.io/en/latest/developing/powerbox/ I just take the capnp file as it is from docs, run the eval command and it fails like this:

descriptor.capnp:16: error: Parse error: Empty list item.
descriptor.capnp:24: error: Parse error: Empty list item.
descriptor.capnp:1:1: error: Missing field name.
descriptor.capnp:28:39-50: error: Type mismatch; expected AnyPointer.
descriptor.capnp:1:1: error: Missing field name.

The same command works well for capnp on Mac OS X 10.12.5 (16F73), capnp installed today from homebrew. Installed file: capnp-0.6.1.sierra.bottle.tar.gz

$ capnp --version
Cap'n Proto version (unknown)

It'd be nice to update docs about minimum capnp version. I've installed capnp in Ubuntu VM with sudo apt-get install capnproto

kentonv commented 7 years ago

Historically Sandstorm has actually required that you always use Cap'n Proto from master, as for a long time we failed to find time to do a capnp release, and we'd often add features to Cap'n Proto and use them immediately in Sandstorm.

Now that capnp 0.6 is out (along with a streamlined release process that should be easier to keep up with going forward) and Sandstorm developed has slowed, this may not be an issue anymore. 0.6 should be good enough.

I agree the docs should indicate required capnp version so I'll leave this bug open.