Skip to content

Commit

Permalink
Fix build on macOS with newer P4 SDK versions by linking with the p4s…
Browse files Browse the repository at this point in the history
…cript_cstub library, for salesforce#78

- The p4client library seems to have unconditional references to the P4Script machinery, even if that feature is not used by the application (in this case, p4-fusion). This causes linking errors (undefined references to sqlite, and curl functions)
- The p4script_cstub library seems to be an official solution for this situation, as it contains stubs (dummy, empty implementations) of all relevant functions
  • Loading branch information
peter-esik committed Jun 27, 2024
1 parent c51700b commit 48f6239
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions p4-fusion/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -45,10 +45,9 @@ target_link_libraries(p4-fusion PUBLIC
client
rpc
supp
p4script_cstub
${OPENSSL_SSL_LIBRARIES}
${OPENSSL_CRYPTO_LIBRARIES}
p4script
p4script_c
git2
minitrace
)

0 comments on commit 48f6239

Please sign in to comment.