Ensure code compiles on all supported platforms

[whereswaldon]

This PR makes TravisCI attempt to build our code for windows, darwin, linux, and openbsd after every change, which should prevent us from being unaware when compilation fails on a specific supported OS (e.g. @Caton101 realizing that I broke the build on Windows).

I did not include dragonflybsd, freebsd, or netbsd because they all failed to compile one of our dependencies. It looks like we might not be able to support them and also rely on godbus. I think that's probably a separate discussion though.

[BitiTiger]

Are you sure? FreeBSD worked just fine for me. Sent by, Cameron Himes On Jan 8, 2019, at 9:56 PM, Christopher Waldon <[email protected]<mailto:[email protected]>> wrote: This PR makes TravisCI attempt to build our code for windows, darwin, linux, and openbsd after every change, which should prevent us from being unaware when compilation fails on a specific supported OS (e.g. @Caton101<https://github.com/Caton101> realizing that I broke the build on Windows). I did not include dragonflybsd, freebsd, or netbsd because they all failed to compile one of our dependencies. It looks like we might not be able to support them and also rely on godbus. I think that's probably a separate discussion though.

…

________________________________ You can view, comment on, or merge this pull request online at: #75 Commit Summary * Ensure code compiles on all supported platforms File Changes * M .travis.yml<https://github.com/arborchat/muscadine/pull/75/files#diff-0> (1) Patch Links: * https://github.com/arborchat/muscadine/pull/75.patch * https://github.com/arborchat/muscadine/pull/75.diff — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#75>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AdlB5xWulMSytihcHmYeY_93zYMwj5Rsks5vBVpqgaJpZM4Z2q_u>.

[whereswaldon]

@Caton101 I get this when I attempt to build it:

env GOOS=freebsd go build
# github.com/arborchat/muscadine/vendor/github.com/godbus/dbus
vendor/github.com/godbus/dbus/transport_unix.go:52:3: cannot use t (type *unixTransport) as type transport in return argument:
        *unixTransport does not implement transport (missing SendNullByte method)
vendor/github.com/godbus/dbus/transport_unix.go:58:3: cannot use t (type *unixTransport) as type transport in return argument:
        *unixTransport does not implement transport (missing SendNullByte method)

It's possible that this problem has been fixed in godbus, but that we aren't on the latest version.

[jwhett]

🕺

[whereswaldon]

@Caton101 I think we should definitely try to get it building on those BSD platforms consistently, but that's not a reason to wait on ensuring that it builds for the ones listed in this PR. We can add the others as we have the opportunity.