View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0015480 | CMake | CMake | public | 2015-03-27 17:07 | 2015-11-02 09:13 | ||||
Reporter | Felix Schwitzer | ||||||||
Assigned To | |||||||||
Priority | normal | Severity | minor | Reproducibility | have not tried | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | |||||||||
Target Version | Fixed in Version | ||||||||
Summary | 0015480: UseSWIG.cmake warns about CMP0057 | ||||||||
Description | When 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 Reproduce | unpack 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 | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | tst.tgz [^] (1,463 bytes) 2015-03-27 17:07 0001-UseSWIG-fix-warning-about-CMP0057.patch [^] (1,101 bytes) 2015-03-27 17:08 [Show Content] | ||||||||
Relationships | |
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. |
Notes |
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 |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |