-
Notifications
You must be signed in to change notification settings - Fork 308
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
license in transmission_interface #326
Comments
Thanks for the heads up, you are right! Could you please parse the rest of the packages, also the ones at https://github.com/ros-controls/ros_controllers/ ? I'd like to resolve all at once. |
There were only two package.xml files in each repository. I have created PRs fixing those two. HTH |
Thank you! |
Thank you for the quick response! |
Could you please backport these fixes to kinetic? |
I'm sorry to still commenting on this, but in kinetic-devel the issue still exists. |
Sorted!
…On 20 April 2018 at 13:27, Patrick Holthaus ***@***.***> wrote:
I'm sorry to still commenting on this, but in kinetic-devel the issue
still exists.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#326 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ADXH4cgeO11l3jEnnwOwV5d9U2TnSOP3ks5tqdRPgaJpZM4TZ4b8>
.
|
Perfect, thank you! |
In package.xml of transmission_interface, the license is specified as "modified bsd" while that seems to be the standard 3-clause BSD license as in the other packages in this repository as well. This can lead to problems when automatically parsing the file, such as
ros/ros-overlay#567
Could you please unify the specification in package.xml to match the LICENSE file? Thank you
The text was updated successfully, but these errors were encountered: