Re: New package: wired-notify-0.10.2

[classabbyamp <classabbyamp@users.noreply.github.com>]

[-- Attachment #1: Type: text/plain, Size: 1093 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/39738#issuecomment-1269070921

Comment:
many other distros don't support the architectures void supports.

the build only failed on 32-bit architectures (i686, armv6l, and armv7l).

[the error](https://github.com/void-linux/void-packages/actions/runs/3191379385/jobs/5210738618#step:7:406) was caused by the upstream not ensuring their types matched, i.e. they were not checking if [this `u64`](https://github.com/Toqozz/wired-notify/blob/4bb3dfff38934c47be818f497af65d1e3a01311c/src/config.rs#L120) and [this `c_ulong`](https://docs.rs/x11/2.20.0/x11/xss/struct.XScreenSaverInfo.html#structfield.idle) could actually be compared ([`c_ulong`](https://doc.rust-lang.org/nightly/std/os/raw/type.c_ulong.html) can be `u32` or `u64`). there may be other errors (because the devs obviously aren't testing anything on 32-bit systems), but this one can be fixed by (or something similar):

```diff
- if info.idle / 1000 >= threshold {
+ if (info.idle as u64) / 1000 >= threshold {
```