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.

Sign up for GitHub

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-fixup-cmake-targets] Rewrite the traversal code with -framework NAME on OSX #19629

Closed
wants to merge 12 commits into from
Closed

[vcpkg-fixup-cmake-targets] Rewrite the traversal code with -framework NAME on OSX #19629

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
3354458
[vcpkg baseline][vcpkg-fixup-cmake-targets] Add double quotes to `-fr…
Aug 18, 2021
f8a4530
version
Aug 18, 2021
d4fbc48
Update ports/vcpkg-cmake-config/vcpkg_cmake_config_fixup.cmake
JackBoosY Aug 18, 2021
27f7ee8
Update scripts/cmake/vcpkg_fixup_cmake_targets.cmake
JackBoosY Aug 18, 2021
abbb4cd
Update scripts/cmake/vcpkg_fixup_cmake_targets.cmake
JackBoosY Aug 18, 2021
fd228ab
Update ports/vcpkg-cmake-config/vcpkg_cmake_config_fixup.cmake
JackBoosY Aug 18, 2021
c2e31ed
Update ports/vcpkg-cmake-config/vcpkg_cmake_config_fixup.cmake
JackBoosY Aug 18, 2021
f61baf4
Update scripts/cmake/vcpkg_fixup_cmake_targets.cmake
JackBoosY Aug 18, 2021
87d30e8
Rewrite the traversal code
Aug 20, 2021
6ab6c60
Merge branch 'dev/jack/fix-vcpkg-fixup-cmake-targets' of https://gith…
Aug 20, 2021
a9bd069
Fix the order issue
Aug 20, 2021
1b9c073
Merge branch 'master' into dev/jack/fix-vcpkg-fixup-cmake-targets
JackBoosY Aug 23, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ports/vcpkg-cmake-config/vcpkg.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
{
"name": "vcpkg-cmake-config",
"version-date": "2021-08-11"
"version-date": "2021-08-18"
}
2 changes: 1 addition & 1 deletion ports/vcpkg-cmake-config/vcpkg_cmake_config_fixup.cmake
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -250,7 +250,7 @@ get_filename_component(_IMPORT_PREFIX "${_IMPORT_PREFIX}" PATH)]]
endif()
list(FIND VCPKG_DETECTED_CMAKE_CXX_IMPLICIT_LINK_FRAMEWORK_DIRECTORIES "${path}" index)
if(NOT index EQUAL -1)
string(REPLACE "${framework}" "-framework ${name}" fixed_line "${fixed_line}")
string(REPLACE "${framework}" "\\\"-framework ${name}\\\"" fixed_line "${fixed_line}")
JackBoosY marked this conversation as resolved.
Show resolved Hide resolved
endif()
endforeach()
string(REPLACE "${line}" "${fixed_line}" targets_content "${targets_content}")
Expand Down
2 changes: 1 addition & 1 deletion scripts/cmake/vcpkg_fixup_cmake_targets.cmake
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -249,7 +249,7 @@ function(vcpkg_fixup_cmake_targets)
endif()
list(FIND VCPKG_DETECTED_CMAKE_CXX_IMPLICIT_LINK_FRAMEWORK_DIRECTORIES "${path}" index)
if(NOT index EQUAL -1)
string(REPLACE "${framework}" "-framework ${name}" fixed_line "${fixed_line}")
string(REPLACE "${framework}" "\\\"-framework ${name}\\\"" fixed_line "${fixed_line}")
JackBoosY marked this conversation as resolved.
Show resolved Hide resolved
endif()
endforeach()
string(REPLACE "${line}" "${fixed_line}" targets_content "${targets_content}")
Expand Down
2 changes: 1 addition & 1 deletion versions/baseline.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6617,7 +6617,7 @@
"port-version": 0
},
"vcpkg-cmake-config": {
"baseline": "2021-08-11",
"baseline": "2021-08-18",
"port-version": 0
},
"vcpkg-gfortran": {
Expand Down
5 changes: 5 additions & 0 deletions versions/v-/vcpkg-cmake-config.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
{
"versions": [
{
"git-tree": "dd7d67c81c2442c831a298d136b019a2bd138565",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
"git-tree": "dd7d67c81c2442c831a298d136b019a2bd138565",
"git-tree": "d4b5ea86aea3416a957fbe2384610feb6106f730",

"version-date": "2021-08-18",
"port-version": 0
},
{
"git-tree": "b3abb12ba8ab43770aea4e5a8d4915319bd295ee",
"version-date": "2021-08-11",
Expand Down