-
Notifications
You must be signed in to change notification settings - Fork 751
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
Enable native builds of szip via libaec #1334
Conversation
After sorting out Windows, I may know how to convert everything to cmake now. That might be another pull request. |
Why do we need to build libaec just for szip? Let's remove that dependency if we're not using it. |
How else do you propose building szip? Basically libaec provides a compatability layer that provides a szip interface. szip itself is otherwise provided via a non-free license. |
Ah, yes, I remember. So it doesn't matter if we use it via libaec, it's still not free and it's just libaec not being compliant isn't? |
As far as I know the license of libaec is https://gitlab.dkrz.de/k202009/libaec/-/blob/master/LICENSE.txt . It only provides the SZIP API and does not use any code from proprietary SZIP as far as I know. https://gitlab.dkrz.de/k202009/libaec/-/blob/master/README.SZIP
|
According to NASA, there are no known patents on this standard compression either. https://public.ccsds.org/Pubs/121x0b3.pdf
|
Thank you for the edits and the merge! |
szip is a compressor that can be built into HDF5. A free implementation of szip is available via libaec.