[CMake] InstallRequiredSystemLibraries.cmake

[David Cole]

Thanks for this report.

The commit that fixes this is now in our 'next' branch:
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=fc144924a0782b37c320378ac76482f0534c7530

Fixing it, and adding a warning about required files that do not exist
revealed several other issues when using this file with VS10.

The commit also fixes those issues.

We'll do our best to include this fix in the 2.8.4-rc2 candidate next week.


Thanks,
David


On Wed, Jan 12, 2011 at 7:16 PM, J Decker <d3ck0r at gmail.com> wrote:
> This currently has no support for watcom's libraries, I'm working on
> adding that in the same method basically that visual studio uses for
> all of its versions; this is still in progress but I'll come up with
> soemhting like a patch in abit.
>
> While doing this I found that there is a bug with VS2010 support..
>
> There is a reference of VC90 that should be VC100 for copying the
> debug libraries.
>
>
> line 135
>      SET(MSVC10_CRT_DIR
>        "${MSVC10_REDIST_DIR}/Debug_NonRedist/${CMAKE_MSVC_ARCH}/Microsoft.VC90.DebugCRT")
>      SET(__install__libs ${__install__libs}
>        "${MSVC10_CRT_DIR}/Microsoft.VC100.DebugCRT.manifest"
>        "${MSVC10_CRT_DIR}/msvcp100d.dll"
>        "${MSVC10_CRT_DIR}/msvcr100d.dll"
>        )
>
> should be
>
>      SET(MSVC10_CRT_DIR
>        "${MSVC10_REDIST_DIR}/Debug_NonRedist/${CMAKE_MSVC_ARCH}/Microsoft.VC100.DebugCRT")
>      SET(__install__libs ${__install__libs}
>        "${MSVC10_CRT_DIR}/Microsoft.VC100.DebugCRT.manifest"
>        "${MSVC10_CRT_DIR}/msvcp100d.dll"
>        "${MSVC10_CRT_DIR}/msvcr100d.dll"
>        )
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html
>
> Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ
>
> Follow this link to subscribe/unsubscribe:
> http://www.cmake.org/mailman/listinfo/cmake
>