Re: [cmake-developers] [PATCH] Extend copy and copy_if_different commands with support multiple files

2015-12-04 Thread Brad King
On 12/04/2015 09:18 AM, Bartosz Kosiorek wrote: > Finally I manage to add wildcard support. > I have taken SimpleGlob.h library from: > https://github.com/brofield/simpleopt We don't want to do wildcard expansion in CMake commands. We want to leave it up to the shell to expand wildcards on the

Re: [cmake-developers] [PATCH] Extend copy and copy_if_different commands with support multiple files

2015-12-04 Thread Bartosz Kosiorek
Thanks Brad. Unfortunately we cannot use file(GLOB), because it is executed during generate step, and we would like to copy files which is created during Build step. Currently for filtering these files, we are using "cmake -E tar" to filter these files (so it packing and upacking). I think it is

Re: [cmake-developers] [PATCH] Extend copy and copy_if_different commands with support multiple files

2015-12-04 Thread Brad King
On 12/04/2015 11:15 AM, Bartosz Kosiorek wrote: > we cannot use file(GLOB), because it is executed during generate step, > and we would like to copy files which is created during Build step. The custom commands always execute inside a shell that can do glob expansion. Alternatively, use a "cmake

Re: [cmake-developers] [PATCH] Extend copy and copy_if_different commands with support multiple files

2015-12-03 Thread Brad King
On 12/02/2015 07:05 PM, Bartosz Kosiorek wrote: > This patch allows to use multiple files in commands "copy" and > "copy_if_different". Good, thanks. Please also extend the Tests/RunCMake/CommandLine test to cover this these operations. Thanks, -Brad -- Powered by www.kitware.com Please

Re: [cmake-developers] [PATCH] Extend copy and copy_if_different commands with support multiple files

2015-12-03 Thread Brad King
On 12/03/2015 11:05 AM, Bartosz Kosiorek wrote: > After every step I will need to clean it up . If you add these as cases in the RunCMake.CommandLine test then each one can get its own directory and the RunCMake infrastructure will take care of cleaning it up for each run. -Brad -- Powered by

Re: [cmake-developers] [PATCH] Extend copy and copy_if_different commands with support multiple files

2015-12-03 Thread Bartosz Kosiorek
What I'm doing wrong ? From: Brad King <brad.k...@kitware.com> Sent: Thursday, December 3, 2015 5:38 PM To: Bartosz Kosiorek Cc: cmake-developers@cmake.org Subject: Re: [cmake-developers] [PATCH] Extend copy and copy_if_different commands with support multiple files On 12/03/2015 11:05 AM, Barto

Re: [cmake-developers] [PATCH] Extend copy and copy_if_different commands with support multiple files

2015-12-03 Thread Bartosz Kosiorek
: Re: [cmake-developers] [PATCH] Extend copy and copy_if_different commands with support multiple files On 12/02/2015 07:05 PM, Bartosz Kosiorek wrote: > This patch allows to use multiple files in commands "copy" and > "copy_if_different". Good, thanks. Please also exte

Re: [cmake-developers] [PATCH] Extend copy and copy_if_different commands with support multiple files

2015-12-03 Thread Bartosz Kosiorek
Hello. Here is the patch with tests. From: Brad King <brad.k...@kitware.com> Sent: Thursday, December 3, 2015 7:07 PM To: Bartosz Kosiorek Cc: cmake-developers@cmake.org Subject: Re: [cmake-developers] [PATCH] Extend copy and copy_if_different co

Re: [cmake-developers] [PATCH] Extend copy and copy_if_different commands with support multiple files

2015-12-03 Thread Brad King
On 12/03/2015 12:31 PM, Bartosz Kosiorek wrote: > When I'm trying to run test with wildcard: > run_cmake_command(E_copy-wildcard-source-files-target-is-directory > ${CMAKE_COMMAND} -E copy > ${RunCMake_copy_TEST_SOURCE_DIR}/directory1/* >

[cmake-developers] [PATCH] Extend copy and copy_if_different commands with support multiple files

2015-12-02 Thread Bartosz Kosiorek
Hi. This patch allows to use multiple files in commands "copy" and "copy_if_different". Input files could be merged with wildcards. For example: ./bin/cmake -E copy bin/* CMakeFiles/* file.tar dupajasia If wrong file path were provided as input then the error will be displayed, but rest