Skip to content

[TheOneric] Patch cleanup #40

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

Merged
merged 4 commits into from
Dec 23, 2022

Conversation

dmitrylyzo
Copy link
Collaborator

Pulling from the upstream.
Original author: TheOneric

Part of libass#135

We shouldn't need any patches for libass anymore.

Cherry-picked from: libass@0ae2292
It does nothing since we only build static libs.

Cherry-picked from: libass@0d3cb9f
When HarfBuzz-enabled FreeType is built, HarfBuzz gets located via its
pc-file which references FreeType’s pc-file. If there’s no such pc-file
in the search path it errors out.
Add the previously built HarfBuzz-less FreeType’s pc-file to the search
path to avoid this. It has no ill effect on the static lib we build.

Cherry-picked from: libass@2081393
Together with the prior emscripten bump,
this removes the need for the fcstat patch.
Closes: libass#103

Cherry-picked from: libass@7250e6e
@dmitrylyzo dmitrylyzo merged commit de5cf9a into jellyfin:master Dec 23, 2022
@dmitrylyzo dmitrylyzo deleted the TheOneric/patch_cleanup branch December 23, 2022 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants