[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: kdelibs/kinit
From:       Waldo Bastian <bastian () kde ! org>
Date:       2004-09-26 15:19:03
Message-ID: 200409261719.07715.bastian () kde ! org
[Download RAW message or body]


I was wondering if, when changing the process name works, we should set the 
proc title to something like
	konqueror [kdeinit] http://ww.kde.org
or
	konqueror[kdeinit] http://ww.kde.org
instead of
	kdeinit: konqueror http://ww.kde.org

What do you think?

Cheers,
Waldo

On Sunday 26 September 2004 03:09, Dirk Mueller wrote:
> CVS commit by mueller:
>
> set the process name on linux, so that killall works correctly and you
> don't need ugly hacks like kdekillall.
>
> I'd like to backport this.. any objections?
>
>
>   M +12 -1     kinit.cpp   1.143
>
>
> --- kdelibs/kinit/kinit.cpp  #1.142:1.143
> @@ -60,4 +60,11 @@
>  #include <klocale.h>
>
> +#ifdef Q_OS_LINUX
> +#include <sys/prctl.h>
> +#ifndef PR_SET_NAME
> +#define PR_SET_NAME 15
> +#endif
> +#endif
> +
>  #if defined Q_WS_X11 && ! defined K_WS_QTONLY
>  #include <kstartupinfo.h> // schroder
> @@ -508,4 +515,8 @@ static pid_t launch(int argc, const char
>         /** Give the process a new name **/
>         kdeinit_setproctitle( "%s", procTitle.data() );
> +#ifdef Q_OS_LINUX
> +       /* set the process name, so that killall works like intended */
> +       prctl(PR_SET_NAME, (unsigned long) name.data(), 0, 0, 0);
> +#endif
>       }

-- 
bastian@kde.org  |  Wanted: Talented KDE developer  |  bastian@suse.com
  http://www.suse.de/de/company/suse/jobs/suse_pbu/developer_kde.html

[Attachment #3 (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]