Re: [cmake-developers] Warnings with CMake 2.8.12-rc3 driving dashboards

2013-09-11 Thread Brad King
Steve,

On 09/11/2013 11:10 AM, David Cole wrote:
 Configure policy warnings with CMake 2.8.12-rc3 in a Qt-enabled VTK 
 'master' dashboard:
 
 http://open.cdash.org/viewConfigure.php?buildid=3025085

The warning reads

CMake Warning (dev) in GUISupport/Qt/CMakeLists.txt:
  Policy CMP0022 is not set: INTERFACE_LINK_LIBRARIES defines the link
  interface.  Run cmake --help-policy CMP0022 for policy details.  Use the
  cmake_policy command to set the policy and suppress this warning.

  Target vtkGUISupportQt has a INTERFACE_LINK_LIBRARIES property which
  differs from its LINK_INTERFACE_LIBRARIES properties.

Can you please teach it to print out the value of each of the two
properties so we have some hint about what is different?

Thanks,
-Brad
--

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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers


Re: [cmake-developers] Warnings with CMake 2.8.12-rc3 driving dashboards

2013-09-11 Thread Brad King
On 09/11/2013 11:10 AM, David Cole wrote:
 Build warnings about unrecognized option '/link'; ignored on ITK, VTK 

Please post a sample link line that generates this warning.

Thanks,
-Brad
--

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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers


[cmake-developers] Warnings with CMake 2.8.12-rc3 driving dashboards

2013-09-11 Thread David Cole
Configure policy warnings with CMake 2.8.12-rc3 in a Qt-enabled VTK 
'master' dashboard:


   http://open.cdash.org/viewConfigure.php?buildid=3025085

Build warnings about unrecognized option '/link'; ignored on ITK, VTK 
and VTKWikiExamples dashboards (with 2.8.12-rc3 driving, ninja 
generator, MS compiler...):


   http://open.cdash.org/viewBuildError.php?type=1buildid=3025085
   (and several others...)

These warnings did not occur with CMake 2.8.11.2 driving -- see 
yesterday's equivalents.



David C.

--

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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers


Re: [cmake-developers] Warnings with CMake 2.8.12-rc3 driving dashboards

2013-09-11 Thread David Cole
C:\dev\My Tests\Nightly\VTK Win32-ninja-Debugninja -v 
vtkChartsCoreCxxTests
[1/1] cmd.exe /c cd .  C:\Program Files\CMake 2.8\bin\cmake.exe -E 
vs_link_exe C:\PROGRA~1\MICRO
S~1.0\VC\bin\link.exe /nologo @CMakeFiles/vtkChartsCoreCxxTests.rsp  
/out:bin\vtkChartsCoreCxxTests.
exe /implib:lib\vtkChartsCoreCxxTests.lib 
/pdb:bin\vtkChartsCoreCxxTests.pdb /version:0.0  /machine:

X86   /debug /INCREMENTAL  /subsystem:console   cd .
LINK : warning LNK4044: unrecognized option '/link'; ignored
LINK : bin\vtkChartsCoreCxxTests.exe not found or not built by the last 
incremental link; performing

full link

It must be in the CMakeFiles/vtkChartsCoreCxxTests.rsp file, but that 
file's gone after the command executes. Is there an easy way to keep 
the response files around, or capture their contents, or just avoid 
their use without re-building CMake...?


--

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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers


Re: [cmake-developers] Warnings with CMake 2.8.12-rc3 driving dashboards

2013-09-11 Thread Brad King
On 09/11/2013 11:10 AM, David Cole wrote:
 Configure policy warnings with CMake 2.8.12-rc3 in a Qt-enabled VTK 
 'master' dashboard:
 
 http://open.cdash.org/viewConfigure.php?buildid=3025085

Here is a fix for the CMP0022 warnings:

 http://review.source.kitware.com/#/t/3233/

-Brad
--

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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers


Re: [cmake-developers] Warnings with CMake 2.8.12-rc3 driving dashboards

2013-09-11 Thread Brad King
On 09/11/2013 12:05 PM, David Cole wrote:
 C:\PROGRA~1\MICROS~1.0\VC\bin\link.exe /nologo 
 @CMakeFiles/vtkChartsCoreCxxTests.rsp  
[snip]
 It must be in the CMakeFiles/vtkChartsCoreCxxTests.rsp file, but that 
 file's gone after the command executes.

I think ninja writes that file out temporarily.  Therefore the
ninja build files must have the /link option in them somewhere.

-Brad
--

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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers


Re: [cmake-developers] Warnings with CMake 2.8.12-rc3 driving dashboards

2013-09-11 Thread Brad King
On 09/11/2013 12:32 PM, Brad King wrote:
 I think ninja writes that file out temporarily.  Therefore the
 ninja build files must have the /link option in them somewhere.

Okay, I found the problem.  There was a Ninja-specific change made
here to do linking with response files:

 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=e31df039

That commit added special handling for /link with executables
of which I was not aware when I made this change in 2.8.12:

 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=fb9f73de

The latter change switches from cl.exe to link.exe for linking
executables and of course the latter does not know about /link.

Try this patch:

diff --git a/Modules/Platform/Windows-MSVC.cmake 
b/Modules/Platform/Windows-MSVC.cmake
index 685638a..8cb7fd5 100644
--- a/Modules/Platform/Windows-MSVC.cmake
+++ b/Modules/Platform/Windows-MSVC.cmake
@@ -231,7 +231,6 @@ macro(__windows_compiler_msvc lang)
   set(CMAKE_${lang}_CREATE_ASSEMBLY_SOURCE
 CMAKE_${lang}_COMPILER ${CMAKE_START_TEMP_FILE} 
${CMAKE_CL_NOLOGO}${_COMPILE_${lang}} FLAGS DEFINES /FoNUL /FAs 
/FaASSEMBLY_SOURCE /c SOURCE${CMAKE_END_TEMP_FILE})

-  set(CMAKE_${lang}_COMPILER_LINKER_OPTION_FLAG_EXECUTABLE /link)
   set(CMAKE_${lang}_USE_RESPONSE_FILE_FOR_OBJECTS 1)
   set(CMAKE_${lang}_LINK_EXECUTABLE

-Brad
--

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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers


Re: [cmake-developers] Warnings with CMake 2.8.12-rc3 driving dashboards

2013-09-11 Thread Stephen Kelly
Brad King wrote:

 Can you please teach it to print out the value of each of the two
 properties so we have some hint about what is different?
 

I pushed print-CMP0022-content-conflict and merged it to next.

Thanks,

Steve.


--

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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers