wooster0 / yayagram

Play nonograms/picross in your terminal
https://crates.io/crates/yayagram
38 stars 2 forks source link

`cargo install yayagram` fails in tanmatsu #3

Closed gamma-delta closed 2 years ago

gamma-delta commented 2 years ago

Hello,

Trying to install yayagram gives me the following error.

$ cargo install yayagram
# ... listing of downloaded crates ...
error[E0277]: the trait bound `fn(u16) -> MoveToNextLine {MoveToNextLine}: crossterm::Command` is not satisfied
   --> C:\Users\anon\.cargo\registry\src\github.com-1ecc6299db9ec823\tanmatsu-0.6.3\src\sys\other.rs:222:27
    |
222 |         self.stdout.queue(cursor::MoveToNextLine).unwrap();
    |                     ----- ^^^^^^^^^^^^^^^^^^^^^^ the trait `crossterm::Command` is not implemented for `fn(u16) -> MoveToNextLine {MoveToNextLine}`
    |                     |
    |                     required by a bound introduced by this call
    |
note: required by a bound in `queue`
   --> C:\Users\anon\.cargo\registry\src\github.com-1ecc6299db9ec823\crossterm-0.20.0\src\command.rs:60:39
    |
60  |     fn queue(&mut self, command: impl Command) -> Result<&mut Self>;
    |                                       ^^^^^^^ required by this bound in `queue`

error[E0277]: the trait bound `fn(u16) -> MoveToPreviousLine {MoveToPreviousLine}: crossterm::Command` is not satisfied
   --> C:\Users\anon\.cargo\registry\src\github.com-1ecc6299db9ec823\tanmatsu-0.6.3\src\sys\other.rs:226:27
    |
226 |         self.stdout.queue(cursor::MoveToPreviousLine).unwrap();
    |                     ----- ^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `crossterm::Command` is not implemented for `fn(u16) -> MoveToPreviousLine {MoveToPreviousLine}`
    |                     |
    |                     required by a bound introduced by this call
    |
note: required by a bound in `queue`
   --> C:\Users\anon\.cargo\registry\src\github.com-1ecc6299db9ec823\crossterm-0.20.0\src\command.rs:60:39
    |
60  |     fn queue(&mut self, command: impl Command) -> Result<&mut Self>;
    |                                       ^^^^^^^ required by this bound in `queue`

For more information about this error, try `rustc --explain E0277`.
error: could not compile `tanmatsu` due to 2 previous errors
warning: build failed, waiting for other jobs to finish...
error: failed to compile `yayagram v0.8.0`, intermediate artifacts can be found at `C:\cygwin64\tmp\cargo-installklkVGp`

Caused by:
  build failed
wooster0 commented 2 years ago

Hello, I made the tanmatsu repository public again. It didn't make much sense to make it private considering that the source is public anyway. I suppose you are on Windows. Apparently I never really tested those two or read the docs close enough to realize that those structs require a u16. Sorry about that. That terminal interface is more or less abandoned anyway now that I rewrote it. I haven't had time to finish up that interface though and I'm not sure the new API will ever be integrated with yayagram.

I just pushed a fix. Are you able to run cargo install yayagram?

To be honest, I don't think this game works all that well on Windows. Can you kindly report back to me how the experience is? I'll consider removing Windows as a platform from the README.