diff options
| author | David Kreitschmann <dkreitschmann@seemoo.tu-darmstadt.de> | 2018-06-15 12:44:47 +0200 |
|---|---|---|
| committer | Anders Broman <a.broman58@gmail.com> | 2018-06-21 03:33:03 +0000 |
| commit | 11ba10dd4b8865d1d4b866cd808ec8fe4defe3b2 (patch) | |
| tree | 2ff94017b1d563c78bae94cb5266c94137617d9e /wsutil/CMakeLists.txt | |
| parent | 17604f15a1a1fe4199449150675c2613874ae4a5 (diff) | |
Fix build paths for cmake's Xcode project generator on macOS.
Multi-configuration generators (such as Xcode or VS) append the current build configuration to most paths (eg. Debug/Release). Currently this results in inconsistent paths for the application bundle and the included command line tools. This commit sets the correct path information for multi-configuration generators for macOS application bundles. The standard Makefile behaviour is untouched.
One Windows specific configuration was changed, as it was conflicting with these changes. This needs to be checked before merging.
Additionally the wrapper scripts are omitted for Xcode, as the path to the binaries depends on the configuration chosen in Xcode. Therefore it is not viable to create these scripts in the cmake run.
Bug: 11816
Change-Id: Ib43d82eb04600a0e2f2b020afb44b579ffc7a7c9
Reviewed-on: https://code.wireshark.org/review/28291
Petri-Dish: Guy Harris <guy@alum.mit.edu>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'wsutil/CMakeLists.txt')
| -rw-r--r-- | wsutil/CMakeLists.txt | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/wsutil/CMakeLists.txt b/wsutil/CMakeLists.txt index 698bc5bfbd..87f111c1e3 100644 --- a/wsutil/CMakeLists.txt +++ b/wsutil/CMakeLists.txt @@ -267,9 +267,17 @@ set_target_properties(wsutil PROPERTIES ) if(ENABLE_APPLICATION_BUNDLE) - set_target_properties(wsutil PROPERTIES - LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/run/Wireshark.app/Contents/Frameworks - ) + + if(NOT CMAKE_CFG_INTDIR STREQUAL ".") + # Xcode + set_target_properties(wsutil PROPERTIES + LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/run/$<CONFIG>/Wireshark.app/Contents/Frameworks + ) + else() + set_target_properties(wsutil PROPERTIES + LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/run/Wireshark.app/Contents/Frameworks + ) + endif() endif() ABICHECK(libwsutil) |