Re: [PATCH][WIP] Elogind update

2018-11-15 Thread Stefan Stefanović
Hello, Guix. I solved my problem. Please use this new patch, which will be attached to this message. I must apologize for my carelessness. I unnecessary added dependency cycle between elogind and polkit. At the time I added polkit, as input dependency of elogind, I was looking at service extensi

Re: [PATCH][WIP] Elogind update

2018-11-14 Thread Marius Bakke
Hello Stefan, Thank you very much for this work! Stefan Stefanović writes: > Hello, > Ludo’. > > Thank you, for the response. > > Sorry for the verbose and long message, > it is low priority, so read it only you have time. :) > >>> Stefan Stefanović skribis: >>> >>> All that remains is to figu

Re: [PATCH][WIP] Elogind update

2018-11-14 Thread Stefan Stefanović
Hello, Ludo’. Thank you, for the response. Sorry for the verbose and long message, it is low priority, so read it only you have time. :) >> Stefan Stefanović skribis: >> >> All that remains is to figure out how to integrate this update to >> Guix, following the next release of elogind (v239.2).

Re: [PATCH][WIP] Elogind update

2018-11-14 Thread Ludovic Courtès
Hello, Stefan Stefanović skribis: > Status update: The solution Andy provided works. The PID file has a > new path: "/run/systemd/elogind.pid". Additional patch that is > needed, will be provided with this message. The patch LGTM. > All that remains is to figure out how to integrate this upd

Re: [PATCH][WIP] Elogind update

2018-11-13 Thread Stefan Stefanović
Status update: The solution Andy provided works. The PID file has a new path: "/run/systemd/elogind.pid". Additional patch that is needed, will be provided with this message. All that remains is to figure out how to integrate this update to Guix, following the next release of elogind (v239.2). A

Re: [PATCH][WIP] Elogind update

2018-11-12 Thread Stefan Stefanović
>Perhaps we could put the PID file in /run/systemd or something. We could change the PID file path, I have changed it to "/var/run/elogind.pid". I will change it to "/run/systemd/elogind.pid", I will test it and report tomorrow. I am thinking of making the PID file check more robust by covering th

Re: [PATCH][WIP] Elogind update

2018-11-12 Thread Andy Wingo
On Sat 10 Nov 2018 17:00, Stefan Stefanović writes: > But the problem is with the stale PID found after a forced/abrupt reboot. > As far as I know other distributions do not have to deal with this problem, > because they mount tmpfs to /run and /var/run. > We (Guix) need to find the 2nd part of t

[PATCH][WIP] Elogind update

2018-11-10 Thread Stefan Stefanović
Hello, Guix. As I announced, here is patch that let's us update elogind. Note: This patch is work in progress. Please do not mind the lack of proper commit message. Since I expect more changes to this patch, I did not bother writing it. As I far as I know, the last work on elogind is by Marius