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

Copyright headers missing from files #86

Closed
richmattes opened this issue Jan 5, 2016 · 9 comments
Closed

Copyright headers missing from files #86

richmattes opened this issue Jan 5, 2016 · 9 comments

Comments

@richmattes
Copy link
Contributor

The following files from the fcl-0.3.2 release contain no copyright headers:

# licensecheck -r .  | grep UNKNOW
./src/math/sampling.cpp: *No copyright* UNKNOWN
./src/continuous_collision.cpp: *No copyright* UNKNOWN
./include/fcl/exception.h: *No copyright* UNKNOWN
./include/fcl/math/vec_nf.h: *No copyright* UNKNOWN
./include/fcl/math/sampling.h: *No copyright* UNKNOWN
./include/fcl/ccd/simplex.h: UNKNOWN
./include/fcl/ccd/support.h: UNKNOWN
./include/fcl/continuous_collision.h: *No copyright* UNKNOWN
./include/fcl/learning/classifier.h: *No copyright* UNKNOWN
./test/libsvm_classifier.h: *No copyright* UNKNOWN
./test/general_test.cpp: *No copyright* UNKNOWN
./test/libsvm/svm.cpp: *No copyright* UNKNOWN
./test/libsvm/svm.h: *No copyright* UNKNOWN
./test/test_fcl_utility.cpp: *No copyright* UNKNOWN
./test/test_fcl_simple.cpp: *No copyright* UNKNOWN

Please consider adding proper copyright headers to the aforementioned files.

@scpeters
Copy link
Contributor

The following files (added in bfc62b9) have BSD license headers; I'm not sure why licensecheck doesn't see them:

The following file as migrated from svn (I think) by @isucan in 43ba254 without a license header:

  • ./test/test_fcl_utility.cpp: No copyright UNKNOWN

The rest of the files were added without license headers by @panjia1983 in the following commits:

de8666f

  • ./src/math/sampling.cpp: No copyright UNKNOWN
  • ./include/fcl/exception.h: No copyright UNKNOWN
  • ./include/fcl/learning/classifier.h: No copyright UNKNOWN

928b175

  • ./src/continuous_collision.cpp: No copyright UNKNOWN
  • ./include/fcl/continuous_collision.h: No copyright UNKNOWN

b16d32c

  • ./include/fcl/math/vec_nf.h: No copyright UNKNOWN
  • ./include/fcl/math/sampling.h: No copyright UNKNOWN

89887e9

  • ./test/general_test.cpp: No copyright UNKNOWN
  • ./test/libsvm_classifier.h: No copyright UNKNOWN
  • ./test/test_fcl_simple.cpp: No copyright UNKNOWN

6330ad3

  • ./test/libsvm/svm.cpp: No copyright UNKNOWN
  • ./test/libsvm/svm.h: No copyright UNKNOWN

@sherm1
Copy link
Member

sherm1 commented Mar 24, 2016

Steve, I also have an earlier email from you about possible non-BSD code in FCL:

On Tue, Jun 16, 2015 at 11:48 PM, Steven Peters <scpeters@osrfoundation.org> wrote:
I was peeking at the FCL source code, and I think I remember Karen saying that there was
some non-open code from UNC in FCL. Grepping for UNC, I found comments in the following 
files that reference the CATCH library ( http://graphics.ewha.ac.kr/CATCH/ ), which requires 
a password to download. I don't know if there is directly copied code there or not.

https://github.com/flexible-collision-library/fcl/blob/master/include/fcl/ccd/interval.h#L36-L37
https://github.com/flexible-collision-library/fcl/blob/master/include/fcl/ccd/interval_matrix.h#L35-L36
https://github.com/flexible-collision-library/fcl/blob/master/include/fcl/ccd/interval_vector.h#L35-L36
https://github.com/flexible-collision-library/fcl/blob/master/include/fcl/ccd/taylor_matrix.h#L35-L36
https://github.com/flexible-collision-library/fcl/blob/master/include/fcl/ccd/taylor_model.h#L35-L36
https://github.com/flexible-collision-library/fcl/blob/master/include/fcl/ccd/taylor_vector.h#L35-L36

Following links on the CATCH website, I also found the SWIFT++ library at UNC. This would be something to check as well.
http://gamma.cs.unc.edu/SWIFT++/download.html

Steve

It would be good to resolve these as well as the ones you mentioned here -- @panjia1983, can you help us figure out what are the correct licenses for this code?

@scpeters
Copy link
Contributor

Good point @sherm1

Those header files we discussed in the email have a BSD license listed, but the comment made we want to double-check that it was a correct license.

@mamoll
Copy link
Member

mamoll commented Mar 24, 2016

I remember talking about this code with Stephane Redon at RSS2014. I think what happened is that some legacy code within UNC's Gamma lab made its way into FCL. Redon was a postdoc there, and so was Kim (now at EWHA). Jia was a graduate student there. Tracing the licensing / copyright to the “real” license / copyright holders is tricky.

@sherm1
Copy link
Member

sherm1 commented Mar 25, 2016

Tracing the licensing / copyright to the “real” license / copyright holders is tricky.

I emailed Jia to see if he can weigh in. It occurred to me that he might not be getting notifications.

@richmattes
Copy link
Contributor Author

Any updates?

@jslee02
Copy link
Member

jslee02 commented Jul 24, 2016

#149 is created to move forward.

@jslee02
Copy link
Member

jslee02 commented Aug 11, 2016

Could we close this issue?

@jslee02
Copy link
Member

jslee02 commented Sep 9, 2016

I believe this is addressed by #149, please reopen if it's not.

@jslee02 jslee02 closed this as completed Sep 9, 2016
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

5 participants