jckuester / weather-station

A Prometheus exporter for 433 MHz temperature and humidity sensors
Apache License 2.0
64 stars 6 forks source link

Darwin build failed #23

Open jckuester opened 5 years ago

jckuester commented 5 years ago
   ⨯ release failed after 47.97s error=failed to build for darwin_amd64: # github.com/tarm/serial
../../../go/pkg/mod/github.com/tarm/serial@v0.0.0-20180830185346-98f6abe2eb07/serial.go:128:28: undefined: Port
../../../go/pkg/mod/github.com/tarm/serial@v0.0.0-20180830185346-98f6abe2eb07/serial.go:139:9: undefined: openPort

@martinhoefling just to let you know, after merging PR https://github.com/jckuester/weather-station/pull/17, build for darwin fails.

martinhoefling commented 5 years ago

I can check this out. Is Darwin platform a use case? I just did cross builds to Linux from my Mac so far.

martinhoefling commented 5 years ago

Interesting, after installing the command line tools via xcode-select, I can build weather-station via go build (go1.13.1 darwin/amd64 on Catalina).

martinhoefling commented 5 years ago

@jckuester: Are you building on a mac? Goreleaser also runs fine. Maybe cross compilation does not work without xcode tools due to tarm/serial?

╰─ goreleaser --snapshot --skip-publish --rm-dist

   • releasing using goreleaser 0.119.0...
   • loading config file       file=.goreleaser.yml
   • RUNNING BEFORE HOOKS
   • LOADING ENVIRONMENT VARIABLES
      • pipe skipped              error=publishing is disabled
   • GETTING AND VALIDATING GIT STATE
      • releasing v0.3.0, commit 45502fcc83f5dcff9f945a495745d4de0e08f182
      • pipe skipped              error=disabled during snapshot mode
   • PARSING TAG
   • SETTING DEFAULTS
      • LOADING ENVIRONMENT VARIABLES
      • SNAPSHOTING
      • GITHUB/GITLAB/GITEA RELEASES
      • PROJECT NAME
      • BUILDING BINARIES
      • ARCHIVES
      • LINUX PACKAGES WITH NFPM
      • SNAPCRAFT PACKAGES
      • CALCULATING CHECKSUMS
      • SIGNING ARTIFACTS
      • DOCKER IMAGES
      • ARTIFACTORY
      • S3
      • BLOB
      • HOMEBREW TAP FORMULA
         • optimistically guessing `brew[0].installs`, double check
      • SCOOP MANIFEST
   • SNAPSHOTING
   • CHECKING ./DIST
   • WRITING EFFECTIVE CONFIG FILE
      • writing                   config=dist/config.yaml
   • GENERATING CHANGELOG
      • pipe skipped              error=not available for snapshots
   • BUILDING BINARIES
      • building                  binary=dist/weather-station_linux_amd64/weather-station
      • building                  binary=dist/weather-station_linux_arm64/weather-station
      • building                  binary=dist/weather-station_windows_amd64/weather-station.exe
      • building                  binary=dist/weather-station_linux_arm_6/weather-station
      • building                  binary=dist/weather-station_linux_arm_7/weather-station
      • building                  binary=dist/weather-station_darwin_amd64/weather-station
   • ARCHIVES
      • creating                  archive=dist/weather-station_SNAPSHOT-45502fc_linux_amd64.tar.gz
      • creating                  archive=dist/weather-station_SNAPSHOT-45502fc_linux_arm64.tar.gz
      • creating                  archive=dist/weather-station_SNAPSHOT-45502fc_windows_amd64.tar.gz
      • creating                  archive=dist/weather-station_SNAPSHOT-45502fc_linux_armv7.tar.gz
      • creating                  archive=dist/weather-station_SNAPSHOT-45502fc_linux_armv6.tar.gz
      • creating                  archive=dist/weather-station_SNAPSHOT-45502fc_darwin_amd64.tar.gz
   • LINUX PACKAGES WITH NFPM
      • pipe skipped              error=no output formats configured
   • SNAPCRAFT PACKAGES
      • pipe skipped              error=no summary nor description were provided
   • CALCULATING CHECKSUMS
      • checksumming              file=weather-station_SNAPSHOT-45502fc_windows_amd64.tar.gz
      • checksumming              file=weather-station_SNAPSHOT-45502fc_darwin_amd64.tar.gz
      • checksumming              file=weather-station_SNAPSHOT-45502fc_linux_amd64.tar.gz
      • checksumming              file=weather-station_SNAPSHOT-45502fc_linux_armv7.tar.gz
      • checksumming              file=weather-station_SNAPSHOT-45502fc_linux_armv6.tar.gz
      • checksumming              file=weather-station_SNAPSHOT-45502fc_linux_arm64.tar.gz
   • SIGNING ARTIFACTS
      • pipe skipped              error=artifact signing is disabled
   • DOCKER IMAGES
      • pipe skipped              error=docker section is not configured
   • PUBLISHING
      • pipe skipped              error=publishing is disabled
   • release succeeded after 43.80s
jckuester commented 5 years ago

Hi @martinhoefling. Thanks for looking into this. I am using goreleaser under Linux. Mac/darwin is not a use-case for me (I use Rasberry Pi/ARM), but I thought it might be nice to offer compiled binaries for people out there.