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

Reply via email to