lxde-base/lxdm-0.4.1-r5 supports both PAM and ConsoleKit, and can very well be used without ConsoleKit, with PAM. I have looked at the code and it seems to be well made and uses ConsoleKit only if it is there. I have tested it on a ConsoleKit less system and it builds/runs well without ConsoleKit. See output of the ./configure attached. Please add a consolekit USE flag for the dependency to sys-auth/consolekit Reproducible: Always Steps to Reproduce: 1. emerge lxdm 2. requires sys-auth/consolekit Actual Results: hard dependency to consolekit Expected Results: a USE flag should be there to tell if we want to use consolekit or not.
Created attachment 329740 [details] Output of ./configure on a ConsoleKit free system
Created attachment 329742 [details, diff] Requested ebuild diff
also present in the lxdm.spec.in file: "" BuildRequires: gtk2-devel pam-devel intltool ""
(In reply to comment #2) > Created attachment 329742 [details, diff] [details, diff] > Requested ebuild diff This is wrong. the dependency should be made optional instead of just removing it
Created attachment 329770 [details, diff] Requested ebuild change
Created attachment 329772 [details, diff] Requested ebuild change with consolekit USE flag
(In reply to comment #4) > This is wrong. the dependency should be made optional instead of just > removing it Yes, as mentioned in my bug report, new diff is up! :)
Still the logic is not correct. Even if you use '-consolekit' to build the package, but you have consolekit installed, then it will use consolekit. There must be a way to disable consolekit in the source code or using --enable/--with consolekit when you are configuring the package
Created attachment 329826 [details, diff] Patch to add --enable-consolekit The configure.ac does not support a flag for ConsoleKit support, only environment variables. This adds the option in configure.ac.
Created attachment 329828 [details, diff] Requested ebuild change with consolekit USE flag New ebuild patch, supporting the --enable-consolekit configure option.
actually this patch defaults to the current behaviour for --enable-consolekit, which means: --enable-consolekit(default value): will check for ConsoleKit but if not found will issue a warning and continue to build without it. --disable-console-kit: will not check for ConsoleKit and will not use it at all. If we want to stop the build when having --enable-consolekit, but no ConsoleKit found, you can replace: "echo "ConsoleKit devel package not found"" with "AC_MSG_ERROR([ConsoleKit devel package not found])" in the patch, to have the build stopped.
Hey guys can we get this closed up? A year old now... anyway here is the patch modified with your suggestion, (not that its needed anymore r7 seems to build without error against non-consolekit users i.e. me... also attached is the new ebuild file
Created attachment 365718 [details, diff] Modified Patch for fail on enableconsolekit but no consolekit
Created attachment 365720 [details, diff] New ebuild for consolekit use flag Ok good to go now?? I hope so.
I don't think the patch works. x86_64-pc-linux-gnu-gcc -DHAVE_CONFIG_H -I. -I.. -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -DCONFIG_FILE=\"/etc/lxdm/lxdm.conf\" -DXSESSIONS_DIR=\"/usr/share/xsessions\" -DLXDM_DATA_DIR=/usr/share/lxdm -DLXDM_NUMLOCK_PATH=\"/usr/libexec/lxdm-numlock\" -Werror-implicit-function-declaration -Wall -O2 -march=native -pipe -DENABLE_PASSWORD -DLXDM_DEBUG -DLXDM_XCONN_XCB -MT lxdm_binary-lxdm.o -MD -MP -MF .deps/lxdm_binary-lxdm.Tpo -c -o lxdm_binary-lxdm.o `test -f 'lxdm.c' || echo './'`lxdm.c lxdm.c:70:26: fatal error: ck-connector.h: No such file or directory
The upstream patch is wrong. I will fix it.
Upstream patch fixed http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/lxde-base/lxdm/files/lxdm-0.4.1-optional-consolekit.patch?revision=1.1&view=markup and reported upstream. Ebuild committed http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/lxde-base/lxdm/lxdm-0.4.1-r8.ebuild?view=log Sorry for the delay.
