frankbraun / gosignify

gosignify is a Go reimplementation of OpenBSD's signify
The Unlicense
17 stars 2 forks source link

Build fails on OpenBSD due to undefined syscalls #3

Closed errhammr closed 4 years ago

errhammr commented 4 years ago

I'm trying to build gosignify on OpenBSD but it fails.

$ git log -1
commit 76c6db581730e7cbbe8c138a27c3325c0b784430
Author: Frank Braun <frank@cryptogroup.net>
Date:   Wed Feb 27 14:34:07 2019 +0000

    travis: bump Go version to 1.12
$ go version
go version go1.13.9 openbsd/amd64
$ go build 
go: downloading golang.org/x/crypto v0.0.0-20181030102418-4d3f4d9ffa16
go: downloading github.com/ebfe/bcrypt_pbkdf v0.0.0-20140212075826-3c8d2dcb253a
go: extracting github.com/ebfe/bcrypt_pbkdf v0.0.0-20140212075826-3c8d2dcb253a
go: extracting golang.org/x/crypto v0.0.0-20181030102418-4d3f4d9ffa16
go: downloading golang.org/x/sys v0.0.0-20181031143558-9b800f95dbbc
go: extracting golang.org/x/sys v0.0.0-20181031143558-9b800f95dbbc
go: finding github.com/ebfe/bcrypt_pbkdf v0.0.0-20140212075826-3c8d2dcb253a
go: finding golang.org/x/crypto v0.0.0-20181030102418-4d3f4d9ffa16
go: finding golang.org/x/sys v0.0.0-20181031143558-9b800f95dbbc
# github.com/frankbraun/gosignify/internal/util
internal/util/mlock_other.go:11:2: undefined: syscall.Mlock
internal/util/mlock_other.go:16:2: undefined: syscall.Munlock
$ echo $?
2

Is OpenBSD a target OS for gosignify?

I know, I can just use the C version of signify that ships with OpenBSD but what's the fun in that?

frankbraun commented 4 years ago

Thanks for the report, it's fixed now. Compiled and tested on my OpenBSD machine.

I know, I can just use the C version of signify that ships with OpenBSD but what's the fun in that?

It wouldn't be fun, that's for sure ;)

errhammr commented 4 years ago

Thank you!