[Libvirt-cim] [PATCH] Fix include of internal headers

Heidi Eckhart heidieck at linux.vnet.ibm.com
Thu Jan 17 08:41:07 UTC 2008


Jay Gagnon wrote:
> Kaitlin Rupert wrote:
>   
>> Dan Smith wrote:
>>     
>>> # HG changeset patch
>>> # User Dan Smith <danms at us.ibm.com>
>>> # Date 1200498221 28800
>>> # Node ID fd28a485f90399d166eb62816c32e2d7727cced6
>>> # Parent  ef0dd32ef7faf101f97acfa0f0268bf18c22dcac
>>> Fix include of internal headers
>>>
>>> This was causing me problems when installing libcmpiutil into /usr/local
>>> for testing.  I think this is the right solution, and seems to work
>>> for me
>>> in both cases.
>>>
>>> Signed-off-by: Dan Smith <danms at us.ibm.com>
>>>
>>> diff -r ef0dd32ef7fa -r fd28a485f903 std_indication.h
>>> --- a/std_indication.h    Wed Jan 16 06:33:30 2008 -0800
>>> +++ b/std_indication.h    Wed Jan 16 07:43:41 2008 -0800
>>> @@ -25,8 +25,8 @@
>>>  #include <cmpift.h>
>>>  #include <cmpimacs.h>
>>>
>>> -#include <libcmpiutil.h>
>>> -#include <std_invokemethod.h>
>>> +#include "libcmpiutil.h"
>>> +#include "std_invokemethod.h"
>>>
>>>  CMPIStatus stdi_trigger_indication(const CMPIBroker *broker,
>>>                                     const CMPIContext *context,
>>>       
>> Agreed. I think a project shouldn't need to use <> for its own header
>> files (in most cases anyway).  +1
>>
>>     
> Yea, I'm on board with that.
>
>   
Me too. +1

-- 
Regards

Heidi Eckhart
Software Engineer
IBM Linux Technology Center - Open Hypervisor




More information about the Libvirt-cim mailing list