honzasp / rust-turbojpeg

Rust bindings for TurboJPEG library
MIT License
16 stars 12 forks source link

Licensing #11

Closed bhagen55 closed 1 year ago

bhagen55 commented 1 year ago

Hi, Could I potentially convince you to convert the licensing of this Crate/project to a more defined license perhaps MIT? There are some issues with "unlicensed" software that can make life a bit difficult. I specifically would like to use it in code I am writing for my job, so I realize asking for the license to be changed so a corporation can use your work for free is probably a bit annoying, but I figured I'd try asking anyways :slightly_smiling_face: .

honzasp commented 1 year ago

Hi, I'm willing to perform any legal rituals that would help you use the library! :) I would like to keep the library in the public domain, but I can dual-license it under MIT. Do you have some specific arrangement that would work for your job?

honzasp commented 1 year ago

And btw, looking at your Github profile, are you working for Zipline, the drone delivery company? :)

bhagen55 commented 1 year ago

I don't have anything specific other than using a general open source license, like MIP or Apache-2.0. Dual licensing is fine!

Yes I do work for Zipline!

honzasp commented 1 year ago

I added an MIT license in this commit on the mit branch: https://github.com/honzasp/rust-turbojpeg/commit/7ce56ede68fc2c40e04e4d1a01d5ffc20ef0052b

Could you please have a look, whether this would work for you? If so, I will merge this and release a new version to crates.io.

I really like what you are doing at Zipline, and I'm glad that my little library can help you :)

bhagen55 commented 1 year ago

That is perfect! Thank you very much.

honzasp commented 1 year ago

Good, I merged the changes and released version 0.5.3 to crates.io. I hope that the library will be useful to you!

micolous commented 1 year ago

Hi, I don't think this is fully fixed: turbojpeg-sys still declares it is Unlicense only:

https://github.com/honzasp/rust-turbojpeg/blob/abbc5567d59220cb7ba0cf88c334bdadfa50c258/turbojpeg-sys/Cargo.toml#L10

Thanks!

honzasp commented 8 months ago

This is going to be fixed in version 1.0.0 (I just released a pre-release 1.0.0-pre.0)