kenglxn / QRGen

a simple QRCode generation api for java built on top ZXING
http://glxn.net/2012/03/10/qrgen-a-small-wrapper-on-top-of-zxing-for-generating-qrcodes-in-java
1.49k stars 299 forks source link

License compatibility issue #69

Closed ducciovigolo closed 8 years ago

ducciovigolo commented 8 years ago

Hi, I've seen that under the hood your library is leveraging jfreesvg that is licensed under GPL v3. This situation is compatible with the Apache 2 License of QRGen? It seems that the 2 license are not compatible: http://www.apache.org/licenses/GPL-compatibility.html

I'm missing something? Can I use your library in my commercial solution?

Thanks very much

Duccio

kenglxn commented 8 years ago

Thank you for making me aware of this issue. I will look into it, and either remove the functionality completely, or find a valid licensed alternative.

kenglxn commented 8 years ago

Considering the license incompatibility and the lack of a viable alternative, I think it is best to remove this feature completely.

@ducciovigolo You could use an older pre 2.0 version to get around the license for now, for example v1.4. Or if you have time you could fork and fix it for me in a PR :)

atomfrede commented 8 years ago

@kenglxn As I am the one introduced svg support I can provide a PR.

kenglxn commented 8 years ago

@atomfrede Cool man. Thank you :)

ducciovigolo commented 8 years ago

@kenglxn, @atomfrede thank you very much for your answers and your time.

@kenglxn I'll wait for the PR and a new version of your great library!

ducciovigolo commented 8 years ago

@kenglxn, @atomfrede thanks for your fix

@kenglxn any idea about the next release date? I would prefer to use a "official" build instead of a "self-made" one...

thank again

atomfrede commented 8 years ago

@ducciovigolo You can just use this dependency, its more or less like an official version.

<dependency>
        <groupId>com.github.kenglxn</groupId>
        <artifactId>QRGen</artifactId>
        <version>-SNAPSHOT</version>
</dependency>
kenglxn commented 8 years ago

i'll get one out in the next couple of days, in the meantime you can use the commit hash https://jitpack.io/docs/#snapshots

kenglxn commented 8 years ago

@ducciovigolo here you go: https://jitpack.io/#kenglxn/QRGen/2.2.0