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 <xinglin@xxxxxxxxxxx> 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 majordomo@xxxxxxxxxxxxxxx > 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
