logiops icon indicating copy to clipboard operation
logiops copied to clipboard

logid seems completely broken

Open hendriktews opened this issue 1 year ago • 6 comments

Hi,

with the following config

devices: ({
  name: "Wireless Mouse MX Master 3";

  buttons: (
    { cid: 0xc3;
      action = {
        type: "Keypress";
        keys: ["BTN_MIDDLE"];
      };
    }
  );
});

with the current version v0.3.2 sudo ./logid -v -c ~/logid.cfg outputs

[INFO] Device found: Wireless Mouse MX Master 3 on /dev/hidraw0:255
[DEBUG] /dev/hidraw0:255 remappable buttons:
[DEBUG] CID  | reprog? | fn key? | mouse key? | gesture support?
[DEBUG] 0x50 |         |         | YES        | 
[DEBUG] 0x51 |         |         | YES        | 
[DEBUG] 0x52 | YES     |         | YES        | YES
[DEBUG] 0x53 | YES     |         | YES        | YES
[DEBUG] 0x56 | YES     |         | YES        | YES
[DEBUG] 0xc3 | YES     |         | YES        | YES
[DEBUG] 0xc4 | YES     |         | YES        | YES
[DEBUG] 0xd7 | YES     |         |            | YES
[DEBUG] Thumb wheel detected (0x2150), capabilities:
[DEBUG] timestamp | touch | proximity | single tap
[DEBUG] YES       | YES   | YES       | YES       
[DEBUG] Thumb wheel resolution: native (18), diverted (120)

and terminates. This config works fine in v0.2.4. With version v0.3.1 logid does not terminate, but the thumb button does not work as expected.

hendriktews avatar May 28 '23 17:05 hendriktews

The difference in behavior between v0.3.1 and v0.3.2 seems to come from be5ee9f7934a9a8b8443c9590bc45c41ebb4a844. I also tried to bisect where the problem was introduced after v0.2.4, but I hit too many intermediate versions with other problems.

hendriktews avatar May 28 '23 20:05 hendriktews

Yeah, such breaking changes are way too bad for bisect, I also hit this trouble while debugging an other problem. The code should be buildable and runnable at each commit in the master branch (or at least it should print something like 'dbus config files does not found - please run make install to install them - terminating').

and terminates.

Please run make install at least once after you have switched to the v0.3.2 (it absolutely requires this or it will terminate after initialization). You can't just run logid v0.3.2 if it was not installed because it needs to have some dbus configs installed in your system.

S-trace avatar May 29 '23 00:05 S-trace

This issue still persists on v0.3.2 after installing the dbus config file (make install)

MKuranowski avatar May 29 '23 02:05 MKuranowski

I have the same finding, 0.2.4 works fine; 0.3.1 and 0.3.2 no (0.3.0 does not build). git bisect suggest that the bug was introduced in 918ea63755a66d69373d7da431b61203fa878b41.

MKuranowski avatar May 29 '23 02:05 MKuranowski

Thanks for the feedback! The debian packages 0.3.1 and 0.3.2-1 work for me, which suggests to me that I missed the installation. I will try make install later.

hendriktews avatar Jun 06 '23 12:06 hendriktews

This is still an issue on v0.3.3

MKuranowski avatar Mar 15 '24 19:03 MKuranowski