This pull request (https://github.com/ceph/ceph/pull/812) reverts the
patch that changed the struct initialization. That is C99 style, but
apparently it isn't common in compilers, and GNU has just added it as
an extension.

I don't have a better solution at the moment, but as Greg mentioned,
perhaps detecting C++11 is an option, but that means a big nasty
ifdef. Another solution might be to just put the struct initialization
in a C file.

On Sun, Nov 3, 2013 at 2:33 PM, Xing Lin <[email protected]> wrote:
> Thanks, Noah!
>
> Xing
>
> On 11/3/2013 3:17 PM, Noah Watkins wrote:
>>
>> Thanks for looking at this. Unless there is a good solution I think
>> reverting it is ok as breaking the compile on a few platflorms is not ok.
>> Ill be lookong at this tonight.
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to [email protected]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to