Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Humble (and format with Elixir 1.15.5) #241

Merged
merged 7 commits into from
Sep 6, 2023
Merged

Conversation

pojiro
Copy link
Contributor

@pojiro pojiro commented Sep 4, 2023

From Humble, C++ headers are installed in a subdirectory
https://docs.ros.org/en/rolling/Releases/Release-Humble-Hawksbill.html#c-headers-are-installed-in-a-subdirectory

To support it, we add each package's subdirectory by wildcard

@pojiro pojiro changed the title Support subdirectory headers which is changed from Humble Support Humble Sep 4, 2023
@pojiro pojiro marked this pull request as ready for review September 5, 2023 02:52
@pojiro pojiro mentioned this pull request Sep 6, 2023
@pojiro pojiro linked an issue Sep 6, 2023 that may be closed by this pull request
@pojiro
Copy link
Contributor Author

pojiro commented Sep 6, 2023

新たに選定していただいたターゲット 7 つでテストが通るよう修正しましたので、お手すきの時にレビューお願いいたします。

以下、ポイントです。

  • Makefile を humbleとそれより前で分けた 3cac5e0
  • Makefile の rmw_opensplice のコメント行を削除、サポートがなくなっていたため fd7ee3d
  • ROS 2 version default の DDS を使用するため、不要な lib リンクを削除 610e2ce

@pojiro
Copy link
Contributor Author

pojiro commented Sep 6, 2023

Makefile を humbleとそれより前で分けた

はやりすぎであったため、 Makefile 内で条件分岐するように修正しました。
将来的に foxy, galactic のサポートを落とす際に、条件分岐を削除できます。

Copy link
Member

@takasehideki takasehideki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome thanks for wonderful contributions!!
this will fix to #114 #243

@takasehideki takasehideki changed the title Support Humble Support Humble (and format with Elixir 1.15.5) Sep 6, 2023
@takasehideki takasehideki merged commit e9b8c20 into main Sep 6, 2023
11 checks passed
@takasehideki takasehideki deleted the support-humble branch September 6, 2023 03:12
takasehideki added a commit that referenced this pull request Sep 11, 2023
**Full Changelog**: v0.8.5...v0.9.0

* New features:
  * Support Humble (and format with Elixir 1.15.5) by @pojiro in #241
  * Support humble, galactic Nerves by @pojiro in #244
  * change the recommended environment and versions by @takasehideki in #247
  * Add humble support to arm32v7_ros_distros by @pojiro in #248
* Code Improvements/Fixes:
  * Fix bug of `mix rclex.gen.msgs` by @pojiro in #236
  * fix docs about ROS_DISTRO from `foxy` to `humble` by @takasehideki in #249
* Bumps:
  * Bump ex_doc from 0.29.4 to 0.30.6 by @dependabot in #234 #237 #238
  * Bump dialyxir from 1.3.0 to 1.4.1 by @dependabot in #240
  * Bump mix_test_watch from 1.1.0 to 1.1.1 by @dependabot in #242
* Known issues to be addressed in the near future:
  * Lock `git_hooks` to 0.6.5 due to its issue in #138
  * Release rcl nif resources when GerServer terminates in #160
  * `publish/2` sometimes failed just after `create_publisher/3` in #212
  * CI fails randomly at mix test in #246
  * Bump to Iron Irwini in #228
* Note in this release:
  * Please welcome Humble Hawksbill as the new supported distribution for Rclex!! 🎉
@takasehideki takasehideki mentioned this pull request Sep 11, 2023
takasehideki added a commit that referenced this pull request Sep 11, 2023
**Full Changelog**: v0.8.5...v0.9.0

* New features:
  * Support Humble (and format with Elixir 1.15.5) by @pojiro in #241
  * Support humble, galactic Nerves by @pojiro in #244
  * change the recommended environment and versions by @takasehideki in #247
  * Add humble support to arm32v7_ros_distros by @pojiro in #248
* Code Improvements/Fixes:
  * Fix bug of `mix rclex.gen.msgs` by @pojiro in #236
  * fix docs about ROS_DISTRO from `foxy` to `humble` by @takasehideki in #249
* Bumps:
  * Bump ex_doc from 0.29.4 to 0.30.6 by @dependabot in #234 #237 #238
  * Bump dialyxir from 1.3.0 to 1.4.1 by @dependabot in #240
  * Bump mix_test_watch from 1.1.0 to 1.1.1 by @dependabot in #242
* Known issues to be addressed in the near future:
  * Lock `git_hooks` to 0.6.5 due to its issue in #138
  * Release rcl nif resources when GerServer terminates in #160
  * `publish/2` sometimes failed just after `create_publisher/3` in #212
  * CI fails randomly at mix test in #246
  * Bump to Iron Irwini in #228
* Note in this release:
  * Please welcome Humble Hawksbill (and Galactic Geochelone) as the new supported distribution for Rclex!! 🎉
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support galactic memo
2 participants