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

No launch directory when installed with apt #19

Open
rkoyama1623-2021 opened this issue Sep 7, 2020 · 4 comments
Open

No launch directory when installed with apt #19

rkoyama1623-2021 opened this issue Sep 7, 2020 · 4 comments

Comments

@rkoyama1623-2021
Copy link

はじめまして、本パッケージでは、いつもお世話になっております。なお日本語で失礼します。
本パッケージをaptコマンドでインストールを行うと、
Ubuntu18の場合、timed_roslaunch.launchrospack find timed_roslauch直下に配置されますが、
Ubuntu16の場合、$(rospack find timed_roslauch)/launch以下に配置されます。

# ubuntu 18.04
sudo apt install ros-melodic-timed-roslaunch
ls $(rospack find timed_roslaunch)/timed_roslaunch.launch

# ubuntu 16.04
sudo apt install ros-kinetic-timed-roslaunch
ls $(rospack find timed_roslaunch)/launch/timed_roslaunch.launch

Ubuntu18では、 c335736 が反映されていないのではないかと推測しますが、今後反映される予定はあるでしょうか?
どうぞよろしくお願いいたします。

@sebastienmascha
Copy link

I confirm for ros-melodic.
Here is the content inside the package installed with apt: cmake package.xml timed_roslaunch.launch timed_roslaunch_test.launch
It works well by installing from the source.

@rkoyama1623-2021 rkoyama1623-2021 changed the title No launch file when installed with apt No launch directory when installed with apt Oct 1, 2020
@rkoyama1623-2021
Copy link
Author

@sebastienmascha, Thank you for your comment.
I meant the roslaunch directory (You are right, so I changed the issue title).
When we use this package across the different ros environment, current directory structure is inconvenient.
c335736 should generate roslaunch directory and put launch file in it, even if we use this package in melodic.
So, I want the author to update the apt package if possible.

@ajithcodesit
Copy link

This is definitely required, the apt needs to be updated to include the launch directory.

@rkoyama1623-2021
Copy link
Author

Temporary solution:

  <arg name="timed_roslaunch_dir" value="launch/" if="$(eval env('ROS_DISTRO')=='kinetic')"/>
  <arg name="timed_roslaunch_dir" value="" unless="$(eval env('ROS_DISTRO')=='kinetic')"/>
  <include file="$(find timed_roslaunch)/$(arg timed_roslaunch_dir)timed_roslaunch.launch">
    <arg name="time" value="5"/>
    <arg name="pkg" value="some_package"/>
    <arg name="file" value="something.launch"/>
  </include>

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

No branches or pull requests

3 participants