jistr / waketimed Goto Github PK

It seems that now on PinePhone the modem number in DBus API gets incremented with each sleep/wake cycle. Previously the rule relied on listing Modems only when waketimed started, and it seemed to work, but now the modem number gets incremented at device runtime. E.g. now my phone doesn't have /org/freedesktop/ModemManager1/Modem/1 anymore as when it booted up, but it has /org/freedesktop/ModemManager1/Modem/6. I guess we should just query the modems each time we poll the variable.

[2022-10-30T16:59:48Z ERROR waketimed::worker::var_worker] Could not send EngineMsg::ReturnVarPoll
    
    Caused by:
        channel closed

This doesn't cause any trouble since it only happens on termination of the daemon, but still it would be nice to get rid of this error, to keep logs clean. We should make sure that the engine message receiver outlives the worker thread.