• Linux
• Linux-aarch64
• macOS
• Windows

@EduPonz It looks like rmw_fastrtps_shared_cpp is failing to build.

@jacobperron This should have been fixed on eProsima/Fast-DDS@30c143f

@MiguelCompany Thanks. Looks like there's still an issue with the Windows build:

C:\ci\ws\install\include\fastdds/rtps/builtin/discovery/endpoint/EDP.h(32,10): fatal error C1083: Cannot open include file: 'foonathan/memory/container.hpp': No such file or directory

@jacobperron The only difference I've seen is that --merge-install is only used on Windows.

I've been investigating a bit, and checked that adding foonathan_memory here also solves the problem. I don't know if that would be the best solution, though, as the error comes from including a header that is not necessary. That's why I have created ros2/rmw_fastrtps#487 instead.

Is my reasoning correct?

Certainly ros2/rmw_fastrtps#487 makes sense.

Considering rmw_fastrtps does not directly depend on foonathan_memory, we should have to explicitly link against foonathan_memory. Sounds like maybe fastdds is not exporting all the necessary info about it's dependency on foonathan_memory, and doing a merge install revealed this. This is just all speculation though, I haven't dug into it.

@jacobperron Now that ros2/rmw_fastrtps#487 has been merged, could you relaunch CI on this one?

• Windows (edit: retriggered with fast-dds only)

Hi @jacobperron @clalancette ,

Is their any blocker for this? Anything we can do on our side?

Let me run CI on the whole thing again and see what it looks like:

• Linux
• Linux-aarch64
• macOS
• Windows

All right, the failures on Windows are well-known. So I'm going to go ahead and merge this in.

Awesome!