Re: [PATCH] daemon: Fix compilation error of dispatch.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jul 8, 2010 at 6:26 PM, Daniel P. Berrange <berrange@xxxxxxxxxx> wrote:
> On Thu, Jul 08, 2010 at 03:53:06AM +0900, Ryota Ozaki wrote:
>> On Thu, Jul 8, 2010 at 3:31 AM, Ryota Ozaki <ozaki.ryota@xxxxxxxxx> wrote:
>> > On Thu, Jul 8, 2010 at 2:40 AM, Daniel Veillard <veillard@xxxxxxxxxx> wrote:
>> >> On Thu, Jul 08, 2010 at 01:55:23AM +0900, Ryota Ozaki wrote:
>> >>> > Unfortunately the errors still happen after updating. Even worse, they happen
>> >>> > on an old commit that I was able to compile successfully so far. So I guess
>> >>> > my box went wrong at some point.
>> >>>
>> >>> FYI. My box has repaired by removing /usr/local/include/rpc/ which is
>> >>> installed by
>> >>> portablexdr to build MinGW cross-compile environment. I don´t know why/how the
>> >>> headers affect to libvirt compilation.
>> >>
>> >>  Huh, that's weird, maybe the compiler takes /usr/local/include/blah...
>> >> before /usr/include/blah...
>> >
>> > My box's gcc seems trying to search /usr/local/include firstly...
>> > I'm not sure this is expected behavior or not.
>>
>> Anyway, dispatch.c requires stdio.h but now it implicitly depends on that
>> rpc/xdr.h includes stdio.h.
>>
>> [Inclusions]
>> dispatch.c
>>   libvirtd.h
>>     rpc/xdr.h
>>       stdio.h
>>
>> I think including stdio.h explicitly in dispatch.c is not bad idea.
>> How do you think, Daniel?
>
> Yes, we shouldn't rely on rpc/xdr.h to include it

OK, Thanks. I'll soon resend the patch with updating the comment.

  ozaki-r

>
> Daniel
> --
> |: Red Hat, Engineering, London    -o-   http://people.redhat.com/berrange/ :|
> |: http://libvirt.org -o- http://virt-manager.org -o- http://deltacloud.org :|
> |: http://autobuild.org        -o-         http://search.cpan.org/~danberr/ :|
> |: GnuPG: 7D3B9505  -o-   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|
>

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]