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

License issue with included files #9

Closed
neeraj9 opened this issue Jun 4, 2015 · 6 comments
Closed

License issue with included files #9

neeraj9 opened this issue Jun 4, 2015 · 6 comments

Comments

@neeraj9
Copy link

neeraj9 commented Jun 4, 2015

I notice that you have moved to LGPL, which is a good news for me but I notice that you have included sources which are having GPLv2 and GPLv3 licenses. Are you planning to remove those to cleanup the license conflict?

@stefanocasazza
Copy link
Owner

Hi,

I am not sure to understand, you may be point to miniLZO for GPLv2 but I
don't see which is GPLv3.
I am not an expert of license conflict , can you suggest some more details
about and eventually a resolution...?

Greetings

2015-06-04 12:39 GMT+02:00 Neeraj notifications@github.com:

I notice that you have moved to LGPL, which is a good news for me but I
notice that you have included sources which are having GPLv2 and GPLv3
licenses. Are you planning to remove those to cleanup the license conflict?


Reply to this email directly or view it on GitHub
#9.

@neeraj9
Copy link
Author

neeraj9 commented Jun 4, 2015

A quick rundown shows the following:

./tests/ulib/test_bison.* are under GPLv3.
./include/ulib/replace/strstream.h is under GPLv3.
./src/ulib/replace/strndup.c is under GPLv2
./tests/examples/benchmark/docroot/README.txt still indicates GPLv2 when you have moved to LGPLv3.
./src/ulib/base/lzo/* is GPLv2
./include/ulib/base/lzo/* is GPLv2
./include/ulib/flex/bison.skl is under GPLv2.

@neeraj9
Copy link
Author

neeraj9 commented Jun 4, 2015

  1. I think strndup and strstream can be easily removed though its probably added for certain platforms where these arent available.
  2. tests for bison should carry a readme indicating the license if at all you want to keep those and also add a reference to those somewhere in root path of your project.
  3. lzo needs to go and should be replaced with an equivalent LGPL or another liberal license.

@stefanocasazza
Copy link
Owner

Hi,

ok, I tried to clean the issue, and I commit the change to the repository...

About miniLZO I find this alternative:
https://code.google.com/p/miniz/

What do you think?

Greetings

2015-06-04 13:44 GMT+02:00 Neeraj notifications@github.com:

  1. I think strndup and strstream can be easily removed though its
    probably added for certain platforms where these arent available.
  2. tests for bison should carry a readme indicating the license if at
    all you want to keep those and also add a reference to those somewhere in
    root path of your project.
  3. lzo needs to go and should be replaced with an equivalent LGPL or
    another liberal license.


Reply to this email directly or view it on GitHub
#9 (comment).

@neeraj9
Copy link
Author

neeraj9 commented Jul 6, 2015

looks good

@stefanocasazza
Copy link
Owner

Hi,

ok, thank you...

Greetings

2015-07-06 10:17 GMT+02:00 Neeraj notifications@github.com:

looks good


Reply to this email directly or view it on GitHub
#9 (comment).

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