View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0015480CMakeCMakepublic2015-03-27 17:072015-11-02 09:13
ReporterFelix Schwitzer 
Assigned To 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0015480: UseSWIG.cmake warns about CMP0057
DescriptionWhen calling swig_add_module (from module UseSWIG) with the same interface file for different scripting languages, cmake emits a warning about CMP0057 as swig_add_module adds the interface file as MAIN_DEPENDENCY to the generated wrapper file.
This happens with an actual cmake from master (7910cb7);
see http://article.gmane.org/gmane.comp.programming.tools.cmake.devel/12835 [^]

I added a minimal example to reproduce and a patch
Steps To Reproduceunpack the attached example tst.tgz and change into the directory tst, then
    mkdir build && cd build && cmake ..

Without patching cmake fails as CMP0057 is set to new. With the patch the command runs fine.

On a system with swig, ruby-dev and python-dev installed, compilation should run fine and also the dependencies are kept right:
    make
-> runs swig and compiles the wrappers
    touch ../ltt.i && make
-> runs swig again and compiles the wrappers
TagsNo tags attached.
Attached Filestgz file icon tst.tgz [^] (1,463 bytes) 2015-03-27 17:07
patch file icon 0001-UseSWIG-fix-warning-about-CMP0057.patch [^] (1,101 bytes) 2015-03-27 17:08 [Show Content]

 Relationships

  Notes
(0038362)
Brad King (manager)
2015-03-30 09:39

Applied, thanks:

 UseSWIG: Do not use MAIN_DEPENDENCY on custom commands
 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=b372a99a [^]
(0038659)
Brad King (manager)
2015-04-30 10:33

It turns out that the original CMP0057 addressing MAIN_DEPENDENCY was not correct and has been reverted along with the change linked in 0015480:0038362, so this is now a non-issue:

 Revert "add_custom_command: Diagnose MAIN_DEPENDENCY limitation."
 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=32a2f414 [^]

(There is a new CMP0057 that means something else.)
(0039798)
Robert Maynard (manager)
2015-11-02 09:13

Closing resolved issues that have not been updated in more than 4 months.

 Issue History
Date Modified Username Field Change
2015-03-27 17:07 Felix Schwitzer New Issue
2015-03-27 17:07 Felix Schwitzer File Added: tst.tgz
2015-03-27 17:08 Felix Schwitzer File Added: 0001-UseSWIG-fix-warning-about-CMP0057.patch
2015-03-30 09:39 Brad King Note Added: 0038362
2015-03-30 09:39 Brad King Status new => resolved
2015-03-30 09:39 Brad King Resolution open => fixed
2015-03-30 09:39 Brad King Fixed in Version => CMake 3.3
2015-03-30 09:39 Brad King Target Version => CMake 3.3
2015-04-30 10:33 Brad King Note Added: 0038659
2015-04-30 10:33 Brad King Fixed in Version CMake 3.3 =>
2015-04-30 10:33 Brad King Target Version CMake 3.3 =>
2015-11-02 09:13 Robert Maynard Note Added: 0039798
2015-11-02 09:13 Robert Maynard Status resolved => closed


Copyright © 2000 - 2018 MantisBT Team