Re: [cmake-developers] iOS support

2014-09-29 Thread Ruslan Baratov via cmake-developers
On 23-Sep-14 19:42, Florent Castelli wrote: On 23 Sep 2014, at 16:56, Bill Hoffman wrote: That said it would be really cool to beef up the xcode support enough to be able to create an actual ios app. I have not dug into that enough. Should be able to do most of it with CMAKE_XCODE_ATTRIBUTE.

[cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-10-05 Thread Ruslan Baratov via cmake-developers
Hi, I have CMake modules that can share resources between different instances. For now I manage it by invoking `mkdir /lock-directory` and checking the result, then removing this directory so the next instance can do the modification. The problem occurs when somebody terminate CMake. Since I

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-10-06 Thread Ruslan Baratov via cmake-developers
On 06-Oct-14 16:54, Brad King wrote: On 10/05/2014 04:59 PM, Ruslan Baratov via cmake-developers wrote: So it can't be resolved without some low-level management. I'm thinking about `file` command sub-option like LOCK_DIRECTORY: `file(LOCK_DIRECTORY "/path/to/shared-dir&quo

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-10-08 Thread Ruslan Baratov via cmake-developers
On 07-Oct-14 16:25, Brad King wrote: On 10/07/2014 02:49 AM, Ruslan Baratov wrote: How it will looks like in terms of CMake code? That's what I'm asking you to think through and propose ;) Thanks, -Brad Okay :) I just not sure that I understand "to pass to some other process" goal correctly.

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-10-10 Thread Ruslan Baratov via cmake-developers
On 08-Oct-14 16:45, Brad King wrote: On 10/08/2014 07:52 AM, Ruslan Baratov wrote: Okay :) I just not sure that I understand "to pass to some other process" goal correctly. That was just an example of why one might want to drop management of the lock by CMake without actually unlocking it. Per

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-10-10 Thread Ruslan Baratov via cmake-developers
Brad K. (may be I've just understand them wrong). Ruslo Thanks, David C. On Fri, Oct 10, 2014 at 7:45 AM, Ruslan Baratov via cmake-developers wrote: On 08-Oct-14 16:45, Brad King wrote: On 10/08/2014 07:52 AM, Ruslan Baratov wrote: Okay :) I just not sure that I understand "to p

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-10-28 Thread Ruslan Baratov via cmake-developers
On 13-Oct-14 18:39, Brad King wrote: With all the above in mind, please brainstorm and propose a more complete signature set. You can use the keyword/value pairing common in other commands to avoid needing a positional argument for every value. What do you think about this: file( LOCK

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-10-31 Thread Ruslan Baratov via cmake-developers
Does anybody ready to implement it or you want me to send the patches? Ruslo On 29-Oct-14 16:48, Brad King wrote: On 10/28/2014 04:28 PM, Ruslan Baratov wrote: What do you think about this: Thanks for drafting the signature. file( LOCK [DIRECTORY] # if present locked file will

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-11-10 Thread Ruslan Baratov via cmake-developers
Done On 31-Oct-14 16:39, Brad King wrote: On 10/31/2014 09:07 AM, Ruslan Baratov wrote: Does anybody ready to implement it or you want me to send the patches? Please work on the patches. You can use "git format-patch" to format them and post here either inline or as attachments. Thanks, -Bra

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-11-10 Thread Ruslan Baratov via cmake-developers
Done On 31-Oct-14 16:39, Brad King wrote: On 10/31/2014 09:07 AM, Ruslan Baratov wrote: Does anybody ready to implement it or you want me to send the patches? Please work on the patches. You can use "git format-patch" to format them and post here either inline or as attachments. Thanks, -Bra

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-11-13 Thread Ruslan Baratov via cmake-developers
On 11-Nov-14 21:28, Brad King wrote: * All headers and sources need to include cmStandardIncludes.h first, directly or indirectly through other headers. It can affect the way certain standard library headers are preprocessed on some platforms, and needs to be consistent across all trans

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-11-14 Thread Ruslan Baratov via cmake-developers
ConvertToWindowsExtendedPath fix + tests >From e994378d61e64136043d9db614762e79927766cb Mon Sep 17 00:00:00 2001 From: Ruslan Baratov Date: Fri, 14 Nov 2014 21:09:35 +0400 Subject: [PATCH 1/6] Add class cmFileLockResult --- Source/cmFileLockResult.cxx | 111

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-11-14 Thread Ruslan Baratov via cmake-developers
On 14-Nov-14 22:58, Brad King wrote: - Please add more test cases covering all the file(LOCK) argument processing error messages. Done. Also I've found parse issue which is based on `sscanf` behaviour, e.g. string '123xyz' will be successfully parsed as a integer (%d). Same issue for example

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-11-17 Thread Ruslan Baratov via cmake-developers
On 17-Nov-14 18:06, Brad King wrote: Yes, please change to that so a proper error can be produced for the new API when "123xyz" is given as a timeout, for example. Function added cmSystemTools::StringToInt Please look at adding a case to the test suite for this. An outer process could take th

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-11-25 Thread Ruslan Baratov via cmake-developers
On 25-Nov-14 21:29, Brad King wrote: * The background.(bat|sh) files need to have quoting to work with spaces in the path. Please try running tests with a source/build tree with spaces. * Instead of background scripts, can you have a parent process take a lock, run a child with a timed

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-12-05 Thread Ruslan Baratov via cmake-developers
On 01-Dec-14 23:32, Brad King wrote: On 12/01/2014 03:26 PM, Stephen Kelly wrote: Existing uses of strtol also check errno. I guess your implementation should too. Yes, IIUC it is a range check on whether the value can be represented. I think also you need to zero errno before strtol since errn

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-12-05 Thread Ruslan Baratov via cmake-developers
On 03-Dec-14 18:10, Brad King wrote: On 12/01/2014 10:52 AM, Brad King wrote: cmSystemTools: Add StringToInt helper http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=5121f118 file: Add LOCK subcommand to do file and directory locking http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=f

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-12-05 Thread Ruslan Baratov via cmake-developers
On 05-Dec-14 17:12, Brad King wrote: On 12/05/2014 09:10 AM, Ruslan Baratov wrote: file: Add LOCK subcommand to do file and directory locking http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=e6db4c5a Great, thank you! So will this be applied to cmake 3.1.0? No, sorry. The feature dea

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-12-05 Thread Ruslan Baratov via cmake-developers
On 05-Dec-14 17:09, Brad King wrote: On 12/05/2014 09:03 AM, Ruslan Baratov wrote: Actually why not use 'strtoll' and 'long long' ? I'm not sure that function or type exists portably on some of the older host platforms we support. The "long" type should be plenty big for timeout values, and th

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-12-17 Thread Ruslan Baratov via cmake-developers
On 17-Dec-14 21:11, Matthew Woehlke wrote: On 2014-10-13 10:39, Brad King wrote: On 10/10/2014 07:45 AM, Ruslan Baratov wrote: Locking directory is equivalent to locking file `cmake.lock` in directory "/path/to/shared/directory/": I think this can be activated buy a DIRECTORY option. Why do w

Re: [cmake-developers] New command 'file(LOCK_DIRECTORY ...)'

2014-12-18 Thread Ruslan Baratov via cmake-developers
On 18-Dec-14 01:49, Matthew Woehlke wrote: On 2014-12-17 16:58, Ruslan Baratov via cmake-developers wrote: On 17-Dec-14 21:11, Matthew Woehlke wrote: On 2014-10-13 10:39, Brad King wrote: On 10/10/2014 07:45 AM, Ruslan Baratov wrote: Locking directory is equivalent to locking file

[cmake-developers] BundleUtilities check exit code

2015-02-09 Thread Ruslan Baratov via cmake-developers
Hi, This patch let do the check that exit code is 0, i.e. install_name_tool exits successfully. Ruslan From 74a52160f7b74907acfcbca91b925ea5589019b7 Mon Sep 17 00:00:00 2001 From: Ruslan Baratov Date: Mon, 9 Feb 2015 20:09:41 +0300 Subject: [PATCH] fixup_bundle: check execute_process succeed

Re: [cmake-developers] BundleUtilities check exit code

2015-02-11 Thread Ruslan Baratov via cmake-developers
On 10-Feb-15 19:01, Brad King wrote: On 02/09/2015 12:17 PM, Ruslan Baratov via cmake-developers wrote: This patch let do the check that exit code is 0, i.e. install_name_tool exits successfully. Applied with minor refactoring: BundleUtilities: Teach fixup_bundle to check install_name_tool

[cmake-developers] [patch] One more pattern for extracting file name from URL

2016-07-01 Thread Ruslan Baratov via cmake-developers
Hi, With attached patch it's possible to extract name from URLs like 'https://.../archive.tar.gz?a=x&b=y'. Ruslo >From 0bab30e859541dde51e9e21d3a22ec80a649a30a Mon Sep 17 00:00:00 2001 From: Ruslan Baratov Date: Fri, 1 Jul 2016 14:55:29 +0300 Subject: [PATCH] Add pattern for extracting file n

Re: [cmake-developers] [patch] One more pattern for extracting file name from URL

2016-07-01 Thread Ruslan Baratov via cmake-developers
On 01-Jul-16 15:58, Brad King wrote: On 07/01/2016 08:01 AM, Ruslan Baratov via cmake-developers wrote: With attached patch it's possible to extract name from URLs like 'https://.../archive.tar.gz?a=x&b=y'. Thanks. elseif(NOT "${fname}" MATCHES "

Re: [cmake-developers] [patch] One more pattern for extracting file name from URL

2016-07-01 Thread Ruslan Baratov via cmake-developers
On 01-Jul-16 16:04, Ben Boeckel wrote: On Fri, Jul 01, 2016 at 08:58:16 -0400, Brad King wrote: Also, the `([^/]*)\\?.*` part of the regex should be more like `([^/?]*)\\?.*` to avoid eagerly matching early `?`. This should also probably skip '#' characters for URLs with anchors. --Ben 'frage

Re: [cmake-developers] [patch] Document -H and -B

2016-07-05 Thread Ruslan Baratov via cmake-developers
On 01-Jul-16 15:58, Brad King wrote: * Allow the form `-H -B ` with the new names. I don't see the point of introducing new options. Developers will still use -H/-B because it's backward compatible, well known and will work 100% because of internal CMake testing. That said if you want to imp

Re: [cmake-developers] [patch] One more pattern for extracting file name from URL

2016-07-05 Thread Ruslan Baratov via cmake-developers
ch attached. Ruslo On 01-Jul-16 15:01, Ruslan Baratov via cmake-developers wrote: Hi, With attached patch it's possible to extract name from URLs like 'https://.../archive.tar.gz?a=x&b=y'. Ruslo >From 5284eedbd1b038aab2eab73252eb41914afb14e1 Mon Sep 17 00:00:00 2

Re: [cmake-developers] [patch] One more pattern for extracting file name from URL

2016-07-05 Thread Ruslan Baratov via cmake-developers
By default not, goes to -prefix/src/archive.tar . On 05-Jul-16 16:49, Ben Boeckel wrote: On Tue, Jul 05, 2016 at 16:26:54 +0300, Ruslan Baratov via cmake-developers wrote: + # Do not put warning message here. If everything works OK it will + # only distract. If unpack will

Re: [cmake-developers] [patch] One more pattern for extracting file name from URL

2016-07-05 Thread Ruslan Baratov via cmake-developers
On 05-Jul-16 18:50, Ben Boeckel wrote: On Tue, Jul 05, 2016 at 18:43:25 +0300, Ruslan Baratov wrote: By default not, goes to -prefix/src/archive.tar . Right, but if a download directory is set, this won't work. Can the project name at least be added to the name? If user are forcing directory th

Re: [cmake-developers] [patch] One more pattern for extracting file name from URL

2016-07-05 Thread Ruslan Baratov via cmake-developers
On 05-Jul-16 20:31, Ben Boeckel wrote: On Tue, Jul 05, 2016 at 19:49:46 +0300, Ruslan Baratov wrote: Archive can be unpacked by "cmake -E tar xf" which detect type automatically as far as I understand. At least this one used internally by ExternalProject (see the testing I've made before). CMak

Re: [cmake-developers] [patch] One more pattern for extracting file name from URL

2016-07-05 Thread Ruslan Baratov via cmake-developers
On 05-Jul-16 22:03, Brad King wrote: On 07/01/2016 03:35 PM, Brad King wrote: I don't think we should duplicate the "(\\.|=)(7z|tar|tar\\.bz2|tar\\.gz|tar\\.xz|tbz2|tgz|txz|zip)$" expression. The stripping of ?.* can be done earlier, or done as part of the main match. Please try this comm

Re: [cmake-developers] [patch] One more pattern for extracting file name from URL

2016-07-05 Thread Ruslan Baratov via cmake-developers
On 05-Jul-16 23:26, Brad King wrote: On 07/05/2016 04:13 PM, Ruslan Baratov wrote: Works fine on my tests. Thanks for testing! However I still vote for using default value for 'fname' instead of FATAL_ERROR. I think that behavior would be surprising. Not such surprising as some error in inte

Re: [cmake-developers] ExternalProject default downloaded file name

2016-07-06 Thread Ruslan Baratov via cmake-developers
Attaching rebased patch. Thanks! On 06-Jul-16 17:36, Brad King wrote: On 07/05/2016 05:14 PM, Ruslan Baratov wrote: if it will work in most cases then why not? Most users will not even notice this. Okay. The changes for the new pattern are now in `master` and are worthwhile regardless of hav

[cmake-developers] Downloading file from S3 private bucket

2016-09-13 Thread Ruslan Baratov via cmake-developers
Hi, Currently I'm investigating possibilities of downloading files from private S3 bucket using CMake. The only real issue I see for now is the computing of HMAC + SHA1. This look like a work for some new cryptographic module in family of 'string' functions similar to 'string(MD5|SHA1|...)'.

Re: [cmake-developers] Downloading file from S3 private bucket

2016-09-13 Thread Ruslan Baratov via cmake-developers
On 13-Sep-16 15:58, Brad King wrote: On 09/13/2016 07:42 AM, Ruslan Baratov via cmake-developers wrote: Will it be okay if CMake will optionally depend on OpenSSL? Or other third party crypto library? Currently the only place that needs it is through `curl`, and that uses either OpenSSL or the

Re: [cmake-developers] Downloading file from S3 private bucket

2016-09-19 Thread Ruslan Baratov via cmake-developers
quests/108 Ruslo On 13-Sep-16 16:28, Ruslan Baratov wrote: On 13-Sep-16 15:58, Brad King wrote: On 09/13/2016 07:42 AM, Ruslan Baratov via cmake-developers wrote: Will it be okay if CMake will optionally depend on OpenSSL? Or other third party crypto library? Currently the only place that needs it

[cmake-developers] Remove RPATH on install

2016-09-28 Thread Ruslan Baratov via cmake-developers
Hi, I want to link the library to all executables in project so I'm adding ` -Wl,-rpath,` to CMAKE_EXE_LINKER_FLAGS. That works fine however after install step this RPATH was not removed and full path to remains as part of executable in LC_RPATH. As far as I understand there is no such varia

Re: [cmake-developers] Remove RPATH on install

2016-09-28 Thread Ruslan Baratov via cmake-developers
On 28-Sep-16 23:34, Brad King wrote: On 09/28/2016 04:09 PM, Ruslan Baratov via cmake-developers wrote: ` -Wl,-rpath,` to CMAKE_EXE_LINKER_FLAGS. That adds it in a way that hides its presence from CMake. CMake has its own way to specify RPATH entries in the build tree. It will replace the

[cmake-developers] Android variables

2016-11-10 Thread Ruslan Baratov via cmake-developers
Hi, I wonder if it's possible to introduce next variables describing Android tools: * C preprocessor. Similar to CMAKE_CXX_COMPILER the variable that will contain the path to preprocessor. Example: * CMAKE_CXX_COMPILER = /toolchains/arm-linux-androideabi-4.9/prebuilt/linux-x86_64/bin/arm-linux-an

Re: [cmake-developers] Android variables

2016-11-10 Thread Ruslan Baratov via cmake-developers
||string(REGEX REPLACE "-$" "" ||ANDROID_TOOLCHAIN_MACHINE_NAME|| "${||ANDROID_TOOLCHAIN_MACHINE_NAME||}") || ||? Ruslo ||On 11-Nov-16 00:29, Brad King wrote: > On 11/10/2016 10:05 AM, Ruslan Baratov via cmake-developers wrote: >> variables describing Android t

Re: [cmake-developers] Android variables

2016-11-11 Thread Ruslan Baratov via cmake-developers
On 10-Nov-16 23:05, Ruslan Baratov via cmake-developers wrote: > * CMAKE_CXX_COMPILER = > /toolchains/${ANDROID_TOOLCHAIN_MACHINE_NAME}-4.9/prebuilt/linux-x86_64/bin/${ANDROID_TOOLCHAIN_MACHINE_NAME}-g++ Actually there are more variables here, ANDROID_COMPILER_VERSI

Re: [cmake-developers] Android variables

2016-11-11 Thread Ruslan Baratov via cmake-developers
On 11-Nov-16 21:09, Brad King wrote: > On 11/10/2016 10:46 PM, Ruslan Baratov wrote: >> the value >> "${CMAKE_CXX_ANDROID_TOOLCHAIN_PREFIX}cpp${CMAKE_CXX_ANDROID_TOOLCHAIN_SUFFIX}" > This expression is the documented way to get the paths to the Android tools. > I see no reason to add a pre-evaluat

Re: [cmake-developers] Android variables

2016-11-11 Thread Ruslan Baratov via cmake-developers
On 12-Nov-16 08:21, Florent Castelli wrote: > On 10/11/2016 16:05, Ruslan Baratov via cmake-developers wrote: >> Hi, >> >> I wonder if it's possible to introduce next variables describing Android >> tools: >> * C preprocessor. Similar to CMAKE_CXX_COMPILER

Re: [cmake-developers] Android variables

2016-11-12 Thread Ruslan Baratov via cmake-developers
On 12-Nov-16 19:09, Florent Castelli wrote: > On 12/11/2016 06:53, Ruslan Baratov wrote: >> On 12-Nov-16 08:21, Florent Castelli wrote: >>> On 10/11/2016 16:05, Ruslan Baratov via cmake-developers wrote: >>>> Hi, >>>> >>>> I wonder i

Re: [cmake-developers] Android variables

2016-11-12 Thread Ruslan Baratov via cmake-developers
On 13-Nov-16 00:48, Florent Castelli wrote: > On 12/11/2016 13:35, Ruslan Baratov wrote: >> On 12-Nov-16 19:09, Florent Castelli wrote: >>> On 12/11/2016 06:53, Ruslan Baratov wrote: >>>> On 12-Nov-16 08:21, Florent Castelli wrote: >>>>> On 10/11/20

[cmake-developers] find_dependency with optional dependencies

2015-04-21 Thread Ruslan Baratov via cmake-developers
Hi, I'm trying to figure out what is the best way to use 'find_dependency' if dependent package is optional (e.g. can be controlled by some build option): # CMakeLists.txt option(BOO_WITH_FOO "Build Boo with Foo" ON) if(BOO_WITH_FOO) find_package(Foo CONFIG REQUIRED) target_link_libraries

Re: [cmake-developers] find_package config mode search path rule #5 error prone

2015-05-07 Thread Ruslan Baratov via cmake-developers
On 07-May-15 00:00, Tamás Kenéz wrote: Hi, The search path rule #5 reads: Search project build trees recently configured in a cmake-gui(1). This can be skipped if NO_CMAKE_BUILDS_PATH is passed. It is intended for the case when a user is building multiple dependent projects one after another

Re: [cmake-developers] [MODERN] CMake 3.3 vs Qt 5.5 vs MSVC2015

2015-07-23 Thread Ruslan Baratov via cmake-developers
On 23-Jul-15 12:32, Konstantin Podsvirov wrote: Hi dear CMake developers! Meet modern CMake built on MSVC2015 c QtDialog based on Qt 5.5 from today :-) Windows 32bit: http://ifw.podsvirov.pro/cmake/cmake-master-win32-online.exe Windows 64bit: http://ifw.podsvirov.pro/cmake/cmake-master-win64

[cmake-developers] Fixing bug #12506 (iOS install)

2015-08-06 Thread Ruslan Baratov via cmake-developers
Hi, I'm thinking about fixing bug http://public.kitware.com/Bug/view.php?id=12506 and before I go deeper in CMake internals just want to ask is there any work-in-progress attempts already or hints/directions of how it can be done? Thanks, Ruslo -- Powered by www.kitware.com Please keep mes

Re: [cmake-developers] Fixing bug #12506 (iOS install)

2015-08-07 Thread Ruslan Baratov via cmake-developers
On 07-Aug-15 11:31, Gregor Jasny wrote: Hi, On 07/08/15 00:43, Ruslan Baratov via cmake-developers wrote: Hi, I'm thinking about fixing bug http://public.kitware.com/Bug/view.php?id=12506 and before I go deeper in CMake internals just want to ask is there any work-in-progress attempts al

Re: [cmake-developers] Fixing bug #12506 (iOS install)

2015-08-12 Thread Ruslan Baratov via cmake-developers
On 07-Aug-15 01:43, Ruslan Baratov via cmake-developers wrote: Hi, I'm thinking about fixing bug http://public.kitware.com/Bug/view.php?id=12506 and before I go deeper in CMake internals just want to ask is there any work-in-progress attempts already or hints/directions of how it can be

Re: [cmake-developers] Fixing bug #12506 (iOS install)

2015-08-13 Thread Ruslan Baratov via cmake-developers
On 13-Aug-15 08:46, Gregor Jasny wrote: Hi, On 13/08/15 01:44, Ruslan Baratov via cmake-developers wrote: Sending patches with fix. Now it's possible to install simulator libraries by: > cmake --build _builds --config Release --target install -- -sdk iphonesimulator and device libr

Re: [cmake-developers] Setting up environment using ExternalProject_Add

2015-08-18 Thread Ruslan Baratov via cmake-developers
On 11-Aug-15 07:49, James Johnston wrote: I have found it annoyingly difficult to set up an environment for a build when using ExternalProject_Add. I can confirm that there is an additional mess with the environment sometimes and not just about setting it (like vcvarsall.bat for Visual Studio

Re: [cmake-developers] Fixing bug #12506 (iOS install)

2015-08-27 Thread Ruslan Baratov via cmake-developers
On 26-Aug-15 22:53, Gregor Jasny wrote: Hallo, On 13/08/15 12:56, Ruslan Baratov wrote: On 13-Aug-15 08:46, Gregor Jasny wrote: On 13/08/15 01:44, Ruslan Baratov via cmake-developers wrote: Sending patches with fix. Now it's possible to install simulator libraries by: > cmake

[cmake-developers] Patch: install universal iOS libraries

2015-09-24 Thread Ruslan Baratov via cmake-developers
Hi, Patches help to install universal iOS (device + simulator) libraries by triggering some extra instructions (build + fuse) after "regular" library installation finished. This behavior controlled by CMake variable CMAKE_IOS_INSTALL_UNIVERSAL_LIBS. = Example = > cat CMakeLists.txt cmake_mi

Re: [cmake-developers] Patch: install universal iOS libraries

2015-10-07 Thread Ruslan Baratov via cmake-developers
On 07-Oct-15 16:57, Gregor Jasny wrote: Hello, thank you for working on this. This is really hairy stuff. On 24/09/15 11:10, Ruslan Baratov via cmake-developers wrote: Patches help to install universal iOS (device + simulator) libraries by triggering some extra instructions (build + fuse

Re: [cmake-developers] [CMake 0015769]: OS X: Filesystem timestamp checks use only 1s resolution

2015-10-09 Thread Ruslan Baratov via cmake-developers
On 08-Oct-15 21:03, Mantis Bug Tracker wrote: The following issue has been ASSIGNED. == https://cmake.org/Bug/view.php?id=15769 == Reported By:Ru

Re: [cmake-developers] [CMake 0015769]: OS X: Filesystem timestamp checks use only 1s resolution

2015-10-09 Thread Ruslan Baratov via cmake-developers
On 10-Oct-15 01:00, Ruslan Baratov via cmake-developers wrote: On 08-Oct-15 21:03, Mantis Bug Tracker wrote: The following issue has been ASSIGNED. == https://cmake.org/Bug/view.php?id=15769

Re: [cmake-developers] automatically download library

2015-10-09 Thread Ruslan Baratov via cmake-developers
On 08-Oct-15 18:06, Marsel Galimullin wrote: The idea is that CMake has possibility to contact with package managers for download necessary libraries. If “find_package” can’t find a library, it need to call apt-get install. Example: auto_downland(apt-get) find_package (Boost COMPONENTS s

[cmake-developers] Filesystem timestamp checks

2015-10-15 Thread Ruslan Baratov via cmake-developers
Originally from: https://cmake.org/Bug/view.php?id=15769 Here is the scenario of not triggering rebuild of project while CMakeLists.txt changed: 1. Create CMakeLists.txt (modification time 100 units) 2. Run generate (create project from scratch): `cmake -H. -B_builds`. Takes 5 units, Makefile

Re: [cmake-developers] Filesystem timestamp checks

2015-10-19 Thread Ruslan Baratov via cmake-developers
On 19-Oct-15 19:30, Brad King wrote: On 10/15/2015 03:58 PM, Ruslan Baratov wrote: 3. Apply changes to CMakeLists.txt "immediately", CMakeLists.txt modification time 105 4. Run rebuild: `cmake --build _builds`. Since CMakeLists.txt (105) is not "newer" then Makefile (105) there will be no rege

Re: [cmake-developers] Filesystem timestamp checks

2015-10-20 Thread Ruslan Baratov via cmake-developers
On 20-Oct-15 21:42, Ben Boeckel wrote: On Tue, Oct 20, 2015 at 04:56:47 +0600, Ruslan Baratov via cmake-developers wrote: * I don't want to patch every script. To be precise I prefer to "Fix something in one place so it works 100% and forget about it" then "Every time I sta

[cmake-developers] [Patch] Documentation file(GLOB): ordering note

2015-10-20 Thread Ruslan Baratov via cmake-developers
Add a note that order of files is not defined >From 9cd9fbb12ef73b72526190573cba0392ead5fd9f Mon Sep 17 00:00:00 2001 From: Ruslan Baratov Date: Wed, 21 Oct 2015 02:53:37 +0600 Subject: [PATCH] Documentation: file(GLOB*) order is not guaranteed Since this command use 'readdir' under the hood the

Re: [cmake-developers] Filesystem timestamp checks

2015-10-20 Thread Ruslan Baratov via cmake-developers
On 21-Oct-15 03:25, Ben Boeckel wrote: On Wed, Oct 21, 2015 at 01:53:42 +0600, Ruslan Baratov wrote: On 20-Oct-15 21:42, Ben Boeckel wrote: On Tue, Oct 20, 2015 at 04:56:47 +0600, Ruslan Baratov via cmake-developers wrote: * I don't want to patch every script. To be precise I prefer to

Re: [cmake-developers] Patch: install universal iOS libraries

2015-10-25 Thread Ruslan Baratov via cmake-developers
On 24-Oct-15 02:56, Gregor Jasny wrote: On 08/10/15 02:37, Ruslan Baratov wrote: export CORRESPONDING_DEVICE_PLATFORM_NAME=iphoneos export CORRESPONDING_DEVICE_SDK_NAME=iphoneos9.0 export SDK_NAME=iphonesimulator9.0 Could you use those variables to avoid hardcoding iphoneos/simulator in the mod

Re: [cmake-developers] Patch: install universal iOS libraries

2015-10-27 Thread Ruslan Baratov via cmake-developers
On 28-Oct-15 00:56, Gregor Jasny wrote: Hello, I hope to have finished the review by end of this week. On 25/10/15 15:34, Ruslan Baratov wrote: + set(cmd lipo -info "${filename}") Minor nitpick: lipo should probably be called via xcrun. Otherwise it might not know how to handle new architect

Re: [cmake-developers] [PATCH] iOS Framework Bundle support

2015-11-16 Thread Ruslan Baratov via cmake-developers
On 13-Nov-15 20:59, Bartosz Kosiorek wrote: Hello. Main reason for using Makefile instead of Xcode is performance reason. Xcode generator is much slower that Makefile in our project. The performance was improved in Cmake 3.2 but still it is much slower on Xcode. Similar slowness could be obser

Re: [cmake-developers] [PATCH] FindBoost: Add imported targets

2015-11-16 Thread Ruslan Baratov via cmake-developers
On 16-Nov-15 21:01, rle...@codelibre.net wrote: I have attached a patch to add imported targets to FindBoost, in the form of Boost:: (e.g. Boost::date_time) or Boost::Boost as a generic interface library for header-only components. Since it's `Boost::date_time` I expect that it will be `Boost::bo

Re: [cmake-developers] [PATCH] FindBoost: Add imported targets

2015-11-17 Thread Ruslan Baratov via cmake-developers
On 17-Nov-15 15:48, Roger Leigh wrote: On 17/11/2015 07:53, Ruslan Baratov wrote: On 16-Nov-15 21:01, rle...@codelibre.net wrote: I have attached a patch to add imported targets to FindBoost, in the form of Boost:: (e.g. Boost::date_time) or Boost::Boost as a generic interface library for head

Re: [cmake-developers] Fw: Please comment on ios-universal topic

2015-12-02 Thread Ruslan Baratov via cmake-developers
On 02-Dec-15 19:44, Bartosz Kosiorek wrote: Hi. Currently we already support similar target properties: - INSTALL_NAME_DIR - INSTALL_RPATH - INSTALL_RPATH_USE_LINK_PATH It will be great to follow the same name convention. Because this new property is only applicable for INSTALL step, I would l

Re: [cmake-developers] Fw: Please comment on ios-universal topic

2015-12-03 Thread Ruslan Baratov via cmake-developers
On 03-Dec-15 14:57, Bartosz Kosiorek wrote: Thanks Ruslan for feedback. I think that this new property should work also for FRAMEWORK and should support both OSX and iOS In that way it is already done for: - for SHARED library adding FRAMEWORK property produce correct Framework for iOS or O

Re: [cmake-developers] Fw: Please comment on ios-universal topic

2015-12-03 Thread Ruslan Baratov via cmake-developers
On 03-Dec-15 18:45, Bartosz Kosiorek wrote: Hello Ruslan The Android supports both Universal Apk and Multi Apk. https://androidbycode.wordpress.com/2015/07/07/android-ndk-a-guide-to-deploying-apps-with-native-libraries/ http://developer.android.com/google/play/publishing/multiple-apks.html

Re: [cmake-developers] Please comment on ios-universal topic

2015-12-09 Thread Ruslan Baratov via cmake-developers
On 09-Dec-15 04:51, Gregor Jasny wrote: Hello, On 02/12/15 13:44, Bartosz Kosiorek wrote: I would like to propose to rename property name to INSTALL_IOS_UNIVERSAL_LIBS. My thoughts about the property name: I added the necessary functionality to also combine Frameworks and App Bundles. So the

Re: [cmake-developers] Please comment on ios-universal topic

2015-12-10 Thread Ruslan Baratov via cmake-developers
On 10-Dec-15 13:55, Gregor Jasny wrote: Hello, On 09/12/15 10:21, Ruslan Baratov wrote: IOS_INSTALL_COMBINED_BINARY Just to clarify you want to leave only one variable IOS_INSTALL_COMBINED_BINARY for the device + simulator on iOS platforms. Other platforms (in future) will be controlled by us

Re: [cmake-developers] Create subdirectories in Resource directory for Frameworks and Application bundle.

2015-12-11 Thread Ruslan Baratov via cmake-developers
On 12-Dec-15 03:46, Bartosz Kosiorek wrote: Hi To enable iOS build, I'm using following settings in CMakeLists.txt: set(APPLE_PLATFORM "iphonesimulator") I think this one not needed. Can't find any place where it parsed. set(CMAKE_OSX_SYSROOT "/Applications/Xcode.app/Contents/Developer/Pla

Re: [cmake-developers] [PATCH] Re: Create subdirectories in Resource directory for Frameworks and Application bundle.

2015-12-11 Thread Ruslan Baratov via cmake-developers
On 12-Dec-15 07:26, Bartosz Kosiorek wrote: Hi I have updated current documentation with examples, to reflect current status of CMake in OSX/iOS support. Such information is very useful for novice CMake developers, and it saves an hours of digging through mailing list and websites. This is tru

Re: [cmake-developers] Create subdirectories in Resource directory for Frameworks and Application bundle.

2015-12-11 Thread Ruslan Baratov via cmake-developers
On 12-Dec-15 10:08, Bartosz Kosiorek wrote: Thanks Ruslan. I would like to create instruction which is universal for all generators. I think it's not possible. E.g. IOS_INSTALL_COMBINED will work only for Xcode. Currently we would like to support both Make/Ninja and Xcode generators, ..

Re: [cmake-developers] Create subdirectories in Resource directory for Frameworks and Application bundle.

2015-12-19 Thread Ruslan Baratov via cmake-developers
On 18-Dec-15 22:46, Bartosz Kosiorek wrote: Thank you all for your tips/advice. From our experience IDE generators (eg. Visual Studio, Xcode) is much slower than Make or Ninja. I will attach benchmarks later. Unfortunately I'm unable to build Universal libraries for x86 and arm architectu

Re: [cmake-developers] Create subdirectories in Resource directory for Frameworks and Application bundle.

2015-12-21 Thread Ruslan Baratov via cmake-developers
On 21-Dec-15 17:19, Bartosz Kosiorek wrote: Thanks Ruslan for explanation. Currently we already building fat library for arm (armv7;armv7s;arm64) and x86 (i386;x86_64). That's why my original assumption was that it could be easily enabled for Simulator/device (armv7;armv7s;arm64;i386;x86_

Re: [cmake-developers] Namespaces

2016-01-28 Thread Ruslan Baratov via cmake-developers
On 29-Jan-16 01:17, Stephen Kelly wrote: Pau Garcia i Quiles wrote: add_subdirectory(pugixml/scripts) add_subdirectory(QtZeroConf) add_subdirectory(QtDropBox) add_subdirectory(websocketpp) This is an easy way to work with third-party dependences. Unfortunately this is 'easy, obvious and wrong

[cmake-developers] CMake aliasing system

2016-03-12 Thread Ruslan Baratov via cmake-developers
Hi, I guess it is a well known fact that cmake command is almost never executed alone and for non-trivial examples usually hold some extra arguments (home directory, build directory, verbosity level, toolchains, options, ...). Also I guess that such commands doesn't change from day-to-day dev

[cmake-developers] Support CMAKE_IOS_INSTALL_COMBINED for single architecture targets

2016-03-14 Thread Ruslan Baratov via cmake-developers
Hi, Though user can explicitly set CMAKE_IOS_INSTALL_COMBINED=OFF for targets with empty list of supported architectures for given SDK (say simulator) I think it make sense to handle this case correctly in CMake module too. Patch with fix and test attached. Thanks, Ruslo >From 1fb0986da2b6ec

Re: [cmake-developers] CMake aliasing system

2016-03-14 Thread Ruslan Baratov via cmake-developers
On 14-Mar-16 21:59, Brad King wrote: On 03/12/2016 08:04 AM, Ruslan Baratov via cmake-developers wrote: I guess it is a well known fact that cmake command is almost never executed alone and for non-trivial examples usually hold some extra arguments (home directory, build directory, verbosity

Re: [cmake-developers] CMake aliasing system

2016-03-14 Thread Ruslan Baratov via cmake-developers
a fork anyway and do some experiments even if it will not accepted to upstream. Ruslo Tamas On Mon, Mar 14, 2016 at 4:22 PM, Ruslan Baratov via cmake-developers mailto:cmake-developers@cmake.org>> wrote: On 14-Mar-16 21:59, Brad King wrote: On 03/12/2016 08:04 AM, Ruslan Bar

Re: [cmake-developers] CMake aliasing system

2016-03-19 Thread Ruslan Baratov via cmake-developers
t;john@example.com" <mailto:john@example.com> --branch="master" > git push --remote="git://awesome.example.com <http://awesome.example.com>" This is a complete nonsense! So I'm planning to make a fork anyway and do some experiments even

Re: [cmake-developers] CMake API for warnings

2016-03-27 Thread Ruslan Baratov via cmake-developers
I like an effort but not an implementation: * It would be nice to not to set flags globally since we have more fine control options like target_compile_options (i.e. different target may have different warning settings) * this will not work for Xcode since warnings should be set by XCODE_ATTRIB

Re: [cmake-developers] CMake API for warnings

2016-03-29 Thread Ruslan Baratov via cmake-developers
On 28-Mar-16 21:05, Brad King wrote: On 03/28/2016 01:49 AM, Geoffrey Viola wrote: Thanks for the feedback. I’ll have to look more in-depth at Xcode specific issues. Take a look at this approach: * https://github.com/ruslo/sugar/wiki/Cross-platform-warning-suppression I took a look at your re

Re: [cmake-developers] CMake API for warnings

2016-03-29 Thread Ruslan Baratov via cmake-developers
On 28-Mar-16 12:49, Geoffrey Viola wrote: Thanks for the feedback. I’ll have to look more in-depth at Xcode specific issues. > Take a look at this approach: > * https://github.com/ruslo/sugar/wiki/Cross-platform-warning-suppression I took a look at your repository. It’s very sophisticated.

Re: [cmake-developers] CMake API for warnings

2016-03-29 Thread Ruslan Baratov via cmake-developers
On 29-Mar-16 21:24, Brad King wrote: On 03/29/2016 09:09 AM, Ruslan Baratov wrote: As a general note obviously I've used another approach because I decide to create CMake module which can be used with regular CMake versions :) Of course implementing this in CMake "from the box" is best. Are we

Re: [cmake-developers] CMake API for warnings

2016-03-29 Thread Ruslan Baratov via cmake-developers
On 28-Mar-16 21:05, Brad King wrote: On 03/28/2016 01:49 AM, Geoffrey Viola wrote: Thanks for the feedback. I’ll have to look more in-depth at Xcode specific issues. Take a look at this approach: * https://github.com/ruslo/sugar/wiki/Cross-platform-warning-suppression I took a look at your re

Re: [cmake-developers] CMake API for warnings

2016-03-31 Thread Ruslan Baratov via cmake-developers
On 31-Mar-16 20:37, Brad King wrote: On 03/29/2016 11:24 AM, Ruslan Baratov wrote: OTOH many such switches are so specific that they should not be abstracted and can instead be added to COMPILE_OPTIONS with a $ genex guard. What about generating nothing for such cases? Yes, that makes sense.

Re: [cmake-developers] CMake API for warnings

2016-04-05 Thread Ruslan Baratov via cmake-developers
On 05-Apr-16 21:03, Brad King wrote: On 03/31/2016 12:47 PM, Ruslan Baratov wrote: What about 3 properties containing list of 's (groups unexpanded): * COMPILE_WARNINGS_DISABLE # e.g. "shift-sign-overflow;unused" * COMPILE_WARNINGS_ENABLE # e.g. "ALL" * COMPILE_WARNINGS_TREAT_AS_ERROR # e.g. gr

Re: [cmake-developers] CMake API for warnings

2016-04-10 Thread Ruslan Baratov via cmake-developers
Sounds good? On 06-Apr-16 01:03, Ruslan Baratov via cmake-developers wrote: On 05-Apr-16 21:03, Brad King wrote: On 03/31/2016 12:47 PM, Ruslan Baratov wrote: What about 3 properties containing list of 's (groups unexpanded): * COMPILE_WARNINGS_DISABLE # e.g. "shift-sign-overf

Re: [cmake-developers] CMake API for warnings

2016-04-12 Thread Ruslan Baratov via cmake-developers
On 13-Apr-16 01:00, Brad King wrote: Hi Ruslo, Sorry for taking so long to respond here. I've been hoping to find time to think through the design deeply but I don't know when that may happen. Here is some more feedback. I invite others to jump in here. Without more interest I'm hesitant to p

Re: [cmake-developers] CMake API for warnings

2016-04-20 Thread Ruslan Baratov via cmake-developers
So here is a summary of this feature so far: Introducing new 3 functions for controlling compiler warnings: 1) add_compile_warnings * similar to add_definitions, add_compile_options * modify COMPILE_WARNINGS directory property (append) 2) target_compile_warnings * similar to target_compile

Re: [cmake-developers] CMake API for warnings

2016-04-22 Thread Ruslan Baratov via cmake-developers
On 22-Apr-16 20:36, Brad King wrote: On 04/20/2016 09:50 AM, Ruslan Baratov wrote: 1) add_compile_warnings * similar to add_definitions, add_compile_options * modify COMPILE_WARNINGS directory property (append) 2) target_compile_warnings * similar to target_compile_options, target_co

Re: [cmake-developers] CMake API for warnings

2016-04-26 Thread Ruslan Baratov via cmake-developers
On 26-Apr-16 01:58, Brad King wrote: On 04/23/2016 12:11 AM, Ruslan Baratov wrote: It means ignoring directory properties. So by default we will inherit settings. [snip] To disable inheriting we need to add extra argument (?) to *_compile_warnings: add_compile_warnings(DISABLE warn-A)

Re: [cmake-developers] CMake API for warnings

2016-04-29 Thread Ruslan Baratov via cmake-developers
This one looks like a plan. However I will not be able to work on the feature till ~15 May. So if anybody have a free time to start this work - feel free to do it and share patches :) Thanks, Ruslo On 22-Apr-16 20:36, Brad King wrote: On 04/20/2016 09:50 AM, Ruslan Baratov wrote: 1) add_comp

Re: [cmake-developers] CMake API for warnings

2016-05-06 Thread Ruslan Baratov via cmake-developers
Here is the case when such feature might be useful: stackoverflow.com/questions/37036690 Ruslo On 25-Apr-16 21:58, Brad King wrote: On 04/23/2016 12:11 AM, Ruslan Baratov wrote: It means ignoring directory properties. So by default we will inherit settings. [snip] To disable inheriting we n

  1   2   >