Bug 214963 - [patch] kdm does not default to correct shutdown commands
Summary: [patch] kdm does not default to correct shutdown commands
Status: RESOLVED FIXED
Alias: None
Product: kdm
Classification: Unmaintained
Component: general (show other bugs)
Version: unspecified
Platform: FreeBSD Ports FreeBSD
: NOR normal
Target Milestone: ---
Assignee: kdm bugs tracker
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-11-17 13:47 UTC by David Naylor
Modified: 2010-02-21 16:39 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Naylor 2009-11-17 13:47:08 UTC
Version:            (using KDE 4.3.3)
Compiler:          gcc (GCC) 4.2.1 20070719  [FreeBSD] 
OS:                FreeBSD
Installed from:    FreeBSD Ports

genkdmconf does default to the correct values however, unless the command is explicitly set in kdmrc, the shutdown command defaults to the Linux onces (which are incorrect for a BSD system).  

This results as <sys/param.h> is not included before <config.ci>, as a result BSD is not defined (except in the case of genkdmconf.c).  

The following patch makes sure <sys/param.h> is included so BSD is defined (if available):

--- kdm/config.def
+++ kdm/config.def
@@ -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"
Comment 1 Oswald Buddenhagen 2010-02-21 12:00:25 UTC
SVN commit 1093649 by ossi:

make config.ci self-contained

BUG: 214963

 M  +2 -0      config.def  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1093649
Comment 2 Raphael Kubo da Costa 2010-02-21 16:39:08 UTC
SVN commit 1093855 by rkcosta:

Backport r1093649 by ossi

make config.ci self-contained

CCBUG: 214963


 M  +2 -0      config.def  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1093855