-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
CMake: Add LGPL Option #11600
base: master
Are you sure you want to change the base?
CMake: Add LGPL Option #11600
Conversation
caebdb0
to
993ea03
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this removes the usage of Charts then the PX4 tuning page also uses charts. Can't remember if it PX4 only. Might be ArduPilot tuning as well.
Also this is interesting, but really wonder whether the additional complexity is needed? |
I'll have to double check the tuning pages. I don't think it'll really be all that extra complex but we can see what it looks like after we're done with all the other custom cmake stuff. In my opinion this is pretty useful as I have worked with a few companies that won't use anything but LGPL |
Agree with @HTRamsey. I also know several companies that are just thinking about abandoning QGC in favor of using only lgpl solutions. |
If any of those companies actually ever actually gave back to the ecosystem then I'd certainly be more interested :) |
So far these are startups and one even makes a contribution to qgc in the person of me. But no one can move on with gpl. And the prices for commercial use of qt are simply inadequate, especially since it is also subject to the qt gender license for devices, not to mention the fact that qt prohibits mixing a commercial license with an open sources. |
Yeah, the Qt pricing is really kinda crap. |
cedd3fe
to
694c9e8
Compare
3734b7f
to
fc631ed
Compare
Closes #8252