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

[vcpkg] Fix build type in vcpkg_build_make.cmake [x264] Modernize #9602

Conversation

NancyLi1013
Copy link
Contributor

When only building release type use vcpkg_build_make(), it will cause failures like this:
make: *** No rule to make target 'install'. Stop.

The solution:
Add the following code when installing targets to fix this issue.

if(NOT DEFINED VCPKG_BUILD_TYPE OR VCPKG_BUILD_TYPE STREQUAL BUILDTYPE)

endif()

Related issue #9590

Note: No feature needs to test.

@NancyLi1013 NancyLi1013 added the info:internal This PR or Issue was filed by the vcpkg team. label Jan 9, 2020
@NancyLi1013 NancyLi1013 marked this pull request as ready for review January 10, 2020 01:24
@ras0219-msft ras0219-msft changed the title [vcpkg] Fix build type in vcpkg_build_make.cmake [vcpkg] Fix build type in vcpkg_build_make.cmake [x264] Modernize Jan 14, 2020
@ras0219-msft ras0219-msft merged commit 066c7b9 into microsoft:master Jan 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info:internal This PR or Issue was filed by the vcpkg team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants