search

gazebosim / gz-transport

Transport library for component communication based on publication/subscription and service calls.
https://gazebosim.org
Apache License 2.0
29 stars 43 forks source link

No copyright header for a couple of tools. #52

Closed osrf-migration closed 8 years ago

osrf-migration commented 8 years ago

Original report (archived issue) by Rich Mattes (Bitbucket: richmattes).

We're working on including ignition-transport-1.2.0 in Fedora, and there were only two minor issues that came out of the package review. One of them is that licensecheck turned up a couple of scripts in the tools directory that are missing copyright info:

$ licensecheck -r * |grep UNKNOWN
Changelog.md: *No copyright* UNKNOWN
doc/doxygen.css: *No copyright* UNKNOWN
doc/style.css: *No copyright* UNKNOWN
INSTALL_WINDOWS.md: *No copyright* UNKNOWN
README.md: *No copyright* UNKNOWN
tools/cpplint_to_cppcheckxml.py: *No copyright* UNKNOWN <----------
tools/code_check.sh: *No copyright* UNKNOWN             <----------

It's not a big deal but it wouldn't hurt to include copyright info in them.

osrf-migration commented 8 years ago

Original comment by Carlos Agüero (Bitbucket: caguero, GitHub: caguero).

@rmattes, thank for reporting it.

See pull request #161.

osrf-migration commented 8 years ago

Original comment by Carlos Agüero (Bitbucket: caguero, GitHub: caguero).

Merged in issue_52 (pull request #161)

Fix issue #52

→ \<\<cset a6aa6dbeb8a910695cd38024293150d5a7421631>>