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*: Enable webp #26022

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

ffmpeg*: Enable webp #26022

wants to merge 4 commits into from

Conversation

ryandesign
Copy link
Contributor

Description

ffmpeg-devel: Enable webp

ffmpeg7: Enable webp

ffmpeg6: Enable webp

ffmpeg: Enable webp and lzo2

Closes: https://trac.macports.org/ticket/70958

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 12.7.5 21H1222 x86_64
Xcode 14.2 14C18

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL in commit message?
  • checked your Portfile with port lint --nitpick?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?
  • checked that the Portfile's most important variants haven't been broken?

@macportsbot
Copy link

Notifying maintainers:
@mascguy for port ffmpeg-devel, ffmpeg, ffmpeg6, ffmpeg7.
@dbevans for port ffmpeg.
@jeremyhu for port ffmpeg.
@barracuda156 for port ffmpeg7.

@macportsbot macportsbot added maintainer: open Affects an openmaintainer port by: member Created by a member with commit rights labels Sep 30, 2024
@barracuda156
Copy link
Contributor

@ryandesign Maybe combine with #25266 otherwise need to revbump everything twice.

@ryandesign
Copy link
Contributor Author

Good idea. Do the maintainers have any objections to enabling webp support as in this PR? If not, I can add the changes to #25266.

@barracuda156
Copy link
Contributor

@ryandesign From my side I am fine. I recall I wanted to enable it once, but postponed to avoid revbump, and then forgot, of course.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
by: member Created by a member with commit rights maintainer: open Affects an openmaintainer port
Development

Successfully merging this pull request may close these issues.

6 participants