xtypes icon indicating copy to clipboard operation
xtypes copied to clipboard

Is it supposed to work with any os and compilers?

Open mtmarino opened this issue 3 years ago • 4 comments

hi, I downloaded this library and tried to compile in windows environment with LLVM Clang 11 compiler but I got some issue due to linux environment specific inclusions (e.g. execinfo.h). Is this library meant to work in every operating system or is it linux specific?

mtmarino avatar Apr 08 '21 13:04 mtmarino

Hello @mtmarino, Currently, this library has not yet been tested in other architecture rather than Linux. However, the dependencies are not too many and it is feasible that it is compatible with Windows. More specifically, that include can be bypassed by setting the -DXTYPES_EXCEPTIONS=ON flag during the CMake configuration phase. Could you test it that way? Thank you.

jamoralp avatar Apr 08 '21 13:04 jamoralp

I've tried to test setting the -DXTYPES_EXCEPTIONS=ON, i've had to move the inclusion of execinfo.h after if check, to make it work. Then I've changed popen and pclose with _popen and _pcloce in case of window and now I've issues like this

error: use of overloaded operator '[]' is ambiguous (with operand types 'eprosima::xtypes::DynamicData' and 'const char [4]') data["om1"] = 6.7

that seems the problem described here https://stackoverflow.com/questions/8914986/should-this-compile-overload-resolution-and-implicit-conversions.

I'm trying to make it work, I will keep you update.

mtmarino avatar Apr 08 '21 16:04 mtmarino

Oh, that sounds not so good. Maybe GCC is missing that one, that could explain that we have not seen this warning before.

Any contribution would be highly appreciated. I'm sure this library should not be so much difficult to port to Windows, but right now I am busy with other projects and cannot dive into this. Please keep me updated and I will search for a time slot to look into this as well. Thank you!

jamoralp avatar Apr 09 '21 08:04 jamoralp

https://github.com/eProsima/xtypes/pull/113 introduces support in all available Github runners.

MiguelBarro avatar Jan 26 '23 10:01 MiguelBarro