the-cavalry / light-locker

A simple session-locker for lightdm
GNU General Public License v2.0
224 stars 22 forks source link

Adding support for elogind's standalone logind daemon. #117

Open roogus opened 6 years ago

roogus commented 6 years ago

This PR completes the work started by @dg98 in PR #109 . It adds the necessary preprocessor directives to make use of the --with-elogind autoconf option previously introduced, and explicitly links with the elogind library to provide full functionality.

These changes compile and run as expected (afik) on my gentoo machine with: lightdm-1.28.0, elogind-238.1, and openrc-0.38.2 which should resolve #103 .

This is my first go-round with autotools scripting, so any comments/critiques are most welcome.

slavoutich commented 6 years ago

Doesn't run for me with lightdm-1.26.0-r1, system can't be unlocked. Will investigate further.

slavoutich commented 6 years ago

Sorry, stupid me forgot how to patch, everything works for me.

casta commented 5 years ago

Confirmed here too that this patch fixes black screen when unlocking if xorg is compiled with elogind support

a17r commented 5 years ago

This patch fails to build on a non-elogind system though with --without-console-kit --with-systemd --without-elogind.

/usr/x86_64-pc-linux-gnu/bin/ld: cannot find -lelogind

https://github.com/gentoo/gentoo/pull/11460#pullrequestreview-220154071