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

Recommended copyright notice for control_msgs #120

Open
evan-flynn-apexai opened this issue Mar 18, 2024 · 9 comments
Open

Recommended copyright notice for control_msgs #120

evan-flynn-apexai opened this issue Mar 18, 2024 · 9 comments

Comments

@evan-flynn-apexai
Copy link

Would it be possible to add / recommend a copyright notice for this package?

Usually most packages in ROS 2 utilize ament_copyright to automate this chore a bit, but you dont necessarily have to use it if you dont want.

@christophfroehlich
Copy link
Contributor

What is missing in your opinion? ament_copyright only complains about a missing CONTRIBUTING.md file

@evan-flynn-apexai
Copy link
Author

@christophfroehlich so one of the clauses of the license is to "retain the above copyright notice"....so usually with BSD-3 clause license files there is a copyright notice at the top stating who owns the copyrights to the software.

Usually it looks something like Copyright 2024 Christoph Froehlich or something similiar

@evan-flynn-apexai
Copy link
Author

My bad about recommending ament_copyright...I thought it would catch it and fix it automatically for you 😅 but i guess it doesnt

@christophfroehlich
Copy link
Contributor

but this repo only hosts message definitions, and I never saw copyright notices in *.msg files?!

@evan-flynn-apexai
Copy link
Author

but this repo only hosts message definitions, and I never saw copyright notices in *.msg files?!

@christophfroehlich here I am not talking about the .msg files, I am talking about the LICENSE file itself. The BSD-3 license expects / requires there to be a copyright notice at the top.

@christophfroehlich
Copy link
Contributor

christophfroehlich commented Mar 22, 2024

Where do you have this information from?
ament_copyright expects exactly that file:
https://github.com/ament/ament_lint/blob/rolling/ament_copyright/ament_copyright/template/bsd_3clause_license.txt

@evan-flynn-apexai
Copy link
Author

@christophfroehlich it is part of the licnese text of BSD 3. Ill create an issue in ament_copyright though about this topic, it should be updated there as well.

@evan-flynn-apexai
Copy link
Author

spun out the issue in ament_copyright: ament/ament_lint#483

@christophfroehlich in the mean time: can you just advise us on what should the copyright notice be for this package?

@christophfroehlich
Copy link
Contributor

Recently we used something like Copyright 2024 ros2_control Development Team

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

2 participants