Re: [Opensync-devel] Two brakes in libopensync

[Ian M. <ian...@ca...>]

>> > 2. the __sun statement in opensync.h causes frequent troubles:
>> > [ 1%] Building CXX object
>> > libqopensync/CMakeFiles/qopensync.dir/callbackhandler.o
>> > cc1plus: warnings being treated as errors In file included from
>> > /home/murz/temp/2010-09/sync/kitchensync/libqopensync/callbackhandler.cpp
>> > :22: /usr/include/libopensync1/opensync/opensync.h:52: error: "__sun" is
>> > not defined
>> >
>> >
>> > How can we fix this issue? I do it by removing the lines 52-54 in
>> > opensync.h,  but this is only my simple workaround.
>>
>> What compiler do you use?  Maybe very recent versions of gcc are more
>> picky about these kind of if/then/else macro structures.
> I am using gcc 4.5.0. This issue concerns many users out there and it is
> really not a clean and nice solution to tell them to edit the opensync.h file.
>
> What can we do about that? In the worst case scenario I would suggest to
> remove that macro.
>

I imagine kitchensync is being compiled with the -Wundef option

Revision 6150 contains the patch from ticket
http://www.opensync.org/ticket/1263 that fixes this problem

Do people think -Wundef be added to the Hacking build flags?

Ian