Super-productivity: xprintidle does not work in snap package

[endorama]

Your Environment

• Version used: 2.5.3 snap revision 170
• Operating System and version: Ubuntu 16.04.4
• Desktop Environment: Unity

Expected Behavior

Idle checker works.

Current Behavior

Idle checker error pop up is displayed.

Steps to Reproduce (for bugs)

1. Install from snap
2. run it
3. install xprintidle system wide (apt install ...)
4. run it again, same result

Console Output

no relevant console logs

Error Log (Desktop only)

It seems that the error log for snap package is not in the mentioned location (~/.config/superProductivity/log.log).

Anyway I had the same problem with the AppImage (that has been solved by installing xprintidle package) so I guess the error would be the same.

[2019-03-01 15:19:21.596] [error] Something went wrong with the idle checker. You need to install xprintidle. Error: Something went wrong with the idle checker. You need to install xprintidle.
    at Object.exports.errorHandler (/tmp/.mount_superPgm3GTb/resources/app.asar/electron/error-handler.js:10:18)
    at /tmp/.mount_superPgm3GTb/resources/app.asar/electron/get-idle-time.js:33:33
    at ChildProcess.exithandler (child_process.js:306:5)
    at ChildProcess.emit (events.js:182:13)
    at maybeClose (internal/child_process.js:962:16)
    at Socket.stream.socket.on (internal/child_process.js:381:11)
    at Socket.emit (events.js:182:13)
    at Pipe._handle.close (net.js:606:12)

I'm willing to help in debugging this, but I'm now sure if able.
I looked at the Snapcraft docs about dependencies but are lacking. If someone can point me in the right direction I can try fixing this :)

Thanks for the project BTW, is super useful!

[johannesjo]

Thanks for reporting. The solution is easy fortunately. You just need to install xprintidle yourself via apt-get.

Problem is that the snap dependency is not working correctly. I probably should document this somewhere :)

[endorama]

Thanks for the super quick response!

Unfortunately I do have it installed, but still the message appears.

>

[johannesjo]

Oh Sorry. Was reading this on my phone and didn't quite catch that. In this case I have to check :)

[johannesjo]

May I ask what the output of xprintidle is in the command line? The electron layer simply executes the command and grabs the output, so it seems it hasn't access to the command for some reason.

I would to automatically install xprintidle with the snap package, but adding any dependencies is killing the build:
https://github.com/electron-userland/electron-builder/issues/3677

[endorama]

$ xprintidle
42

So it is expected for the snap package to use the "host" xprintidle binary, correct?

Let me know if I can help more on this!

[johannesjo]

Ok that seems to be in order.

Yes you're right. To be me this looks like Super Productivity doesn't have access to xprintidle for some reason. Not sure why to be honest. I will check this tomorrow.

[johannesjo]

The issue should hopefully be fixed now. I'll release a patch version soon. Please let me know if the issue persists or you discover any other.

[endorama]

I do not see the error anymore! Thanks for the quick fix!