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

notify-cli fails to build on mac os x el capitan #10

Closed
sumproxy opened this issue Oct 2, 2015 · 4 comments
Closed

notify-cli fails to build on mac os x el capitan #10

sumproxy opened this issue Oct 2, 2015 · 4 comments

Comments

@sumproxy
Copy link

sumproxy commented Oct 2, 2015

Here's the output

cargo build
    Updating registry `https://github.com/rust-lang/crates.io-index`
 Downloading clap v1.4.3
   Compiling ansi_term v0.6.3
   Compiling libc v0.1.10
   Compiling strsim v0.4.0
   Compiling dbus v0.1.2
   Compiling clap v1.4.3
   Compiling notify-rust v3.0.0 (file:///Users/SumProxy/Projects/rust/notify-rust/notify-cli)
   Compiling notify-rust-cli v0.2.0 (file:///Users/SumProxy/Projects/rust/notify-rust/notify-cli)
src/main.rs:97:21: 97:72 error: use of unstable library feature 'result_expect': newly introduced (see issue #27277)
src/main.rs:97             .show().expect("Was not able to send initial test message");
                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
note: in expansion of if let expansion
src/main.rs:79:5: 170:6 note: expansion site
<std macros>:2:1: 2:54 warning: unused result which must be used, #[warn(unused_must_use)] on by default
<std macros>:2 $ dst . write_fmt ( format_args ! ( $ ( $ arg ) * ) ) )
               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
<std macros>:1:1: 2:56 note: in expansion of write!
<std macros>:2:1: 2:46 note: expansion site
<std macros>:1:1: 4:66 note: in expansion of writeln!
src/main.rs:159:17: 160:64 note: expansion site
note: in expansion of if let expansion
src/main.rs:114:10: 170:6 note: expansion site
note: in expansion of if let expansion
src/main.rs:106:10: 170:6 note: expansion site
note: in expansion of if let expansion
src/main.rs:79:5: 170:6 note: expansion site
<std macros>:2:1: 2:54 warning: unused result which must be used, #[warn(unused_must_use)] on by default
<std macros>:2 $ dst . write_fmt ( format_args ! ( $ ( $ arg ) * ) ) )
               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
<std macros>:1:1: 2:56 note: in expansion of write!
<std macros>:2:1: 2:46 note: expansion site
<std macros>:1:1: 4:66 note: in expansion of writeln!
src/main.rs:166:17: 167:64 note: expansion site
note: in expansion of if let expansion
src/main.rs:114:10: 170:6 note: expansion site
note: in expansion of if let expansion
src/main.rs:106:10: 170:6 note: expansion site
note: in expansion of if let expansion
src/main.rs:79:5: 170:6 note: expansion site
error: aborting due to previous error
Could not compile `notify-rust-cli`.

To learn more, run the command again with --verbose.
rustc --version
rustc 1.3.0 (9a92aaf19 2015-09-15)
cargo --version
cargo 0.4.0-nightly (553b363 2015-08-03) (built 2015-08-02)
@sumproxy sumproxy changed the title notify-cli fails to build on mac os x notify-cli fails to build on mac os x el capitan Oct 2, 2015
@hoodie
Copy link
Owner

hoodie commented Oct 2, 2015

Sorry, this has a dependencie on "org.freedesktop.Notifications" and dbus, so for now, this is Linux only. I suppose I should have said that somewhere.

@sumproxy
Copy link
Author

sumproxy commented Oct 2, 2015

Oh, I see..

@sumproxy sumproxy closed this as completed Oct 2, 2015
@hoodie
Copy link
Owner

hoodie commented Oct 2, 2015

I would love for this lib to also support windows and mac. But they have completely different apis.
But if we could marry the Linux, Mac and Windows worlds in one crate and offer almost the same publich api, that would be great. Though I have no clue what the other NotificationSpecs look like, or if windows even has one.

@sumproxy
Copy link
Author

sumproxy commented Oct 2, 2015

Yeah, I personally would be happy to see one working on mac, I haven't heard of a notification api on windows, though.

jonhoo added a commit to jonhoo/buzz that referenced this issue Mar 2, 2017
Should help with #2, and maybe #1, at least if the comment by @hoodie in
hoodie/notify-rust#10 comes to fruition.
Certainly a step up from running notify-send manually.
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

2 participants