Bug 154708

Summary: Build and functionality fixes for ksysguardd on NetBSD
Product: [Applications] ksysguard Reporter: Mark Davies <mark>
Component: ksysguarddAssignee: KSysGuard Developers <ksysguard-bugs>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: NetBSD pkgsrc   
OS: NetBSD   
Latest Commit: Version Fixed In:
Attachments: NetBSD patches for RC2
NetBSD patches for 3.5 branch

Description Mark Davies 2007-12-27 21:16:31 UTC 
Version:            (using KDE KDE 3.97.0)
Installed from:    NetBSD pkgsrc
Compiler:          gcc4 
OS:                NetBSD

ksysguardd for NetBSD doesn't build in 4.0RC2 and doesn't build for some versions of NetBSD.  Also some of the Process/Memory/CPU values are not calculated correctly or appropriate for NetBSD.

I will attach patches that fix these issues for RC2 and also for the 3.5 branch.

cheers
mark
Comment 1 Mark Davies 2007-12-27 21:18:09 UTC 
Created attachment 22708 [details]
NetBSD patches for RC2
Comment 2 Mark Davies 2007-12-27 21:19:07 UTC 
Created attachment 22709 [details]
NetBSD patches for 3.5 branch
Comment 3 John Tapsell 2007-12-27 21:29:35 UTC 
Mark,
  Thank you!

   Can you put these under LGPLv2 or later please?  It makes life a
bit easier :)
Comment 4 John Tapsell 2007-12-27 21:38:13 UTC 
Presumably the bits of code like:

+#if defined(__DragonFly__)
and
+#if defined(__FreeBSD__) && __FreeBSD_version >= 500015

etc can be removed?
Comment 5 Mark Davies 2007-12-28 05:26:19 UTC 
> Presumably the bits of code like: 
 
> +#if defined(__DragonFly__) 
> and 
> +#if defined(__FreeBSD__) && __FreeBSD_version >= 500015 
 
> etc can be removed? 
 
If you mean in processes_netbsd_p.cpp then yes.  That file was just copied from processes_freebsd_p.cpp and then very slightly tweaked to build.  I haven't actually looked at its contents too closely so it may well need some additional work to give reasonable values (beyond just building) but I was conscious of the approaching release -- whereas the rest of the patches have a longer history, deriving from the 3.5 branch versions.
Comment 6 Mark Davies 2007-12-28 05:29:11 UTC 
>   Can you put these under LGPLv2 or later please?  It makes life a 
> bit easier :) 
 
They're just a few patches to existing files, but if it makes things easier then sure, what do I need to do?
Comment 7 John Tapsell 2007-12-28 10:35:56 UTC 
> They're just a few patches to existing files, but if it makes things easier then sure,

Excellent.
> what do I need to do?

You've already done it :-)
Comment 8 John Tapsell 2007-12-28 10:42:41 UTC 
SVN commit 753714 by johnflux:

Support for NetBSD, thanks to Mark Davies.

BUG:154708


 M  +6 -1      processes_local_p.h  
 A             processes_netbsd_p.cpp   [License: LGPL (v2+)]


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