Re: [cmake-developers] RFC: CMake precompiled header support and custom compiler based implementation

2015-03-13 Thread Daniel Pfeifer
On Wed, Mar 4, 2015 at 8:53 PM, Brad King wrote: > On 03/03/2015 07:43 AM, Adam Strzelecki wrote: > > On 03/01/2015 08:02 PM, Ben Boeckel wrote: > >> On 02/28/2015 11:59 AM, Adam Strzelecki wrote: > >> target_precompiled_header( ) > >> > >> target_precompiled_header( > SHARED > >> > )

Re: [cmake-developers] RFC: CMake precompiled header support and custom compiler based implementation

2015-03-20 Thread Daniel Pfeifer
On Fri, Mar 20, 2015 at 6:13 PM, Brad King wrote: > On 03/13/2015 08:44 AM, Daniel Pfeifer wrote: >> I pushed some work to https://github.com/purpleKarrot/CMake/commits/pch > > Thanks for working on this. I'm hoping others will respond because > I have lit

Re: [cmake-developers] RFC: CMake precompiled header support and custom compiler based implementation

2015-03-20 Thread Daniel Pfeifer
On Fri, Mar 20, 2015 at 6:35 PM, Amine Khaldi wrote: > Two requests please: > > * The option to use existing headers instead of autogenerated ones. That is an implementation detail. It should not make a difference whether the precompiled header is used through your existing header or through an a

[cmake-developers] Regarding the 'run-include-what-you-use' topic

2015-05-19 Thread Daniel Pfeifer
Hi Brad, Hi Bill, I think it is great that CMake gains support for IWYU. Without having tested it, I think the current implementation is compatible with other static analyzers like for examle clang-tidy. I think this is not good. If the options can be used for other tools than IWYU, the option n

Re: [cmake-developers] Regarding the 'run-include-what-you-use' topic

2015-05-20 Thread Daniel Pfeifer
On Wed, May 20, 2015 at 7:34 PM, Bill Hoffman wrote: > I looked quickly at the clang-tidy docs and it looks interesting. I see it > is setup to use the compile_commands.json file. Originally we looked at > running iwyu using that file. However, it became quickly evident that this > would be a p

[cmake-developers] cmXMLWriter

2015-05-25 Thread Daniel Pfeifer
Hi, I wrote a simple XML writer (cmXMLWriter) that takes care of indentation, escaping, and balancing of end tags. I also ported all of CTest's XML generation to cmXMLWriter. https://github.com/purpleKarrot/CMake/commits/xmlwriter Please review and potentially apply. cheers, Daniel -- Powere

[cmake-developers] Composability of INTERFACE properties and custom targets

2015-05-29 Thread Daniel Pfeifer
Hi, in a project, I have to generate some C++ header files at build time. Targets that use these header files need to know the directory of the generated files and depend on the custom target that in turn depends on the generated files. Example: target_include_directories(lib PRIVATE ${gen_dir}

Re: [cmake-developers] A policy for Policies

2015-06-07 Thread Daniel Pfeifer
On Sun, Jun 7, 2015 at 7:09 PM, Mike Gelfand wrote: > On Sat, June 6, 2015 13:36, Stephen Kelly wrote: >> Code for Policies is also often complex. I often encounter Policies which >> are ancient and which get in the way of code clean up generally. > > Exactly the reason I don't understand why you

[cmake-developers] CTest: hide progress ticks in verbose output

2015-07-03 Thread Daniel Pfeifer
progress ticks when the output is shown (-VV). Cheers, Daniel From 31290dd36fff5fd88a66069d962a9e3c80746525 Mon Sep 17 00:00:00 2001 From: Daniel Pfeifer Date: Fri, 3 Jul 2015 22:52:23 +0200 Subject: [PATCH] CTest: hide progress ticks in verbose output --- Source/CTest/cmCTestBuildHandler.cxx | 10

Re: [cmake-developers] PATCH: add subcommand string(APPEND)

2015-07-06 Thread Daniel Pfeifer
On Mon, Jul 6, 2015 at 8:41 PM, Brad King wrote: > On 07/04/2015 06:27 PM, Daniel Pfeifer wrote: >> Attached is a patch that adds a subcommand string(APPEND). >> This allows to write >> >>> string(APPEND string_variable "some string") >> >> inst

Re: [cmake-developers] PATCH: add subcommand string(APPEND)

2015-07-06 Thread Daniel Pfeifer
On Mon, Jul 6, 2015 at 10:55 PM, James Bigler wrote: > list(APPEND) requires at least one element argument, right? No, see https://github.com/Kitware/CMake/blob/master/Source/cmListCommand.cxx#L236 -- Powered by www.kitware.com Please keep messages on-topic and check the CMake FAQ at: http:/

[cmake-developers] topic 'ctest-change-id'

2015-07-15 Thread Daniel Pfeifer
Hi, The new element `ChangeId` is added to Build.xml and Test.xml. Did you consider adding it as an attribute in `cmCTest::StartXML` instead? That would make this (very useful) information available in all xml files. cheers, Daniel -- Powered by www.kitware.com Please keep messages on-topic an

[cmake-developers] [PATCH] fixup: CTest-side support for compiler version

2015-08-11 Thread Daniel Pfeifer
Commit a5aa23 added support for reporting a compiler version to CDash (but with a hardcoded version number: the version of the compiler that was used to compile CMake). From 594c46c8f25e520d0fad984a4a00c228132cdea1 Mon Sep 17 00:00:00 2001 From: Daniel Pfeifer Date: Tue, 11 Aug 2015 09:52:51

Re: [cmake-developers] CTest XML outputs unsafe content

2015-08-27 Thread Daniel Pfeifer
to me. I saw cmXMLSafe is used in some places inside CTest. Since escaping is handled by cmXMLWriter, this may lead to some double encodings. I have attached two patches that remove all uses of cmXMLSafe from CTest. -- Daniel From 6a5962c671373f0c4e90080abc7b7fe7cf731f77 Mon Sep 17 00:00:00 2

Re: [cmake-developers] RFC: CMake precompiled header support and custom compiler based implementation

2015-10-28 Thread Daniel Pfeifer
are free to help with review, testing, and additional generators. Which generators are the most important for you? -- Daniel > Cheers, > Taylor > > On Fri, Mar 20, 2015 at 3:56 PM, Daniel Pfeifer > wrote: >> >> On Fri, Mar 20, 2015 at 6:35 PM, Amine Khaldi >>

Re: [cmake-developers] Code style auto-formatting

2015-11-17 Thread Daniel Pfeifer
On Tue, Nov 17, 2015 at 10:12 PM, Robert Dailey wrote: > On Tue, Nov 17, 2015 at 6:57 AM, Paul Smith wrote: >> On Tue, 2015-11-17 at 08:14 +, Stuermer, Michael SP/HZA-ZSEP wrote: >>> In short, there is no fully automated style checking. If someone would >>> come up with a tool & configuration

Re: [cmake-developers] RFC: CMake precompiled header support and custom compiler based implementation

2015-12-03 Thread Daniel Pfeifer
ss header compilation CPU cycles? :-) > > On Fri, Oct 30, 2015 at 1:48 AM, James Johnston > wrote: >>> -Original Message- >> >>> From: cmake-developers [mailto:cmake-developers-boun...@cmake.org] >> >>> On Behalf Of Daniel Pfeifer >> >>&g

Re: [cmake-developers] RFC: CMake precompiled header support and custom compiler based implementation

2015-12-04 Thread Daniel Pfeifer
> On Friday, December 4, 2015, Daniel Pfeifer wrote: >> >> My working branch is here: >> https://github.com/purpleKarrot/CMake/commits/pch >> >> Feel free to comment, evaluate, contribute. >> >> I am nut fully decided regarding these two questions:

Re: [cmake-developers] RFC: CMake precompiled header support and custom compiler based implementation

2015-12-04 Thread Daniel Pfeifer
On Fri, Dec 4, 2015 at 9:19 PM, David Cole wrote: > Right, I was talking about the pch-binary. > > Why would CMake even need to generate a header file for pre-compiled > headers? Why not just allow the user to say which of his header files > should be the one to use for precompiled headers? Gener

Re: [cmake-developers] RFC: CMake precompiled header support and custom compiler based implementation

2015-12-04 Thread Daniel Pfeifer
On Fri, Dec 4, 2015 at 11:32 PM, David Cole wrote: > Makes sense. > > Can I inject my own "#include " into > the generated stream, or otherwise inject something into it? You should be able to do: target_include_directories(your_target PUBLIC public PRIVATE private ) target_precompile_heade

Re: [cmake-developers] Restriction on target types for GraphViz dependency graph generation

2015-12-20 Thread Daniel Pfeifer
On Wed, Dec 16, 2015 at 5:20 AM, Andrey Mishchenko wrote: > Hi, > > I noticed that the automatic dependency graph generation in CMake targeting > GraphViz only considers targets of certain types. In particular, it only > adds nodes for executables and shared, static, and module libraries. > > Is t

Re: [cmake-developers] Profile Cmake scripts

2016-01-05 Thread Daniel Pfeifer
> Make generation: >real 4m45.089s >user 2m56.117s >sys 0m17.481s > > Ninja generation: > real 2m48.585s > user 2m30.712s > sys 0m6.313s Generators for Xcode and Visual Studio have to generate more files. From 46a5639a27bab0b227b457e1224c702dd143af33 Mon Sep

Re: [cmake-developers] CMake daemon for user tools

2016-01-21 Thread Daniel Pfeifer
On Wed, Jan 20, 2016 at 10:03 PM, Stephen Kelly wrote: > Milian Wolff wrote: > >>> I'm concerned that the memory usage of a daemon implementing the proposed >>> capabilities may be too large to be practical (at least without a major >>> redesign of certain structures that tend to duplicate substri

Re: [cmake-developers] more use of cmXMLWriter

2016-03-22 Thread Daniel Pfeifer
On Mon, Mar 21, 2016 at 3:39 PM Brad King wrote: > On 03/19/2016 05:08 PM, Daniel Pfeifer wrote: > > I ported some more generators to cmXMLWriter. > > Thanks. I've applied and merged to 'next' for testing: > > cmXMLWriter: overload Element() method for empty

[cmake-developers] Integration of clang-tidy

2016-04-08 Thread Daniel Pfeifer
output is recognized by CTest's diagnostic parser. I haven't checked how they look on CDash. cheers, Daniel From c1996b295d2e49134b6f5ef2be690588e5bf2732 Mon Sep 17 00:00:00 2001 From: Daniel Pfeifer Date: Fri, 8 Apr 2016 22:09:27 +0200 Subject: [PATCH] Provide integration for clan

Re: [cmake-developers] Integration of clang-tidy

2016-04-26 Thread Daniel Pfeifer
On Mon, Apr 11, 2016 at 5:26 PM, Brad King wrote: > On 04/08/2016 06:31 PM, Daniel Pfeifer wrote: >> I implemented the integration of `clang-tidy` along the lines of the >> `include-what-you-use` integration. >> There is a new `_CLANG_TIDY` target property that is initialized

Re: [cmake-developers] Integration of clang-tidy

2016-04-27 Thread Daniel Pfeifer
On Wed, Apr 27, 2016 at 10:47 PM, Gregor Jasny wrote: > Hallo Daniel, > > On 09/04/16 00:31, Daniel Pfeifer wrote: >> I implemented the integration of `clang-tidy` along the lines of the >> `include-what-you-use` integration. >> There is a new `_CLANG_TIDY` target p

Re: [cmake-developers] Integration of clang-tidy

2016-04-27 Thread Daniel Pfeifer
On Wed, Apr 27, 2016 at 3:10 PM, Brad King wrote: > On 04/26/2016 05:32 PM, Daniel Pfeifer wrote: >> This is a follow up patch: > > Thanks. > >> -tidy_cmd.insert(tidy_cmd.end(), orig_cmd.begin()+1, orig_cmd.end()); >> +tidy_cmd.insert(tidy_cmd.end(),

Re: [cmake-developers] Code style auto-formatting

2016-04-27 Thread Daniel Pfeifer
the list, because Clang-Format does strange things to @VARIALES@. Please feel free to add/remove options from the .clang-format files. IncludeCategories and Standard should stay as they are. cheers, Daniel From 452087a4b9f05742aed38040ab8d9b970fd05719 Mon Sep 17 00:00:00 2001 From: Daniel P

Re: [cmake-developers] Integration of clang-tidy

2016-05-03 Thread Daniel Pfeifer
PM, Daniel Pfeifer wrote: >> It is unrelated. Clang-Tidy picks the interesting pieces from the >> compiler command line and ignores the rest. Whether cc is given or not >> makes no difference. >> But maybe we should leave the decision what to pick to Clang-Tidy? It >&

Re: [cmake-developers] Code style auto-formatting

2016-05-09 Thread Daniel Pfeifer
On Mon, May 9, 2016 at 11:31 AM, Rolf Eike Beer wrote: > Am Montag, 9. Mai 2016, 13:14:17 schrieb Brad King: >> On 05/02/2016 10:08 AM, Brad King wrote: >> > Next I'll look at the style updates themselves. >> >> I've made some more preparatory commits: > > […] >> Remove `//--...` horizontal s

Re: [cmake-developers] Code style auto-formatting

2016-05-09 Thread Daniel Pfeifer
On Mon, May 9, 2016 at 11:14 AM, Brad King wrote: > On 05/02/2016 10:08 AM, Brad King wrote: >> Next I'll look at the style updates themselves. > > I've made some more preparatory commits: > > Isolate formatted streaming blocks with clang-format off/on > https://cmake.org/gitweb?p=cmake.git;a=co

Re: [cmake-developers] [ANNOUNCE] CMake C++ coding style transition

2016-05-12 Thread Daniel Pfeifer
On Thu, May 12, 2016 at 7:58 AM, Brad King wrote: > Hi Folks, > > As discussed previously on the developer list: > > Code style auto-formatting > http://thread.gmane.org/gmane.comp.programming.tools.cmake.devel/14969 > > http://thread.gmane.org/gmane.comp.programming.tools.cmake.devel/14969/fo

Re: [cmake-developers] Developer tasks - Refactoring

2016-05-19 Thread Daniel Pfeifer
On Wed, Feb 10, 2016 at 12:15 AM Stephen Kelly wrote: > 1) Make cmLocalGenerator not inherit cmOutputConverter > * Change enums like cmLocalGenerator::START_OUTPUT with sed. See Please see attached patch. From 9e1203308ad79d62b24fe6930501e1013b4e52ec Mon Sep 17 00:00:00 2001 From:

Re: [cmake-developers] Developer tasks - Refactoring

2016-05-19 Thread Daniel Pfeifer
On Wed, Feb 10, 2016 at 12:12 AM, Stephen Kelly wrote: > 1) Make cmLocalGenerator not inherit cmOutputConverter > * Change enums like cmLocalGenerator::START_OUTPUT with sed. See > >https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=eac15298 > > for a similar sed command to achieve this. >

Re: [cmake-developers] Developer tasks - Refactoring

2016-05-19 Thread Daniel Pfeifer
On Thu, May 19, 2016 at 11:18 PM Daniel Pfeifer wrote: > On Wed, Feb 10, 2016 at 12:15 AM Stephen Kelly wrote: > >> 1) Make cmLocalGenerator not inherit cmOutputConverter >> * Change enums like cmLocalGenerator::START_OUTPUT with sed. See > > > Please see attac

[cmake-developers] fixup! Simplify boolean expressions

2016-06-02 Thread Daniel Pfeifer
My "Simplify boolean expressions" refactoring caused C4800 warnings for MSVC builds. A fixup is attached, as I cannot ssh at the moment. From ece15fe51c46cb93823ccd8a54243d35ebdc0d3c Mon Sep 17 00:00:00 2001 From: Daniel Pfeifer Date: Thu, 2 Jun 2016 09:59:36 +0200 Subject: [PA

Re: [cmake-developers] Some trivial patches

2016-06-02 Thread Daniel Pfeifer
On Thu, Jun 2, 2016 at 11:15 AM, Tobias Hunger wrote: > On Mi, 2016-06-01 at 11:18 -0400, Brad King wrote: >> On 06/01/2016 05:04 AM, Tobias Hunger wrote: >> > Attached you will find a couple of really small changes that add const to a >> > method and fix a couple of typos in comments. Trivial stu

Re: [cmake-developers] Developer tasks - Refactoring

2016-06-06 Thread Daniel Pfeifer
On Sun, Jun 5, 2016 at 2:15 PM, Stephen Kelly wrote: > On 05/19/2016 11:27 PM, Daniel Pfeifer wrote: >> On Wed, Feb 10, 2016 at 12:12 AM, Stephen Kelly wrote: >>> 1) Make cmLocalGenerator not inherit cmOutputConverter >>> * Change enums like cmLocalGenerator::

Re: [cmake-developers] CMake daemon-mode

2016-06-07 Thread Daniel Pfeifer
On Mon, Jun 6, 2016 at 5:39 PM, Tobias Hunger wrote: > Please help to support your use-cases. A while ago I wrote a graphical cache editor in GTK: https://github.com/purpleKarrot/cmake-gtk The tool reads the cache and provides a graphical view to modify it. It can then write the cache, run cmake,

Re: [cmake-developers] CMake daemon-mode

2016-06-07 Thread Daniel Pfeifer
On Mon, Jun 6, 2016 at 7:24 PM, Brad King wrote: > On 06/06/2016 11:39 AM, Tobias Hunger wrote: > >> A big chunk of Stephen's work has not even landed in my branch yet. Since >> cmake >> reformated all the source in the meantime it is a bit tedious to apply >> patches >> from his tree and I have

Re: [cmake-developers] Review request: extract-cmMessenger branch

2016-06-09 Thread Daniel Pfeifer
On Thu, Jan 28, 2016 at 10:42 PM, Stephen Kelly wrote: > > Hi, > > I have pushed a extract-cmMessenger branch to my clone: > > https://github.com/steveire/CMake/commits/extract-cmMessenger I have rebased it to master and cherry-picked some minor changes. Once those minor changes land in master,

Re: [cmake-developers] Review request: extract-cmMessenger branch

2016-06-09 Thread Daniel Pfeifer
On Thu, Jun 9, 2016 at 2:17 PM, Brad King wrote: > On 06/09/2016 05:06 AM, Tobias Hunger wrote: >> CMake Error at src/1/app/CMakeLists.txt:70 (add_custom_target): >> Cannot find source file: >> >> unknownFile.qml >> >> Tried extensions .c .C .c++ .cc .cpp .cxx .m .M .mm .h .hh .h++ .hm .hp

[cmake-developers] Questions about coding conventions

2016-06-10 Thread Daniel Pfeifer
Hi, By looking at the CMake source code, there are some inconsistencies regarding coding conventions. This is not a big problem and fixing them probably does not have a high priority. I would like to know what conventions to follow for new code. Formatting: No longer an issue. A .clang-format is

Re: [cmake-developers] Questions about coding conventions

2016-06-10 Thread Daniel Pfeifer
On Fri, Jun 10, 2016 at 4:16 PM, Tobias Hunger wrote: > On Fr, 2016-06-10 at 15:30 +0200, Daniel Pfeifer wrote: >> Naming conventions: Classes are named cmLikeThis. Member functions >> and member variables are named LikeThis. Local variables are named >> likeThis. Member

Re: [cmake-developers] Review request: extract-cmMessenger branch

2016-06-10 Thread Daniel Pfeifer
On Fri, Jun 10, 2016 at 8:17 PM, Stephen Kelly wrote: > Tobias Hunger wrote: >> Forcing messages into one consistent format will be a pain, agreed, but >> continuing to add messages in whatever form the developer likes at the >> time of writing the code is even worse. CMake has a lot of different

Re: [cmake-developers] Questions about coding conventions

2016-06-10 Thread Daniel Pfeifer
On Fri, Jun 10, 2016 at 5:19 PM, Tobias Hunger wrote: > On Fr, 2016-06-10 at 16:34 +0200, Daniel Pfeifer wrote: >> If used consistently, it indicates that you are dealing with a member. >> I personally prefer `this->` over `m_`. With semantic syntax >> highlighting you pr

Re: [cmake-developers] Questions about coding conventions

2016-06-10 Thread Daniel Pfeifer
On Fri, Jun 10, 2016 at 3:43 PM, Brad King wrote: > On 06/10/2016 09:30 AM, Daniel Pfeifer wrote: >> By looking at the CMake source code, there are some inconsistencies >> regarding coding conventions. This is not a big problem and fixing >> them probably does not have a high

[cmake-developers] Dogfooding: clang-tidy, include-what-you-use, link-what-you-use

2016-06-12 Thread Daniel Pfeifer
Hi, now that we have integrated include-what-you-use, clang-tidy, and just recently also a mechanism for link-what-you-use, it is time we start eating our own dog food. https://en.wikipedia.org/wiki/Eating_your_own_dog_food I have set up a dashboard build where all three mechanisms are enabled:

Re: [cmake-developers] Questions about coding conventions

2016-06-12 Thread Daniel Pfeifer
On Sun, Jun 12, 2016 at 4:26 PM, Ben Boeckel wrote: > On Fri, Jun 10, 2016 at 15:30:05 +0200, Daniel Pfeifer wrote: >> Passing and returning strings: We have `const char*`, `std::string`, >> and `std::string const&`. Unifying this can affect performance. >> Storing `const

Re: [cmake-developers] Questions about coding conventions

2016-06-13 Thread Daniel Pfeifer
On Mon, Jun 13, 2016 at 2:09 PM, Ben Boeckel wrote: > On Mon, Jun 13, 2016 at 00:03:29 +0200, Daniel Pfeifer wrote: >> Can you show an example? To be clear: We are looking for a function, >> that has a code path for `str == NULL` and a *different* codepath for >> `str[0] = &

[cmake-developers] fixup! Remove redundant arguments from fstream constructors

2016-06-14 Thread Daniel Pfeifer
This is a fixup for the std-fstream topic. From 1d52cfe88e98738a1f1172cd1723a07ac579d2eb Mon Sep 17 00:00:00 2001 From: Daniel Pfeifer Date: Tue, 14 Jun 2016 09:47:46 +0200 Subject: [PATCH] fixup! Remove redundant arguments from fstream constructors --- Tests/AliasTarget/commandgenerator.cpp

Re: [cmake-developers] Questions about coding conventions

2016-06-14 Thread Daniel Pfeifer
On Tue, Jun 14, 2016 at 3:14 PM, Brad King wrote: > On 06/13/2016 10:16 AM, Brad King wrote: >>> Can't `std::ifstream` and `std::ofstream` be used directly? It seams >>> that kwsys does some workarounds >> >> Yes, std::{o,f}stream can be used directly. > > On second thought, std::{i,o}fstream shou

Re: [cmake-developers] daemon-mode meeting last Tuesday

2016-06-24 Thread Daniel Pfeifer
On Thu, Jun 23, 2016 at 2:27 PM, Tobias Hunger wrote: > * We both think it only makes sense to merge the infrastructure part into > cmake (if it passes review first of course) once we have some > functionality > that is genuinely useful. So we want to aim at having the infrastructure >

Re: [cmake-developers] daemon-mode meeting last Tuesday

2016-06-24 Thread Daniel Pfeifer
On Fri, Jun 24, 2016 at 4:56 PM, Tobias Hunger wrote: > Hi Brad, > > On Thu, Jun 23, 2016 at 10:46 PM, Brad King wrote: >>> * This is currently used to set sourcedirectory, builddirectory and >>> generator. >>> >>> These three should be passed in via the command line instead, again >>>

Re: [cmake-developers] daemon-mode meeting last Tuesday

2016-06-24 Thread Daniel Pfeifer
On Fri, Jun 24, 2016 at 5:24 PM, Tobias Hunger wrote: > Hi Daniel, > > On Fri, Jun 24, 2016 at 5:08 PM, Daniel Pfeifer > wrote: >>>> Currently cmake-gui supports switching generators, build trees, etc., so >>>> there is some precedent for such switching withi

Re: [cmake-developers] libuv in cmake (was: daemon-mode meeting last Tuesday)

2016-06-27 Thread Daniel Pfeifer
On Mon, Jun 27, 2016 at 5:02 PM, Brad King wrote: > On 06/24/2016 04:12 AM, Tobias Hunger wrote: >>> Please think about adding libuv earlier. As Brad wrote before, libuv >>> could replace some #ifdef code that we currently have >> >> Feel free to merge it at your own pace. I'll just use your copy

Re: [cmake-developers] Status of pre-compiled headers

2016-06-27 Thread Daniel Pfeifer
On Mon, Jun 27, 2016 at 5:09 PM, Brad King wrote: > On 06/24/2016 12:19 PM, Oleh Kravchenko wrote: >> As far as I can see from issue tracker and the mail list archives of >> "precompiled headers" feature we are still struggling for solution. >> https://cmake.org/Bug/view.php?id=1260 >> https://git

Re: [cmake-developers] Status of pre-compiled headers

2016-06-27 Thread Daniel Pfeifer
On Mon, Jun 27, 2016 at 5:24 PM, Daniel Pfeifer wrote: > On Mon, Jun 27, 2016 at 5:09 PM, Brad King wrote: >> On 06/24/2016 12:19 PM, Oleh Kravchenko wrote: >>> As far as I can see from issue tracker and the mail list archives of >>> "precompiled headers"

[cmake-developers] Script to update liblzma from upstream.

2016-06-28 Thread Daniel Pfeifer
the script is introduced, we may consider switching to the 5.2 branch. Cheers, Daniel From c0401e6f56fe5c9c77e9a80a2c2ab063930f70ed Mon Sep 17 00:00:00 2001 From: Daniel Pfeifer Date: Tue, 28 Jun 2016 23:10:11 +0200 Subject: [PATCH] Add script to update liblzma from upstream --- Utilities/Scripts

Re: [cmake-developers] Patch for CM_OVERRIDE

2016-06-29 Thread Daniel Pfeifer
On Wed, Jun 29, 2016 at 7:15 AM, Roman Wüger wrote: > Hello, > > I've attached a patch for the CM_OVERRIDE commit Hi Roman, I added "override" to the destructor on purpose. The intent is to make the check fail in Visual Studio 10. See the commit message here: https://cmake.org/gitweb?p=stage/cma

Re: [cmake-developers] Please apply patch from https://github.com/hunger/CMake/commits/for-upstream to master

2016-06-29 Thread Daniel Pfeifer
On Wed, Jun 29, 2016 at 10:17 AM, Tobias Hunger wrote: > On Mi, 2016-06-29 at 09:54 +0200, Rolf Eike Beer wrote: >> Am 2016-06-29 09:35, schrieb Tobias Hunger: >> > Hello Brad, >> > >> > I currently get more than 2000 warnings when building cmake master, all >> > about >> > missing overrides. >> >

Re: [cmake-developers] [PATCH] Improve encoding handling on Windows

2016-07-01 Thread Daniel Pfeifer
Hi Dāvis, On Fri, Jul 1, 2016 at 4:18 AM, Dāvis Mosāns wrote: > On Windows getenv uses ANSI codepage so it needs to be encoded to > internally used encoding (eg. UTF-8). Here we use _wgetenv instead > and encode that. Your change to the SystemTools::GetEnv function introduces memory leaks, since

Re: [cmake-developers] [PATCH] Improve encoding handling on Windows

2016-07-01 Thread Daniel Pfeifer
On Fri, Jul 1, 2016 at 4:12 PM, Ben Boeckel wrote: > On Fri, Jul 01, 2016 at 16:44:32 +0300, Dāvis Mosāns wrote: >> 2. change GetEnv to return std::unique_ptr which will be >> automatically deleted once out of scope and we still can check if there >> wasn't such env if it's empty. > > Hrm. I'd

[cmake-developers] QtAutoGenerators output in Ninja

2016-07-12 Thread Daniel Pfeifer
Hi, CMake currently puts messages like "Generating moc source" into the buildlog. This conflicts with the convention "no output == all good". The messages blend nicely in the colorful output of Unix Makefiles, but when using the Ninja generator, they create quite some noise. How can this be impro

[cmake-developers] Support for Precompiled Headers

2012-06-14 Thread Daniel Pfeifer
Hi, In a private mailing with Dave Abrahams and Brad King I wrote a proposal of how I imagine PCH support should be implemented. Brad asked me to send it to this list for further discussion. Please note that even though I use present tense in the proposal, nothing descibed below is implemented yet

Re: [cmake-developers] Support for Precompiled Headers

2012-06-14 Thread Daniel Pfeifer
ectly to the generator(s)? Given that the required changes are probably very small (the implementation that I linked above is less than 100 lines of CMake code), it is maybe more effort for you to get me started than it is to implement it... :-) cheers, Daniel > > On Thu, Jun 14, 2012 at 4:36

Re: [cmake-developers] Support for Precompiled Headers

2012-07-03 Thread Daniel Pfeifer
2012/6/14 Daniel Pfeifer : > 2012/6/14 David Cole : >> Sounds awesome to me! >> >> When can you submit a patch? :-) >> > > Thanks for the motivation! While I would like to contribute a patch, I > have absolutely no idea where to start. > Should

Re: [cmake-developers] Portable STD_CXX11 target property?

2012-08-20 Thread Daniel Pfeifer
2012/8/20 Rolf Eike Beer : > Stephen Kelly wrote: >> Stephen Kelly writes: >> > Brad King wrote: >> > >> I have pushed a std-cxx-target-property branch to my gitorious repo to >> > >> illustrate the idea further, but I don't think there is any way to >> > >> specify which c++ standard to restrict

Re: [cmake-developers] CMake: need developer help / adopt a bug

2012-08-20 Thread Daniel Pfeifer
2012/8/20 David Cole : > Call for CMake developers! Please adopt one or more of these bugs if you > can. > > The following 11 bugs are on the CMake 2.8.10 roadmap, but they are > UNASSIGNED. They do not have anybody actively working on them at the moment. > > ... > http://public.kitware.com/Bug

Re: [cmake-developers] Exporting dependent library targets in multiple export sets

2012-08-22 Thread Daniel Pfeifer
2012/8/22 Yury G. Kudryashov : > 2012/8/23 Brad King : >> On 08/22/2012 04:57 PM, Yury G. Kudryashov wrote: >>> I'm going to push the current state of my work to >>> git pull git://gitorious.org/~urkud1/cmake/urkud-cmake.git w/export-set >>> in a few minutes. >> >> Thanks for working on this! >> >>

Re: [cmake-developers] Generator expressisons in target properties

2012-11-07 Thread Daniel Pfeifer
2012/11/6 Stephen Kelly > Stephen Kelly wrote: > > So that it is fully aware of all of its [transitive] dependencies (and any > includes and compile defintions requirements) and I would use it like this: > > add_executable(foo_exe ...) > target_link_libraries(foo_exe boost::mpl) > Yeah, I

Re: [cmake-developers] Setting include directories via target_link_libraries() ?

2012-12-10 Thread Daniel Pfeifer
2012/12/7 Stephen Kelly > Stephen Kelly wrote: > > > I haven't tried to analyse how much of the complexity is due to whether > > target_use_targets or target_link_libraries is used. I think the harder > > parts of this topic so far have been related to exports. Like I said > > though, I haven't a

[cmake-developers] Fwd: EXPORT_NAME-genex

2013-05-20 Thread Daniel Pfeifer
2013/5/20 Brad King > We had some recent discussion about encouraging the convention of > "namespace::" for imported targets, but perhaps we should reconsider > the value and cost. One of CMake's most powerful features in my opionion is the way it handles sub-projects. I can take two CMake-proj

Re: [cmake-developers] Fwd: EXPORT_NAME-genex

2013-05-25 Thread Daniel Pfeifer
2013/5/22 Alexander Neundorf > On Monday 20 May 2013, Daniel Pfeifer wrote: > > 2013/5/20 Brad King > > > > > We had some recent discussion about encouraging the convention of > > > "namespace::" for imported targets, but perhaps we should reconsid

Re: [cmake-developers] Fwd: EXPORT_NAME-genex

2013-05-25 Thread Daniel Pfeifer
2013/5/25 Alexander Neundorf > On Saturday 25 May 2013, Daniel Pfeifer wrote: > > 2013/5/22 Alexander Neundorf > > > > > On Monday 20 May 2013, Daniel Pfeifer wrote: > > > > 2013/5/20 Brad King > > > > > > > > > We

Re: [cmake-developers] INTERFACE_LIBRARY target type

2013-09-02 Thread Daniel Pfeifer
> 4) The target_* commands always need to be invoked with an explicit > INTERFACE option. > Maybe PUBLIC should be allowed too (providing the same effect)? I don't really have a rationale. I just wrote PUBLIC a few times by accident. That might be a hint for a more intuitive interface. -- Powered

[cmake-developers] confusing documentation: VS_WINRT_EXTENSIONS

2013-09-02 Thread Daniel Pfeifer
Hi, The target property VS_WINRT_EXTENSIONS is documented as: "Can be set to enable C++/CX language extensions." Is that really what this property does? I am trying to build a C++ project (no C++/CX) for ARM with Visual Studio 2012. As an example, I took the following code: http://msdn.mic

Re: [cmake-developers] confusing documentation: VS_WINRT_EXTENSIONS

2013-09-03 Thread Daniel Pfeifer
2013/9/2 Brad King : > On 09/02/2013 11:42 AM, Daniel Pfeifer wrote: >> The target property VS_WINRT_EXTENSIONS is documented as: "Can be set >> to enable C++/CX language extensions." >> Is that really what this property does? > [snip] >> So my questio

Re: [cmake-developers] confusing documentation: VS_WINRT_EXTENSIONS

2013-09-03 Thread Daniel Pfeifer
2013/9/3 Brad King : > On 09/03/2013 03:39 AM, Daniel Pfeifer wrote: >> According to http://www.cmake.org/Bug/view.php?id=12930#c30721, >> WindowsAppContainer will not pass /ZW to the command line. >> Hence, setting VS_WINRT_EXTENSIONS will not enable C++/CX and the >>

Re: [cmake-developers] confusing documentation: VS_WINRT_EXTENSIONS

2013-09-04 Thread Daniel Pfeifer
2013/9/3 Daniel Pfeifer : > 2013/9/3 Brad King : >> On 09/03/2013 03:39 AM, Daniel Pfeifer wrote: >>> According to http://www.cmake.org/Bug/view.php?id=12930#c30721, >>> WindowsAppContainer will not pass /ZW to the command line. >>> Hence, setting VS_WINRT_EXTEN

[cmake-developers] (naming-)convention for ALIAS and INTERFACE targets

2013-09-13 Thread Daniel Pfeifer
Hi Steve and all, we restarted the CMake-ification of molularized Boost with modern CMake features here: https://github.com/boost-cmake/boost-cmake We want to be able to build all Boost libraries together, but also each library on its own. Am I correct that in both cases we use the :: name in tll

Re: [cmake-developers] Converting CMake documentation to reStructuredText and Sphinx

2013-10-10 Thread Daniel Pfeifer
2013/10/10 Brad King : > Hi Folks, > > ... After considering markup languages > such as Markdown and AsciiDoc I settled on reStructuredText: > > http://docutils.sourceforge.net/docs/ref/rst/restructuredtext.html > > It has a great extension mechanism and is supported by fantastic > tools like Sph

Re: [cmake-developers] Converting CMake documentation to reStructuredText and Sphinx

2013-10-11 Thread Daniel Pfeifer
2013/10/11 Brad King : > On 10/11/2013 01:09 AM, Daniel Pfeifer wrote: >> Will it still be able to generate DocBook, so companies can include >> the documentation in their handbook? > > It can generate whatever Sphinx can generate. IIRC Sphinx does not > currently support

Re: [cmake-developers] Additional properties for PCH support

2013-11-26 Thread Daniel Pfeifer
2013/11/26 Stephen Kelly : > Peter Kuemmel wrote: >> So, is this the right way to improve the pch situation? > > I don't know anything about PCH, but Daniel Pfeifer has a branch here: > > https://github.com/purpleKarrot/CMake/tree/WIP-PCH-Support Thanks for pointing a

Re: [cmake-developers] TARGET property LOCATION

2013-12-02 Thread Daniel Pfeifer
2013/12/2 Nils Gladitz : > On 12/02/2013 09:19 AM, Marcel Loose wrote: >> >> What would be the preferred way to pass the location of a built executable >> target to ADD_TEST? By using the COMMAND option of ADD_TEST, or by using the >> $ generator expression? Best regards, Marcel Loose. > > My perso

Re: [cmake-developers] RFC: add version to project() call

2014-01-14 Thread Daniel Pfeifer
2014/1/14 Brad King : > On 01/13/2014 01:38 PM, Alexander Neundorf wrote: >> does this require a policy now ? >> >> Somebody could set Foo_VERSION_MAJOR in the toplevel subdir, and have a >> project(Foo) >> call in a subdir, which would now unset Foo_VERSION_MAJOR. >> The same for PROJECT_VERSION_M

Re: [cmake-developers] RFC: add version to project() call

2014-01-14 Thread Daniel Pfeifer
2014/1/14 Matthew Woehlke : > On 2014-01-14 10:37, Brad King wrote: >> >> On 01/13/2014 01:38 PM, Alexander Neundorf wrote: >>> >>> does this require a policy now ? >>> >>> Somebody could set Foo_VERSION_MAJOR in the toplevel subdir, and have a >>> project(Foo) >>> call in a subdir, which would now

Re: [cmake-developers] Compiler features/extensions remaining/future issues

2014-06-16 Thread Daniel Pfeifer
2014-06-15 22:24 GMT+02:00 Stephen Kelly : > Brad King wrote: > > > On 06/13/2014 05:19 AM, Stephen Kelly wrote: > >> 11) WriteCompilerDetectionHeader vs GenerateExportHeader > > > > IMO these two modules are solving orthogonal problems and should not > > be mixed. > > I'm not sure I agree. > > Ge

Re: [cmake-developers] Optional dependencies in target INTERFACE

2014-06-30 Thread Daniel Pfeifer
2014-06-30 10:18 GMT+02:00 Stephen Kelly : > > Hello, > > QtSQL provides abstracted access to database data. It does not depend on > QtGui or QtWidgets. > > It also provides the header-only qsqlrelationaldelegate.h. The contents of > that header depend on QtWidgets, and are wrapped in an ifdef > Q

[cmake-developers] LZMA support

2014-07-20 Thread Daniel Pfeifer
Hi, In my liblzma branch (https://github.com/purpleKarrot/CMake/commits/liblzma), I did the following: * add the liblzma source from XZ Utils * make it compile with CMake (by adding CMakeLists.txt etc.) * make it compile in VC6 (by porting the source from C99 to C90) * make cmlibarchive use cmlib

Re: [cmake-developers] LZMA support

2014-07-30 Thread Daniel Pfeifer
2014-07-30 15:58 GMT+02:00 Brad King : > On 07/23/2014 10:43 AM, Brad King wrote: > > I've merged the topic to 'next' for testing, but without the CPack > > or ExternalProject pieces. Likely some more portability fixes > > will be required after testing on the dashboard. Once that is clean > > I

Re: [cmake-developers] Modern cross-platform buildsystem design requirements

2014-09-10 Thread Daniel Pfeifer
Hi Steve, 2014-09-10 0:32 GMT+02:00 Stephen Kelly : > > > I wonder what a cmake-based buildsystem would look like for a collection of > executables/targets which should be built for multiple ... > I would generalize even more and finish the sentence with "options at once". This includes Debug/Rel

Re: [cmake-developers] alternative gui for CMAKE (webgui)

2014-09-14 Thread Daniel Pfeifer
2014-09-14 20:15 GMT+02:00 i-love-spam : > Hello Cmake, > > in our project we develop for most of the platforms (android, ios, macos, > WinRT, Win32, blackbery-qnx etc). We are starting to switch more and more > projects to Cmake. > For most of the stuff that we build we use buildservers to make r

Re: [cmake-developers] cmake-gui icons

2014-10-28 Thread Daniel Pfeifer
2014-10-27 21:33 GMT+01:00 Ben Boeckel : > On Mon, Oct 27, 2014 at 11:59:09 -0600, Orion Poplawski wrote: > > Trying to bring a bit more attention to this: > > > > Fedora is pushing to have higher resolution icons for the applications. > There > > already is CMakeSetup128.png, but these would need

Re: [cmake-developers] [CMake] target_include_directories target issue

2014-11-17 Thread Daniel Pfeifer
FWD to developers list. I think this is a documentation issue. Luis Felipe Dominguez Vega schrieb am Mon Nov 17 2014 at 5:42:55 PM: > I can't see into the source code of cmake, into the > cmTargetIncludeDirectoriesCommand.cxx somethink like import the > INCLUDE_DIRECTORIES from target passed to

[cmake-developers] Minimal compiler requirements for CMake itself

2014-12-18 Thread Daniel Pfeifer
Hi, I would like to know what the oldest versions of GCC and Visual Studio are that should be able to compile the CMake source code. I was unable to find any information about that. I expected the oldest supported version of Visual Studio to be 6.0, but that one fails to build CMake. Don't get m

[cmake-developers] variables for top level project version

2016-08-18 Thread Daniel Pfeifer
Hi! The project() command currently sets some variables in both PROJECT_ and _ form. So far good. It also sets CMAKE_PROJECT_NAME to the top level project name. This is not documented. It does not set any other variables of the form CMAKE_PROJECT_. This would be useful for the version variables

Re: [cmake-developers] Security in CMake

2016-08-22 Thread Daniel Pfeifer
Hi Egor, On Sat, Aug 20, 2016 at 1:48 PM, Egor Pugin wrote: > Hi, > > I'm working on a package manager based on cmake. Please rethink that approach. Your package manager should support CMake. It should not be based on it. Your design will be safer and more flexible. Cheers, Daniel -- Powered

Re: [cmake-developers] Developer tasks - Refactoring

2016-09-07 Thread Daniel Pfeifer
On Wed, Feb 10, 2016 at 12:12 AM, Stephen Kelly wrote: > 3) Compute cmGeneratorTarget state non-lazily in its constructor. > * Historically target state for generators was computed lazily because it > might need to be cleared and re-computed. That is no-longer true. SourceFilesMap is cleared i

  1   2   >