Re: [cmake-developers] find_package without REQUIRED can cause fatal error now

2013-04-24 Thread Chris
chris developerchris@... writes: I should add the commandline for this build emerge debug: cmake -G MinGW Makefiles -DBUILD_doc=OFF -DCMAKE_INSTALL_PREFIX=r:/ -DCMAKE_INCLUDE_PATH=r:/include -DCMAKE_LIBRARY_PATH=r:/lib -DCMAKE_PREFIX_PATH=r:/ -DCMAKE_BUILD_TYPE=RelWithDebInfo q:\kate emerge

Re: [cmake-developers] find_package without REQUIRED can cause fatal error now

2013-04-24 Thread Brad King
On 04/24/2013 02:45 AM, chris wrote: when building packages I get errors like the following... CMake Error at /lib/cmake/KActivities/KActivitiesLibraryTargets.cmake:68 (message): [snip] Call Stack (most recent call first): /lib/cmake/KActivities/KActivitiesConfig.cmake:18 (include)

Re: [cmake-developers] find_package without REQUIRED can cause fatal error now

2013-04-24 Thread Brad King
On 04/24/2013 09:29 AM, DeveloperChris wrote: I had added some message lines to try and work out how it functions part of the log snippet I actually cut out was the following... | ||-- Found automoc4: R:/bin/automoc4.exe|| ||-- the import prefix derived from

Re: [cmake-developers] find_package without REQUIRED can cause fatal error now

2013-04-24 Thread Alexander Neundorf
On Wednesday 24 April 2013, Brad King wrote: On 04/24/2013 09:29 AM, DeveloperChris wrote: I had added some message lines to try and work out how it functions part of the log snippet I actually cut out was the following... ||-- Found automoc4: R:/bin/automoc4.exe|| ||-- the import

Re: [cmake-developers] find_package without REQUIRED can cause fatal error now

2012-07-31 Thread Bill Hoffman
On 7/30/2012 3:38 PM, Alexander Neundorf wrote: The idea was that this is a sign of a broken install. Can you uninstall the vtk-devel package (does this exist ?), because it is broken, i.e. does not contain everything is claims to ? I understand that this is maybe not expected behaviour for an

Re: [cmake-developers] find_package without REQUIRED can cause fatal error now

2012-07-31 Thread Alexander Neundorf
On Tuesday 31 July 2012, Bill Hoffman wrote: On 7/30/2012 3:38 PM, Alexander Neundorf wrote: The idea was that this is a sign of a broken install. Can you uninstall the vtk-devel package (does this exist ?), because it is broken, i.e. does not contain everything is claims to ? I

Re: [cmake-developers] find_package without REQUIRED can cause fatal error now

2012-07-31 Thread David Cole
On Tue, Jul 31, 2012 at 2:30 PM, Alexander Neundorf neund...@kde.org wrote: On Tuesday 31 July 2012, Bill Hoffman wrote: On 7/30/2012 3:38 PM, Alexander Neundorf wrote: The idea was that this is a sign of a broken install. Can you uninstall the vtk-devel package (does this exist ?),

Re: [cmake-developers] find_package without REQUIRED can cause fatal error now

2012-07-31 Thread Brad King
On 07/31/2012 02:30 PM, Alexander Neundorf wrote: On Tuesday 31 July 2012, Bill Hoffman wrote: I wonder if we could just disable fatal error at the start of find_package optional. Then turn it back on at the end. As I pointed out elsewhere in this thread fatal errors are just one way a failure

Re: [cmake-developers] find_package without REQUIRED can cause fatal error now

2012-07-30 Thread David Cole
Seems like message(FATAL_ERROR/SEND_ERROR needs to know whether it's in the middle of a find_package( OPTIONAL ) call or not. What would happen if the only line of code in a project config file is: message(FATAL_ERROR not implemented yet) And then somebody did a find_package(OPTIONAL on that

Re: [cmake-developers] find_package without REQUIRED can cause fatal error now

2012-07-30 Thread Brad King
On 07/30/2012 10:41 AM, David Cole wrote: Seems like message(FATAL_ERROR/SEND_ERROR needs to know whether it's in the middle of a find_package( OPTIONAL ) call or not. What would happen if the only line of code in a project config file is: message(FATAL_ERROR not implemented yet) And

Re: [cmake-developers] find_package without REQUIRED can cause fatal error now

2012-07-30 Thread Alexander Neundorf
On Monday 30 July 2012, Bill Hoffman wrote: Alex added this check: http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=a2be068c It guards against broken installations of packages. This is a good thing. However, it is causing a failed test: