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

fix(autoware_auto_tf2): remove duplicated function in tf2_geometry_msg #886

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

zhiwango
Copy link

最近tf2_gemotryの更新の影響で、ビルドが通らない問題が発生している。
(autowarefoundation#5128)
自機環境でsudo apt upgradeをしてから、該当問題が発生することを確認できた。

既に、autowarefoundationの方が以下2つのPRを取り込んでいます。こちらでbeta/v0.11.0へcherry-pickする。
autowarefoundation#5127
autowarefoundation#5089

Copy link

@maxime-clem maxime-clem left a comment

Choose a reason for hiding this comment

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

Tested and it fixes the build issue.

@zhiwango
Copy link
Author

@tkimura4 @0x126
情報ありがとうございます。
古いhumble+v2.3.0の組み合わせでビルドする可能性あるので、実車での混乱を避けるために、autowarefoundation#5127 のみをcherry-pickします。

@TomohitoAndo TomohitoAndo self-requested a review September 28, 2023 08:29
Copy link

@TomohitoAndo TomohitoAndo left a comment

Choose a reason for hiding this comment

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

LGTM

@TomohitoAndo TomohitoAndo merged commit abe7800 into beta/v0.11.0 Sep 28, 2023
10 checks passed
@TomohitoAndo TomohitoAndo deleted the fix/tf2_update branch September 28, 2023 08:29
kazuki527 pushed a commit that referenced this pull request Oct 2, 2023
#886)

chore: add workaround for tf2 update (autowarefoundation#5127)

Co-authored-by: Daisuke Nishimatsu <[email protected]>
# Conflicts:
#	planning/behavior_path_planner/CMakeLists.txt
maxime-clem pushed a commit to maxime-clem/autoware.universe that referenced this pull request Oct 2, 2023
kazuki527 pushed a commit that referenced this pull request Oct 2, 2023
#886)

chore: add workaround for tf2 update (autowarefoundation#5127)

Co-authored-by: Daisuke Nishimatsu <[email protected]>
# Conflicts:
#	planning/behavior_path_planner/CMakeLists.txt
kazuki527 pushed a commit that referenced this pull request Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants