Skip to content
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

[ffmpeg] OpenSSL feature requires LGPL v3, not non-free license #36879

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

horenmar
Copy link
Contributor

OpenSSL has moved to Apache 2 license, which is clearly compatible with LGPL v3 (but not v2.1, which is ffmpeg's default). Note that FFmpeg devs believe that the previous license is also compatible with LGPL (any version), but not GPL.

See https://github.com/FFmpeg/FFmpeg/blob/37702e20663dc8111534229af551763c2a954e73/LICENSE.md#incompatible-libraries for more details.

Note that the upstream also believes that Fraunhofer FDK AAC is also compatible with LGPL, but IANAL and fdk-aac uses its own custom license.

  • Changes comply with the maintainer guide.
  • SHA512s are updated for each updated download.
  • The "supports" clause reflects platforms that may be fixed by this new version.
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

@FrankXie05 FrankXie05 added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Feb 21, 2024
FrankXie05
FrankXie05 previously approved these changes Feb 22, 2024
@FrankXie05 FrankXie05 added the info:reviewed Pull Request changes follow basic guidelines label Feb 22, 2024
@vicroms vicroms added requires:vcpkg-team-review This PR or issue requires someone on the vcpkg team to take a further look. and removed requires:vcpkg-team-review This PR or issue requires someone on the vcpkg team to take a further look. labels Feb 22, 2024
@vicroms
Copy link
Member

vicroms commented Feb 22, 2024

This seems fine to me once the merge conflicts have been fixed.

@vicroms vicroms marked this pull request as draft February 22, 2024 18:27
OpenSSL has moved to Apache 2 license, which is clearly compatible
with LGPL v3 (but not v2.1, which is ffmpeg's default). Note that
FFmpeg devs believe that the previous license is also compatible
with LGPL (any version), but not GPL.

See https://github.com/FFmpeg/FFmpeg/blob/37702e20663dc8111534229af551763c2a954e73/LICENSE.md#incompatible-libraries
for more details.

Note that the upstream also believes that Fraunhofer FDK AAC is
also compatible with LGPL, but IANAL and fdk-aac uses its own
custom license.
@vicroms
Copy link
Member

vicroms commented Feb 27, 2024

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vicroms vicroms merged commit ed325ed into microsoft:master Feb 29, 2024
16 checks passed
@horenmar horenmar deleted the update-ffmpeg-licensing branch February 29, 2024 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants