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

Clarify Licensing Information #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kphillisjr
Copy link

Clarify the license information of SourceCodeKit. This is in response to issue #9 and should have all current committers information added. If I missed anyone please add the appropriate information to the COPYING file and the appropriate source file(s).

This is the license information discussed in issue etoile#9 on github[1]

[1] etoile#9
This is the license header discussed in Issue etoile#9 on github[1]

[1] etoile#9
@qmathe
Copy link
Member

qmathe commented Sep 2, 2016

Thanks a lot for clarifying the license and copyrights!

To keep the license notice compact, we could collapse all the copyright holders on a single line, and use the file creation date as the only date appearing in the license notice.

So the following:

Copyright (c) 2010-2012 David Chisnall
Copyright (c) 2012 Nicolas Roard
Copyright (c) 2013-2014 Quentin Mathe

would be turned into:

Copyright (c) 2010 David Chisnall, Nicolas Roard, Quentin Mathe

David, what's your take on this?

Muhammad Hussein Nasrollahpour is also missing in COPYING and the following files:

  • SCKClangSourceFile
  • SCKIntrospection
  • Tests/ParsingTestFiles/AB
  • Tests/TestRuntimeParsing
  • Tests/TestClangParsing

@davidchisnall
Copy link
Member

We should use the multi-line format. That's the one that I know has been approved by a copyright lawyer. We should, however, drop the (c). The word copyright and the copyright logo have legal meaning, (c) does not.

@qmathe
Copy link
Member

qmathe commented Sep 2, 2016

ok, good to know. We'll stick to the multi-line format then.

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

Successfully merging this pull request may close these issues.

3 participants