Re: [msysGit] [PATCH] compat/mingw: stubs for getpgid() and tcgetpgrp()

2015-04-24 Thread Johannes Schindelin
Hi Rupert,

On 2015-04-23 21:25, rupert thurner wrote:
 On Thursday, April 16, 2015 at 2:45:11 PM UTC+2, Johannes Schindelin wrote:

 However, using this code for `getppid()` would be serious overkill (not to
 mention an unbearable performance hit because you have to enumerate *all*
 processes to get that information).


 is the windows JobObject similar to processgroup? at least killing the 
 parent process in a jobobject will kill all childs as well:
 https://msdn.microsoft.com/en-us/library/windows/desktop/ms681949%28v=vs.85%29.aspx

Reading this page carefully reveals that you have to construct JobObjects 
explicitly. So you cannot get from a process ID to a JobObject; there is 
probably none. Or there is one. Or many.

Ciao,
Johannes
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [msysGit] [PATCH] compat/mingw: stubs for getpgid() and tcgetpgrp()

2015-04-16 Thread Johannes Schindelin
Hi kusma,

On 2015-04-15 21:43, Erik Faye-Lund wrote:
 On Wed, Apr 15, 2015 at 8:29 PM, Johannes Sixt j...@kdbg.org wrote:
 Windows does not have process groups. It is, therefore, the simplest
 to pretend that each process is in its own process group.
 
 Windows does have some concept of process groups, but probably not
 quite what you want:
 
 https://msdn.microsoft.com/en-us/library/windows/desktop/ms682083%28v=vs.85%29.aspx

Yes, and we actually need that in Git for Windows anyway because shooting down 
a process does not kill its child processes:

https://github.com/git-for-windows/msys2-runtime/commit/15f209511985092588b171703e5046eba937b47b#diff-8753cda163376cee6c80aab11eb8701fR402

However, using this code for `getppid()` would be serious overkill (not to 
mention an unbearable performance hit because you have to enumerate *all* 
processes to get that information).

Ciao,
Dscho
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [msysGit] [PATCH] compat/mingw: stubs for getpgid() and tcgetpgrp()

2015-04-15 Thread Erik Faye-Lund
On Wed, Apr 15, 2015 at 8:29 PM, Johannes Sixt j...@kdbg.org wrote:
 Windows does not have process groups. It is, therefore, the simplest
 to pretend that each process is in its own process group.

Windows does have some concept of process groups, but probably not
quite what you want:

https://msdn.microsoft.com/en-us/library/windows/desktop/ms682083%28v=vs.85%29.aspx
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html