Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Stop packaging GPL libquadmath.so #19053

Merged
merged 2 commits into from
Aug 31, 2020
Merged

Stop packaging GPL libquadmath.so #19053

merged 2 commits into from
Aug 31, 2020

Conversation

leezu
Copy link
Contributor

@leezu leezu commented Aug 31, 2020

libquadmath.so is GPL and must not be distributed by Apache projects. Users will need to ensure that libquadmath.so is present on their systems if they use binary builds of MXNet. libquadmath.so has not yet undergone any ABI changes, thus all versions of libquadmath.so are ABI compatible and user just needs to install system version of libquadmath.so.

libgfortran.so can be packaged thanks to GCC Runtime Library Exception. See https://issues.apache.org/jira/browse/LEGAL-523

See https://www.apache.org/legal/resolved.html#category-x

@mxnet-bot
Copy link

Hey @leezu , Thanks for submitting the PR
All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:

  • To trigger all jobs: @mxnet-bot run ci [all]
  • To trigger specific jobs: @mxnet-bot run ci [job1, job2]

CI supported jobs: [centos-gpu, unix-cpu, clang, unix-gpu, sanity, windows-gpu, miscellaneous, edge, website, windows-cpu, centos-cpu]


Note:
Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin.
All CI tests must pass before the PR can be merged.

@leezu leezu mentioned this pull request Aug 31, 2020
@samskalicky
Copy link
Contributor

@leezu can we update the docs too with this PR to say that libquadmath.so is needed and how users can install it?

Copy link
Contributor

@samskalicky samskalicky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for the doc changes!

@leezu leezu merged commit 9b22c8c into apache:master Aug 31, 2020
@leezu leezu deleted the removegpldeps branch August 31, 2020 21:45
leezu added a commit to leezu/mxnet that referenced this pull request Aug 31, 2020
libquadmath.so is GPL and must not be distributed by Apache projects. Users will need to ensure that libquadmath.so is present on their systems if they use binary builds of MXNet. libquadmath.so has not yet undergone any ABI changes, thus all versions of libquadmath.so are ABI compatible and user just needs to install system version of libquadmath.so.

libgfortran.so can be packaged thanks to GCC Runtime Library Exception.

See https://www.apache.org/legal/resolved.html#category-x
leezu added a commit that referenced this pull request Sep 1, 2020
* Stop packaging GPL libquadmath.so (#19053)

libquadmath.so is GPL and must not be distributed by Apache projects. Users will need to ensure that libquadmath.so is present on their systems if they use binary builds of MXNet. libquadmath.so has not yet undergone any ABI changes, thus all versions of libquadmath.so are ABI compatible and user just needs to install system version of libquadmath.so.

libgfortran.so can be packaged thanks to GCC Runtime Library Exception.

See https://www.apache.org/legal/resolved.html#category-x

* Remove unmaintained pip packages

* Workaround pypa/setuptools#2352
@access2rohit access2rohit mentioned this pull request Feb 17, 2021
13 tasks
mseth10 added a commit to mseth10/incubator-mxnet that referenced this pull request May 12, 2021
mseth10 added a commit that referenced this pull request May 12, 2021
* add aarch64 build support

* add publish dockerfile

* add options to build mkldnn w/ apl and acl

add options to build mkldnn w/ arm performance libraries and arm compute library

* remove libquadmath due to licensing issues #19053

Co-authored-by: Zhaoqi Zhu <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants