jonhoo / buzz

A simple system tray application for notifying about unseen e-mail
Apache License 2.0
138 stars 12 forks source link

Buzz does not build using rustc 1.41 #11

Closed ftilde closed 4 years ago

ftilde commented 4 years ago

Snippet from the build:

   Compiling syntex_syntax v0.54.0
error[E0423]: expected function, tuple struct or tuple variant, found struct `ast::Name`
   --> /home/dominik/.cargo/registry/src/github.com-1ecc6299db9ec823/syntex_syntax-0.54.0/src/symbol.rs:146:27
    |
146 |                       name: ast::Name($index),
    |                             ^^^^^^^^^
...
165 | / declare_keywords! {
166 | |     // Invalid identifier
167 | |     (0,  Invalid,        "")
168 | |
...   |
231 | |     (56, CrateRoot, "{{root}}")
232 | | }
    | |_- in this macro invocation

The issue is also described here and happens due to the dependency-chain systray -> libappindicator -> libappindicator-sys -> (an ancient version of) bindgen -> aster -> syntex_syntax.

The issue is fixed in libappindactor and the master branch of systray. However, the latter does not build at the moment due to other issues. The fix for this project is probably to wait until a new, fixed version of systray is published. In the meantime, if anyone wants to build buzz using rustc 1.41, they can temporarliy use the PR-branch that I submitted to systray to fix the upgrade-issues (at least for Linux/*BSD/...) using a patch section in the Cargo.toml of buzz:

[patch.crates-io]
systray = { git = "https://github.com/ftilde/systray-rs" }
jonhoo commented 4 years ago

Yup, I ran into this recently too, and am just holding for systray to be updated (I know @qdot is working on it). Thanks for posting the workaround!

qdot commented 4 years ago

Just pushed 0.4.0 (with @ftilde's patch (thanks! :D ), Rust 2018 support, etc) to crates.io. I compiled it and checked the examples, but that's about as far as testing went for the moment. Lemme know if it works for you.

jonhoo commented 4 years ago

Published in buzz 1.2!