Skip to content

Commit 4ee6af4

Browse files
committed
[CMake] Version of LibPSL not reported
https://bugs.webkit.org/show_bug.cgi?id=219837 Reviewed by Adrian Perez de Castro. When determining the version of LibPSL from the header file LIBPSL_VERSION was set but the VERSION_VAR reported was LibPSL_VERSION. Consistently use LibPSL_VERSION within the file to prevent this. Additionally LibPSL_LIBRARY not LibPSL_LIBRARIES should be marked_as_advanced. * Source/cmake/FindLibPSL.cmake: Identifier: 232414@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@270751 268f45cc-cd09-0410-ab3c-d52691b4dbfc
1 parent c701e88 commit 4ee6af4

File tree

2 files changed

+24
-9
lines changed

2 files changed

+24
-9
lines changed

ChangeLog

+15
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,18 @@
1+
2020-12-13 Don Olmstead <[email protected]>
2+
3+
[CMake] Version of LibPSL not reported
4+
https://bugs.webkit.org/show_bug.cgi?id=219837
5+
6+
Reviewed by Adrian Perez de Castro.
7+
8+
When determining the version of LibPSL from the header file LIBPSL_VERSION was set
9+
but the VERSION_VAR reported was LibPSL_VERSION. Consistently use LibPSL_VERSION
10+
within the file to prevent this.
11+
12+
Additionally LibPSL_LIBRARY not LibPSL_LIBRARIES should be marked_as_advanced.
13+
14+
* Source/cmake/FindLibPSL.cmake:
15+
116
2020-12-11 Don Olmstead <[email protected]>
217

318
[CMake] Determine correct visibility for linked frameworks

Source/cmake/FindLibPSL.cmake

+9-9
Original file line numberDiff line numberDiff line change
@@ -70,18 +70,18 @@ find_library(LibPSL_LIBRARY
7070

7171
if (LibPSL_INCLUDE_DIR AND NOT LibPSL_VERSION)
7272
if (EXISTS "${LibPSL_INCLUDE_DIR}/libpsl.h")
73-
file(READ "${LibPSL_INCLUDE_DIR}/libpsl.h" LIBPSL_VERSION_CONTENT)
73+
file(READ "${LibPSL_INCLUDE_DIR}/libpsl.h" LibPSL_VERSION_CONTENT)
7474

75-
string(REGEX MATCH "#define +PSL_VERSION_MAJOR +([0-9]+)" _dummy "${LIBPSL_VERSION_CONTENT}")
76-
set(LIBPSL_VERSION_MAJOR "${CMAKE_MATCH_1}")
75+
string(REGEX MATCH "#define +PSL_VERSION_MAJOR +([0-9]+)" _dummy "${LibPSL_VERSION_CONTENT}")
76+
set(LibPSL_VERSION_MAJOR "${CMAKE_MATCH_1}")
7777

78-
string(REGEX MATCH "#define +PSL_VERSION_MINOR +([0-9]+)" _dummy "${LIBPSL_VERSION_CONTENT}")
79-
set(LIBPSL_VERSION_MINOR "${CMAKE_MATCH_1}")
78+
string(REGEX MATCH "#define +PSL_VERSION_MINOR +([0-9]+)" _dummy "${LibPSL_VERSION_CONTENT}")
79+
set(LibPSL_VERSION_MINOR "${CMAKE_MATCH_1}")
8080

81-
string(REGEX MATCH "#define +PSL_VERSION_PATCH +([0-9]+)" _dummy "${LIBPSL_VERSION_CONTENT}")
82-
set(LIBPSL_VERSION_PATCH "${CMAKE_MATCH_1}")
81+
string(REGEX MATCH "#define +PSL_VERSION_PATCH +([0-9]+)" _dummy "${LibPSL_VERSION_CONTENT}")
82+
set(LibPSL_VERSION_PATCH "${CMAKE_MATCH_1}")
8383

84-
set(LIBPSL_VERSION "${LIBPSL_VERSION_MAJOR}.${LIBPSL_VERSION_MINOR}.${LIBPSL_VERSION_PATCH}")
84+
set(LibPSL_VERSION "${LibPSL_VERSION_MAJOR}.${LibPSL_VERSION_MINOR}.${LibPSL_VERSION_PATCH}")
8585
endif ()
8686
endif ()
8787

@@ -101,7 +101,7 @@ if (LibPSL_LIBRARY AND NOT TARGET LibPSL::LibPSL)
101101
)
102102
endif ()
103103

104-
mark_as_advanced(LibPSL_INCLUDE_DIR LIBPSL_LIBRARIES)
104+
mark_as_advanced(LibPSL_INCLUDE_DIR LibPSL_LIBRARY)
105105

106106
if (LibPSL_FOUND)
107107
set(LibPSL_LIBRARIES ${LibPSL_LIBRARY})

0 commit comments

Comments
 (0)