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

README page update (license) #1158

Merged
merged 3 commits into from Apr 3, 2016
Merged

README page update (license) #1158

merged 3 commits into from Apr 3, 2016

Conversation

dcoeurjo
Copy link
Member

@dcoeurjo dcoeurjo commented Apr 3, 2016

PR Description

Add more information about licensing in the README page.

Checklist

  • [n.a.] Unit-test of your feature with Catch.
  • [n.a. ] Doxygen documentation of the code completed (classes, methods, types, members...)
  • [ n.a.] Documentation module page added or updated.
  • [ n.a.] New entry in the ChangeLog.md added.
  • [ n.a.] No warning raised in Debug cmake mode (otherwise, Travis C.I. will fail).
  • [ n.a.] All continuous integration tests pass (Travis & appveyor)

License
=======

DGtal core libarary is under the GNU LGPL v3 license (see [LICENSE.txt](LICENSE.txt)). DGtal documentation and website materials are under the [Creative Commons Attribution-NonCommercial-ShareAlike 4.0 ](http://creativecommons.org/licenses/by-nc-sa/4.0/).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it looks not working (due to the .txt )

@dcoeurjo
Copy link
Member Author

dcoeurjo commented Apr 3, 2016

you're right;) thx

DGtal core libarary is under the GNU LGPL v3 license (see [LICENSE.txt](LICENSE.txt)). DGtal documentation and website materials are under the [Creative Commons Attribution-NonCommercial-ShareAlike 4.0 ](http://creativecommons.org/licenses/by-nc-sa/4.0/).

If some dependencies are activated, resulting binaries may follow the dependencies licenses. More precisely:
* If the library is built with the 3D internal viewer using [libQGLViewer](http://libqglviewer.com) (```WITH_QGLVIEWER``` flag set to ```true```), then resulting binaries libQGLViewer license (GNU GPL or commercial, see [libQGLViewer licensing](http://libqglviewer.com/dualLicensing.html)).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it looks something is missing at the end:
=> "then the resulting binaries are under the libQGLViewer license (GNU...)"?

@kerautret
Copy link
Member

fine merging

@kerautret kerautret merged commit ed28393 into DGtal-team:master Apr 3, 2016
@dcoeurjo dcoeurjo deleted the readme branch October 7, 2021 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants