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

sciurus17、sciurus17_toolsのROS 2対応 #145

Merged
merged 7 commits into from
Nov 9, 2023
Merged

Conversation

Kuwamai
Copy link
Contributor

@Kuwamai Kuwamai commented Nov 9, 2023

What does this implement/fix?

不要なファイルの削除とsciurus17、sciurus17_toolsのROS 2対応を行いました。

Does this close any currently open issues?

しません

How has this been tested?

パッケージがビルドできることを確認しました。

Any other comments?

なし

Checklists

@Kuwamai Kuwamai added the Type: Feature New Feature label Nov 9, 2023
@Kuwamai Kuwamai requested a review from ShotaAk November 9, 2023 08:44
@Kuwamai Kuwamai self-assigned this Nov 9, 2023
Copy link
Contributor

@ShotaAk ShotaAk left a comment

Choose a reason for hiding this comment

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

見ました。コメントの確認お願いします。


<license>Apache License 2.0</license>

<author email="[email protected]">Hiroyuki Nomura</author>
<author email="[email protected]">Shota Hirama</author>
<author email="[email protected]">Atsushi Kuwagata</author>
Copy link
Contributor

Choose a reason for hiding this comment

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

ここはrt-net.jpのアドレスにしてください。

<author email="[email protected]">Hiroyuki Nomura</author>
<author email="[email protected]">Shota Hirama</author>
<license>Apache License 2.0</license>
<author email="[email protected]">Atsushi Kuwagata</author>
Copy link
Contributor

Choose a reason for hiding this comment

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

こちらもアドレス修正お願いします。

Comment on lines 15 to 17
<url type="website">http://wiki.ros.org/sciurus17_ros</url>
<url type="bugtracker">https://github.com/rt-net/sciurus17_ros/issues</url>
<url type="repository">https://github.com/rt-net/sciurus17_ros</url>
Copy link
Contributor

Choose a reason for hiding this comment

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

sciurus17パッケージに記載しているので不要です

Suggested change
<url type="website">http://wiki.ros.org/sciurus17_ros</url>
<url type="bugtracker">https://github.com/rt-net/sciurus17_ros/issues</url>
<url type="repository">https://github.com/rt-net/sciurus17_ros</url>
<url type="website">http://wiki.ros.org/sciurus17_ros</url>
<url type="bugtracker">https://github.com/rt-net/sciurus17_ros/issues</url>
<url type="repository">https://github.com/rt-net/sciurus17_ros</url>

<name>sciurus17_tools</name>
<version>2.0.0</version>
<description>The sciurus17_tools package</description>

<description>ROS 2 package suite of Sciurus17</description>
Copy link
Contributor

Choose a reason for hiding this comment

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

ここはもとのdescriptionのままでokです。

RUNTIME DESTINATION ${CATKIN_PACKAGE_BIN_DESTINATION}
)

install(DIRECTORY scripts rules DESTINATION ${CATKIN_PACKAGE_SHARE_DESTINATION})
Copy link
Contributor

Choose a reason for hiding this comment

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

lib以下にインストールすると、ros2 run sciurus17_tools create_udev_rulesでコマンド実行できます。
変更お願いします。

install(DIRECTORY
  scripts
  rules
  DESTINATION lib/${PROJECT_NAME}/
)

Copy link
Contributor

@ShotaAk ShotaAk left a comment

Choose a reason for hiding this comment

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

LGTM

@ShotaAk ShotaAk merged commit 36ab12a into ros2 Nov 9, 2023
2 checks passed
@Kuwamai Kuwamai deleted the support_ros2_other_packages branch November 10, 2023 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants