Hi Fabian, On Wed, Sep 19, 2018 at 09:44:51PM +0200, Fabian Raetz wrote: > @abieber looked into adding support for OpenBSD in 2017 > (https://github.com/bazil/fuse/pull/104). His branch compiles but > go test fails for me. I think there is some more work involved to get this > into a useable shape. > > I propose that we leave fuse deactived for now and don't talk about > fuse in DESCR.
That's fine, thanks for looking. > New port is attached. Here's some more comments. It's all minor stuff from here on. The whitespace looks much better, thanks. I wonder if the user and service name should be '_go-ipfs' and 'go-ipfs' to accommodate different implementations in the future (Maybe /var/go-ipfs too?). I don't feel strongly about this though. The user is created with /bin/sh as a shell. Some other ports create users with /sbin/nologin as a shell. Can we get away with that? (You'd need the `ipfs init` described in the README to work without the user having proper shell). Again, it's not a big deal if it isn't possible. I'm now torn about whether to set the $HOME for the user to /var/ipfs. If we do, then it simplifies the 'ipfs init' command and the RC script. Did you have any other thoughts on that? Also optionally, you could mention the web UI in the README (http://127.0.0.1:5001/webui). It's very shiny. Cheers -- Best Regards Edd Barrett http://www.theunixzoo.co.uk
