View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0014398 | CMake | Modules | public | 2013-09-10 23:07 | 2014-03-05 09:58 | ||||
Reporter | Jean-Christophe Fillion-Robin | ||||||||
Assigned To | Brad King | ||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | |||||||||
Target Version | CMake 2.8.12 | Fixed in Version | CMake 2.8.12 | ||||||
Summary | 0014398: FindPNG - Fixes for 0014301 breaks backward compatibility | ||||||||
Description | For details see http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=22411 [^] The module FindPNG should still be able to understand the option -DPNG_LIBRARY | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | ![]() | ||||||||
Relationships | ||||||
|
Relationships |
Notes | |
(0033818) Brad King (manager) 2013-09-13 11:57 |
Please review and test "0001-FindPNG-Honor-old-PNG_LIBRARY-if-provided-14398.patch". |
(0033820) Rolf Eike Beer (developer) 2013-09-13 13:03 |
That would break as it doesn't cache that variable, no? So if it is once specified on the command line it is taken, on the next CMake run it's not set anymore and the default search is started, changing the value. |
(0033821) Brad King (manager) 2013-09-13 13:06 |
If it is specified on the command line with -D then it will be cached and used every time. |
(0033822) Brad King (manager) 2013-09-13 13:07 |
Re 0014398:0033821: To clarify, "cmake -Dfoo=bar" will put foo in the cache whether it is referenced or used by the project or not. |
(0033828) Brad King (manager) 2013-09-16 08:58 |
Change applied here for next RC: http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=68160448 [^] I hope it works since I haven't heard back here. |
(0033830) Rolf Eike Beer (developer) 2013-09-16 09:50 |
You probably want to move the next 3 lines (2 lines comment and unset(PNG_FOUND)) into the if, too. |
(0033831) Brad King (manager) 2013-09-16 09:53 |
Re 0014398:0033830: The PNG_FOUND still needs to be set below by FPHSA. The other two unset variables set prior to the block in question so they still need to be unset since they are only for local use. |
(0033832) Rolf Eike Beer (developer) 2013-09-16 12:46 |
Read the comment ;) PNG_FOUND is unset because it was set by select_library_configurations(), but we want it to be set by FPHSA later. |
(0033833) Brad King (manager) 2013-09-16 12:49 |
Re 0014398:0033832: Who cares what set PNG_FOUND before? It still needs to be unset so that FPHSA will set it. |
(0033835) Rolf Eike Beer (developer) 2013-09-16 14:17 |
Go for it! |
(0033836) Jean-Christophe Fillion-Robin (reporter) 2013-09-16 15:34 |
Just tested and it works as expected. Thanks for working on this. |
(0035259) Robert Maynard (manager) 2014-03-05 09:58 |
Closing resolved issues that have not been updated in more than 4 months |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2013-09-10 23:07 | Jean-Christophe Fillion-Robin | New Issue | |
2013-09-11 08:29 | Brad King | Assigned To | => Rolf Eike Beer |
2013-09-11 08:29 | Brad King | Status | new => assigned |
2013-09-13 11:57 | Brad King | File Added: 0001-FindPNG-Honor-old-PNG_LIBRARY-if-provided-14398.patch | |
2013-09-13 11:57 | Brad King | Note Added: 0033818 | |
2013-09-13 11:58 | Brad King | Target Version | => CMake 2.8.12 |
2013-09-13 11:58 | Brad King | Description Updated | |
2013-09-13 13:03 | Rolf Eike Beer | Note Added: 0033820 | |
2013-09-13 13:06 | Brad King | Note Added: 0033821 | |
2013-09-13 13:07 | Brad King | Note Added: 0033822 | |
2013-09-16 08:58 | Brad King | Note Added: 0033828 | |
2013-09-16 09:50 | Rolf Eike Beer | Note Added: 0033830 | |
2013-09-16 09:53 | Brad King | Note Added: 0033831 | |
2013-09-16 12:46 | Rolf Eike Beer | Note Added: 0033832 | |
2013-09-16 12:49 | Brad King | Note Added: 0033833 | |
2013-09-16 14:17 | Rolf Eike Beer | Note Added: 0033835 | |
2013-09-16 14:17 | Rolf Eike Beer | Assigned To | Rolf Eike Beer => Brad King |
2013-09-16 15:34 | Jean-Christophe Fillion-Robin | Note Added: 0033836 | |
2013-09-16 16:04 | Brad King | Status | assigned => resolved |
2013-09-16 16:04 | Brad King | Resolution | open => fixed |
2013-09-16 16:04 | Brad King | Fixed in Version | => CMake 2.8.12 |
2013-09-16 16:05 | Brad King | Relationship added | related to 0014416 |
2014-03-05 09:58 | Robert Maynard | Note Added: 0035259 | |
2014-03-05 09:58 | Robert Maynard | Status | resolved => closed |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |