search

gen2brain / beeep

Go cross-platform library for sending desktop notifications, alerts and beeps
BSD 2-Clause "Simplified" License
1.43k stars 89 forks source link

Support BSD's #14

Closed ghost closed 6 years ago

ghost commented 6 years ago

All the BSD's use the same notify-send program as Linux. It would be nice to see this working.

gen2brain commented 6 years ago

Can you try now with https://github.com/gen2brain/beeep/commit/b81ad3c425bfa9174c54a1d650bb65fd13f9cf62 ?

ghost commented 6 years ago

Look's like its failing still.

go run main.go
panic: beeep: unsupported operating system: freebsd
goroutine 1 [running]:
main.main()
        /home/gregf/dev/go/src/github.com/gregf/beeep/main.go:8 +0x4e
exit status 2
ghost commented 6 years ago

Not sure if you are aware but your tests seem to fail as well

 _/home/gregf/beeep
./example_test.go:8:8: too many arguments in call to Notify
        have (string, string, string)
        want (string, string)
./notify_test.go:8:15: too many arguments in call to Notify
        have (string, string, string)
        want (string, string)
FAIL    _/home/gregf/beeep [build failed]
gen2brain commented 6 years ago

It should be ok now. Also, it seems you are using old copy of library, try to update with go get -u.

ghost commented 6 years ago

Working now, thank you so much for the speedy response!