Skip to content
This repository has been archived by the owner on Jul 26, 2024. It is now read-only.

Fixed writing urdf file to package folder #255

Open
wants to merge 1 commit into
base: foxy-devel
Choose a base branch
from

Conversation

progtologist
Copy link

Fixes

Writing to package share folder which when installed in /opt fails without super user permission

Description of the changes:

  • Removed generation of URDF file, instead the URDF is generated inline on the python file and fed to the nodes.

Required before submitting a Pull Request:

I tested changes on:

  • Windows
  • Linux
  • ROS1
  • ROS2

Tested locally on my system

@ghost
Copy link

ghost commented Sep 5, 2022

CLA assistant check
All CLA requirements met.

Copy link
Member

@ooeygui ooeygui left a comment

Choose a reason for hiding this comment

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

Nice! Thank you

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants