Hi,

Although kdm reports the correct shutdown commands in the comments for the 
kdmrc file, these 'defaults' are not actually adhered to.  

This is due to a few missing includes resulting in the correct system not 
being detected (<sys/param.h> is missing, which means BSD never gets defined 
and thus kdm defaults to linux).  

It appears that the only place sys/param is included properly is in genkdmconf 
which is enough for the comments about the default to be correct but nothing 
more.  

See attached for a patch that fixes this.  It might be considered a bit of a 
hack but it fits in nicely with all the other 'hacks'.  

Regards,

David

P.S. I've submitted a bug report: 
diff -ur kdm~/config.def kdm/config.def
--- kdm~/config.def	2009-11-17 12:53:01.000000000 +0200
+++ kdm/config.def	2009-11-17 12:55:35.000000000 +0200
@@ -32,6 +32,9 @@
 #define KDMCONF KDE_CONFDIR "/kdm"
 #define KDMDATA KDE_DATADIR "/kdm"
 
+#include <sys/param.h>  /* This is required for BSD define (how else will we
+                           known it exists?). */
+
 #ifdef _AIX
 # define HALT_CMD	"/usr/sbin/shutdown -h now"
 # define REBOOT_CMD	"/usr/sbin/shutdown -r now"

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
kde-freebsd mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-freebsd
See also http://freebsd.kde.org/ for latest information

Reply via email to