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

sound_play build error #1

Open
gakutasu opened this issue Dec 26, 2024 · 7 comments
Open

sound_play build error #1

gakutasu opened this issue Dec 26, 2024 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@gakutasu
Copy link

gakutasu commented Dec 26, 2024

概要

sound_playパッケージをビルドする際に、ament_python_install_packageがターゲット競合を引き起こし、ビルドが失敗する。

Starting >>> sound_play
--- stderr: sound_play                         
CMake Error at /opt/ros/jazzy/share/ament_cmake_python/cmake/ament_python_install_package.cmake:106 (add_custom_target):
  add_custom_target cannot create target
  "ament_cmake_python_symlink_sound_play" because another target with the
  same name already exists.  The existing target is a custom target created
  in source directory "/home/gakutasu/soar/src/audio_common/sound_play".  See
  documentation for policy CMP0002 for more details.
Call Stack (most recent call first):
  /opt/ros/jazzy/share/ament_cmake_python/cmake/ament_python_install_package.cmake:39 (_ament_cmake_python_install_package)
  CMakeLists.txt:37 (ament_python_install_package)


CMake Error at /opt/ros/jazzy/share/ament_cmake_python/cmake/ament_python_install_package.cmake:144 (add_custom_target):
  add_custom_target cannot create target
  "ament_cmake_python_build_sound_play_egg" because another target with the
  same name already exists.  The existing target is a custom target created
  in source directory "/home/gakutasu/soar/src/audio_common/sound_play".  See
  documentation for policy CMP0002 for more details.
Call Stack (most recent call first):
  /opt/ros/jazzy/share/ament_cmake_python/cmake/ament_python_install_package.cmake:39 (_ament_cmake_python_install_package)
  CMakeLists.txt:37 (ament_python_install_package)


---
Failed   <<< sound_play [1.84s, exited with code 1]

再現手順
omit

修正しないとどう困るか

原因
既存ターゲット名と競合

修正案
ament_python_install_package(${PROJECT_NAME})を削除。
install()があるので問題ない?

@gakutasu gakutasu added the bug Something isn't working label Dec 26, 2024
@gakutasu gakutasu changed the title sound_playbirudoera- sound_play build error Dec 26, 2024
@gakutasu gakutasu mentioned this issue Dec 26, 2024
4 tasks
@gakutasu gakutasu self-assigned this Dec 26, 2024
@gakutasu
Copy link
Author

@Tacha-S @jsupratman13
修正の方法についてご提案いただければ幸いです。

@gakutasu
Copy link
Author

ros2/rosidl_python#141 に関連?

@Tacha-S
Copy link

Tacha-S commented Dec 26, 2024

私は使ってないのでCOLCON_IGNORE置いていますが、
ros2/rosidl_python#141 が解決してリリースされるまでsbgisenのリポジトリ使いたいとかなら #2 のようにしてもいいです。

@gakutasu
Copy link
Author

gakutasu commented Dec 26, 2024

私も使う予定はないのでCOLCON_IGNOREをおいています。
COLCON_IGNOREを置くだけでも問題はないのですが、
soarからvcsで入ってくるものでビルドが通らないものがあるというのも少々使いづらいかなと思っています。

.reposからaudio_commonごと消すか、sound_play以外を使っているならsound_playにCOLCON_IGNOREをおいたブランチを指定するかしたいのですが、いかがでしょうか。

それか #2 のPRを取り込むかですが、今後対応されそうな気配が少なからずあるので現時点での応急処置としては微妙かなと思っています。

@Tacha-S
Copy link

Tacha-S commented Dec 26, 2024

.reposからaudio_commonごと消す

audio_captureがリリースされていないので消せません。

結局ros2/rosidl_python#141 が解決するまでのその場しのぎなので

sound_play以外を使っているならsound_playにCOLCON_IGNOREをおいたブランチを指定する

これでも

#2 のPRを取り込む

これでもいいです。

@gakutasu
Copy link
Author

わかりました。では、

sound_play以外を使っているならsound_playにCOLCON_IGNOREをおいたブランチを指定する

こちらで対応いたします。ご意見いただきありがとうございます。

@Tacha-S
Copy link

Tacha-S commented Dec 26, 2024

sound_play以外を使っているならsound_playにCOLCON_IGNOREをおいたブランチを指定する

これにするにしてもCOLCON_IGNOREをおくというPRをsbgisenのros2ブランチに投げてくれるとありがたいです。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants