[libvirt] [PATCH 1/2] build: fix virpidfile on mingw
Stefan Berger
stefanb at linux.vnet.ibm.com
Wed Aug 17 19:42:50 UTC 2011
On 08/17/2011 02:09 PM, Eric Blake wrote:
>> We already skip the /proc check on non-Linux. So I say we just skip
>> the kill call on Win32. This stuff is only an extra sanity check
>> Nupon loading the pidfile, so I think returning an error is too
>> mean and we just skip the kill check
> Done, and pushing under the build-breaker rule.
>
> src/util/virpidfile.c | 7 +++++--
> 1 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/src/util/virpidfile.c b/src/util/virpidfile.c
> index e64b0b3..8206e1a 100644
> --- a/src/util/virpidfile.c
> +++ b/src/util/virpidfile.c
> @@ -200,15 +200,18 @@ int virPidFileReadPathIfAlive(const char *path,
> if (rc< 0)
> return rc;
>
> - /* Check that it's still alive */
> +#ifndef WIN32
> + /* Check that it's still alive. Safe to skip this sanity check on
> + * mingw, which lacks kill(). */
> if (kill(*pid, 0)< 0) {
virKillProcess (src/util/util.c) handles WIN32 as well.
Stefan
More information about the libvir-list
mailing list