ros2/rmw_connextdds

should return if CONNEXTDDS_ARCH is not set after guessing

Opened this issue · 1 comments

should be:

        if(NOT CONNEXTDDS_ARCH)
          rti_guess_connextdds_arch()
          if(NOT CONNEXTDDS_ARCH)
            return()
          endif()
        endif()

Hi @asobhy-qnx,

Thank you for your suggestion! The change makes sense to me, since at the moment the build will "fall through" without a CONNEXTDDS_ARCH value, and fail more subtly at a later point.

I'll try to update the build files soon, but feel free to open a PR on the repository if you want/have the bandwidth. Community contributions are always welcome 😄