On 15/08/15 21:00, Gregor Jasny wrote:
> On 13/08/15 12:56, Ruslan Baratov wrote:
>> This looks great! I've spent several hours trying to figure out how
>> '$(EFFECTIVE_PLATFORM_NAME)' can be set to Xcode and some CMake friendly
>> string to cmake_install.cmake script. I didn't know Xcode understand
>> `${VAR}` syntax (probably it didn't, it's just expanded as an
>> environment variable). I've tested your fix and it works fine for me.
>> Since we don't need to replace string with path manually I think your
>> solution is neater. I've added description to commit and rebased it, see
>> attachment.
>
> I start working on a unit test for this now.
I have a first test here:
https://github.com/gjasny/CMake/commit/dcfc8f6f637ffdebe0e7f71ba2768c2acd0811eb
Will add some more during the next days.
--
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
