-
Notifications
You must be signed in to change notification settings - Fork 88
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
GPL license found? #139
Comments
I was certainly unaware of this. I'll look into it. We don't like using
GPL. -Chris
…On Tue, Mar 19, 2019 at 7:32 PM Isaac I.Y. Saito ***@***.***> wrote:
Some packages in this repo depend on ceres (e.g. industrial_extrinsic_cal
<https://github.com/ros-industrial/industrial_calibration/blob/826b46b8fd297b1886b1efa94ed4c19ea4a1549c/industrial_extrinsic_cal/src/nodes/range_camera_excal.cpp#L40>)
and rgbd_depth_correction
<https://github.com/ros-industrial/industrial_calibration/blob/826b46b8fd297b1886b1efa94ed4c19ea4a1549c/rgbd_depth_correction/include/depth_calibration/depth_calibration.h#L39>,
then statically link to it.
ceres-solver.org <http://ceres-solver.org/installation.html> reads:
By default, Ceres will link to SuiteSparse if it and all of its
dependencies are present. Turn this OFF to build Ceres without SuiteSparse.
SuiteSparse is licensed under a mixture of GPL/LGPL/Commercial terms.
Ceres requires some components that are only licensed under GPL/Commercial
terms.
IANAL, this gives me an impression that all downstream software of these
packages need to follow what GPL requires to do.
- Was there any discussion regarding this?
- Was there any solution? (I hope I'm just wrong.)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#139>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AECEtjIpNvbnt4aZGPUv-5fzEBHgRztmks5vYYGGgaJpZM4b9eNU>
.
|
@130s wrote:
Only if the I realise it's common to have Ceres link to SuiteSparse, but it's not required. |
I believe the Ceres debian which includes a copywrite declaration is in
error because it include suitsparse which is GPL. It is possible to build
ceres with suitsparse disabled. I don't know what speed ramifications this
might cause. However, I'd recommend that if anyone delivers a system to a
commercial customer or creates a debian using ceres that they build it from
source before linking to it. That way suitsparse can be excluded.
Otherwise,the copywrite for the derived work would have to be GPL. Please
understand that I am not a lawyer, so my recommendations are just an
uneducated opinion. I've only recently started using ceres as a debian.
Before, I'd always built from source. I guess I'll go back to that. -Chris
…On Wed, Mar 20, 2019 at 2:47 AM G.A. vd. Hoorn ***@***.***> wrote:
@130s <https://github.com/130s> wrote:
IANAL, this gives me an impression that all downstream software of these
packages need to follow what GPL requires to do.
Only if the libceres that is being linked to makes use of SuiteSparse, as
also stated in the quote you include from the Ceres installation manual.
I realise it's common to have Ceres link to SuiteSparse, but it's not
required.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#139 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AECEtvZAADLYPQaFj_waPpaN-CFp8G8fks5vYeengaJpZM4b9eNU>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Some packages in this repo depend on
ceres
(e.g. industrial_extrinsic_cal) and rgbd_depth_correction, then statically link to it.ceres-solver.org reads:
IANAL, this gives me an impression that all downstream software of these packages need to follow what GPL requires to do.
The text was updated successfully, but these errors were encountered: