Brad King wrote:

> On 01/10/2013 12:20 PM, Stephen Kelly wrote:
>> Looks like I broke it. I was sure I tested it before pushing. Maybe too
>> much merging and rebasing going on for me. Looking into it now.
> 
> Okay, the latest topic head works well:
> 
>  http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=77d26467
> 
> However, the ExportImport test case only covers install(EXPORT) and not
> export().  The implementation tested is in cmExportFileGenerator which
> covers both types of exports.
> 
> I've merged the topic to master, but please extend everything you've
> added to ExportImport recently (in this topic and others) to cover the
> export()-from-build-tree case too.

Thanks, I've added a test for this latest part now. I'll review it later to 
see what else is missing.

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

Reply via email to