On 07/09/2014 09:40 AM, Tim Blechmann wrote:
> cfbundles are currently not installed as directory, but the binary from
> Contents/MacOS is directly installed to the destination.
>
> attached patch fixes the issue.
Thanks.
I do not understand this line:
> + std::string targetNameBase = targetName.substr(0,
> targetName.find_first_of("/"));
Why do we need to look for the first slash?
Also, look earlier in that function for the blocks starting in:
// Handle OSX Bundles.
if(this->Target->IsAppBundleOnApple())
and
else if(this->Target->IsFrameworkOnApple())
Those cases do special handling for the post-installation tweaks.
Is something similar needed here?
-Brad
--
Powered by www.kitware.com
Please keep messages on-topic and check the CMake FAQ at:
http://www.cmake.org/Wiki/CMake_FAQ
Kitware offers various services to support the CMake community. For more
information on each offering, please visit:
CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html
Visit other Kitware open-source projects at
http://www.kitware.com/opensource/opensource.html
Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers