-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
[libdvdcss] add new port of version 1.4.2 (d0b6a291) for libmpv #40693
base: master
Are you sure you want to change the base?
Conversation
The upstream build system is autotools, and vcpkg supports that with |
I added a CMakeLists.txt file to support find_package in CMake and configured MSVC link flags to support UWP. autotools cannot accomplish these tasks. |
vcpkg builds several autotools ports for uwp. |
OK, I have marked it as unofficial.
|
I'm still opposed to vendoring a build system when it is not necessary. |
I found 248 CMakeLists.txt in ports dir
for example: vulkan, usockets, spdk, lua, live555, libass, imgui, angle |
So what? Either it is still necessary, or it wasn't cleaned up. #40677 removes one of them. |
new file mode 100644 | ||
index 0000000..0a32e72 | ||
--- /dev/null | ||
+++ b/msvc/uwpapi.cpp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is way way too big of a patch to not be accepted by upstream; it would be better to just say "supports": "!uwp"
if upstream explicitly refuses to support that.
|
2. test ok
log
log
log
log
|
@capric8416 Thanks for your PR, I updated the description of this PR, move the test log to a new comment, and update the check list of description. Please double confirm the check list, this is needed to do when adding the new port, the old list applied to modify an existing port. |
#40898 adds ports for pristine libdvdnav and libdvdread based on autotools, and integrates them with ffmpeg. |
1. descriptions
find_package
calls are REQUIRED, are satisfied byvcpkg.json
's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx.vcpkg.json
matches what upstream says.vcpkg.json
matches what upstream says../vcpkg x-add-version --all
and committing the result.libmpv is very hard to build on windows with msvc or with msys2 gcc, but it can be build on latest arch and ubuntu linux, so i download release from https://github.com/shinchiro/mpv-winbuild-cmake.