search

haimgel / display-switch

Turn a $30 USB switch into a full-featured multi-monitor KVM switch
https://haim.dev/posts/2020-07-28-dual-monitor-kvm/
MIT License
2.86k stars 112 forks source link

lexical-core >=0.7.5 required to build on Alpine Linux #68

Closed craftyguy closed 2 years ago

craftyguy commented 3 years ago

I'm running into the issue described here when trying to build display-switch on Alpine: https://github.com/Alexhuszagh/rust-lexical/issues/55

error[E0277]: cannot subtract `usize` from `u32`
    --> /home/clayton/.cargo/registry/src/github.com-1ecc6299db9ec823/lexical-core-0.7.4/src/atof/algorithm/math.rs:2065:25
     |
2065 |     let rs = Limb::BITS - s;
     |                         ^ no implementation for `u32 - usize`

The fix for this is apparently in lexical-core 0.7.5, tested locally and was able to build display-switch after setting lexical-core to 0.7.5 in Cargo.lock. Could you bump the version used in this project?

craftyguy commented 3 years ago

ah I see that was done here: https://github.com/haimgel/display-switch/pull/66

So, any chance of getting a new tagged release with that in it? 😄

haimgel commented 2 years ago

Thanks for the reminder! Done.