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.

Sign up for GitHub

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

Jump to bottom

Missing install targets #10

Closed
gavanderhoorn opened this issue Aug 24, 2013 · 5 comments
Closed

Missing install targets #10

gavanderhoorn opened this issue Aug 24, 2013 · 5 comments

Comments

@gavanderhoorn
Copy link
Member

The CMakeLists.txt of some packages in the groovy-dev branch are missing install targets. This makes it impossible to use them in a install space.

Packages:

  • ur_bringup
  • ur_description
  • ur_driver
  • ur_gazebo
  • ur_kinematics
@ipr-sr2
Copy link
Contributor

ipr-sr2 commented Sep 13, 2013

I tiried this, but even without any changes I can find all targets (launch and run files).

@gavanderhoorn
Copy link
Member Author

In your install space? I'd be rather surprised, as there are no install statements in any of the CMakeLists.txt in any of the above mentioned projects in the groovy-dev branch.

@ipr-sr2
Copy link
Contributor

ipr-sr2 commented Sep 13, 2013

they are not in install space, but I meant I can reach them with roslaunch/rosrun commands.

@gavanderhoorn
Copy link
Member Author

Sure, but the ticket is about missing install targets, which would result in the files being copied to the install space ..

@ipr-sr2 ipr-sr2 mentioned this issue Sep 13, 2013
Closed
shaun-edwards pushed a commit that referenced this issue Oct 1, 2013
@ipr-sr2 @shaun-edwards
Added definitions for adding tergets in install folder. Issue #10.
56cf07f
shaun-edwards added a commit that referenced this issue Oct 1, 2013
@shaun-edwards
Merge pull request #19 from ros-industrial/hydro-devel
eaeb0d0 
Added definitions for adding tergets in install folder. Issue #10.
@shaun-edwards
Copy link
Member

Merged fix, closing.

ipa-nhg pushed a commit to ipa-nhg/universal_robot that referenced this issue Jul 2, 2019
@ThomasTimm
Merge pull request ros-industrial#10 from jeppewalther/ur3_support
b0a896b 
Ur3 support
gtg4059 pushed a commit to gtg4059/universal_robot that referenced this issue Aug 2, 2021
@fmauch
Merge pull request ros-industrial#10 from fmauch/calibration_devel-st…
00447a5 
…aging

Calibration devel staging
cjue pushed a commit to isys-vision/universal_robot that referenced this issue Jun 22, 2022
@Briancbn @cjue
Fix SetPayload srv separater invalid (ros-industrial#10)
eb74dce 
Signed-off-by: Chen Bainian <chenbn@artc.a-star.edu.sg>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@shaun-edwards @gavanderhoorn @ipr-sr2