[libvirt] [PATCH] dnsmasqReload: avoid mingw link failure

[Eric Blake]

On 05/03/2010 12:43 PM, Matthias Bolte wrote:
>> -dnsmasqReload(pid_t pid)
>> +dnsmasqReload(pid_t pid ATTRIBUTE_UNUSED)
>>  {
>> +#ifndef __MINGW32__
> 
> Typically, WIN32 instead of __MINGW32__ is used in the libvirt code
> base. Cygwin doesn't define WIN32, so this won't interfere with a
> Cygwin build.

for exp in MINGW32 WIN32; do
  git grep $exp -- tools lib daemon | wc
done

shows 4 of the former, and 13 of the latter.  Patch updated accordingly,
and I'm working on a followup to make it consistent.

> 
>>     if (kill(pid, SIGHUP) != 0) {
>>         virReportSystemError(errno,
>>             _("Failed to make dnsmasq (PID: %d) reload config files.\n"),
> 
> You could kill the \n here.

Sure, as long as I'm in the area.

> 
> ACK

Thanks, and applied with your suggested updates.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100503/7506e022/attachment-0001.sig>