The function rcutils_join_path currently returns a const char *, but this doesn't make too much sense because it returns malloc'ed memory. Also, this causes every user to have to cast away the const when freeing the memory. This is a tracking bug for changing rcutils_join_path to return a char * instead. Note that all of these PRs have to be merged at the same time to not break the build.