Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support BSD's #14

Closed
ghost opened this issue Jul 18, 2018 · 5 comments
Closed

Support BSD's #14

ghost opened this issue Jul 18, 2018 · 5 comments

Comments

@ghost
Copy link

ghost commented Jul 18, 2018

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

@gen2brain
Copy link
Owner

Can you try now with b81ad3c ?

@ghost
Copy link
Author

ghost commented Jul 18, 2018

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
Copy link
Author

ghost commented Jul 18, 2018

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
Copy link
Owner

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

@ghost
Copy link
Author

ghost commented Jul 18, 2018

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

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant