Skip to content

Commit

Permalink
Fix default PKG_CONFIG_PATH in CMakeLists.txt
Browse files Browse the repository at this point in the history
Since `option()` was used it defaulted to OFF instead of the supplied
string as `option()` is used for booleans only, resulting in
`libprotobuf-mutator.pc` file being installed in
`${prefix}/OFF/libprotobuf-mutator.pc` which is clearly undesirable.
Change it to `set()` so the default value is properly stored and used.
  • Loading branch information
nertpinx committed Aug 2, 2024
1 parent b922c8a commit ba6c728
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ option(LIB_PROTO_MUTATOR_TESTING "Enable test building" ON)
option(LIB_PROTO_MUTATOR_DOWNLOAD_PROTOBUF
"Automatically download working protobuf" OFF)
option(LIB_PROTO_MUTATOR_WITH_ASAN "Enable address sanitizer" OFF)
option(PKG_CONFIG_PATH "Directory to install pkgconfig file" "share/pkgconfig")
set(PKG_CONFIG_PATH "share/pkgconfig" CACHE STRING "Directory to install pkgconfig file")
set(LIB_PROTO_MUTATOR_FUZZER_LIBRARIES "" CACHE STRING "Fuzzing engine libs")

# External dependencies
Expand Down

0 comments on commit ba6c728

Please sign in to comment.