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

[fribidi] Fix cross-compilation in meson. Re-enable arm64-windows. #23063

Merged
merged 1 commit into from
Feb 16, 2022

Conversation

ras0219-msft
Copy link
Contributor

The buildsystem revert back to autoconf in #21948 did not actually fix cross-compilation; rather it just allowed the build to stumble forward hoping that the cross-built executables would run.

This PR actually fixes cross-compilation by adding a proper host dependency and reusing the tables generated from that host build.

This PR is specifically in pursuit of reenabling ffmpeg:arm64-windows.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

You have modified or added at least one vcpkg.json where a "license" field is missing.

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/fribidi/vcpkg.json

Valid values for the license field can be found in the documentation

ports/fribidi/portfile.cmake Outdated Show resolved Hide resolved
ports/fribidi/portfile.cmake Outdated Show resolved Hide resolved
ports/fribidi/portfile.cmake Outdated Show resolved Hide resolved
ports/fribidi/portfile.cmake Outdated Show resolved Hide resolved
@LilyWangLL LilyWangLL added category:port-bug The issue is with a library, which is something the port should already support requires:author-response info:internal This PR or Issue was filed by the vcpkg team. labels Feb 14, 2022
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

You have modified or added at least one vcpkg.json where a "license" field is missing.

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/fribidi/vcpkg.json

Valid values for the license field can be found in the documentation

@LilyWangLL LilyWangLL added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels Feb 16, 2022
@vicroms vicroms merged commit 582b265 into microsoft:master Feb 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support info:internal This PR or Issue was filed by the vcpkg team. info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants