[cmake-developers] [PATCH] User may now specify toolset through CMake GUI
Robert Dailey
rcdailey.lists at gmail.com
Mon Feb 16 11:59:38 EST 2015
Fair points, I will make the changes.
What is the recommended practice for updating patches on mailing
lists? With pull requests it's easy because it picks up my changes I
push to my branch automatically. I think if I push another patch to
the mailing list it will start another email thread, which may cause
lost context since this email chain won't be connected to the new one
(not sure if that matters).
Thanks Brad.
On Mon, Feb 16, 2015 at 10:35 AM, Brad King <brad.king at kitware.com> wrote:
> On 02/15/2015 03:27 PM, rcdailey.lists at gmail.com wrote:
>> From: Robert Dailey <rcdailey at gmail.com>
>>
>> The -T parameter to CMake may now be specified through QtDialog
>> (cmake-gui) via a new text field in the first-time configure
>> wizard (below the generator chooser).
>
> Thanks for working on this. I think QCMake::setBinaryDirectory
> also needs to check for CMAKE_GENERATOR_TOOLSET from an existing
> cache file much like it already does for CMAKE_GENERATOR. One
> may currently use the "Add Entry" button to pre-define an entry
> for CMAKE_GENERATOR_TOOLSET to set this from the GUI indirectly.
>
> Since not all generators support this field, we should not present
> it when a non-supporting generator is selected. This will need
> some type of query on the selected generator to be added.
>
> Thanks,
> -Brad
>
More information about the cmake-developers
mailing list