Re: [cmake-developers] AUTOGEN per-config sources

2017-07-18 Thread Sebastian Holtermann


Am 17.07.2017 um 16:20 schrieb Brad King:
> On 07/15/2017 08:37 PM, comic fans wrote:
>> recent commits shows that QtAutogen is moving to create suffixed files
>> and folders for different config, but with this commit,
>> my generator test failed on Qt5Autogen as following:
>>
>> Target "rccDepends" has source files which vary by configuration.  This is
>>  not supported by the "Fastbuild" generator.
>>
>>  Config "Debug":
>>
>>/working/CMake/Tests/QtAutogen/rccDepends/main.cpp
>>/working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/res1.qrc
>>/working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/res2.qrc
>>
>> /working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/rccDepends_autogen/EJRQKI7XPS/qrc_res1_Debug.cpp
>>
>> /working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/rccDepends_autogen/EJRQKI7XPS/qrc_res2_Debug.cpp
>>
>>  Config "Release":
>>
>>/working/CMake/Tests/QtAutogen/rccDepends/main.cpp
>>/working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/res1.qrc
>>/working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/res2.qrc
>>
>> /working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/rccDepends_autogen/EJRQKI7XPS/qrc_res1_Release.cpp
>>
>> /working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/rccDepends_autogen/EJRQKI7XPS/qrc_res2_Release.cpp
>>
>> How should I make my generator compatible with this ? or this feature
>> didn't complete ,I should wait it complete ?
> 
> The goal for Qt AUTOGEN features is to have the generated files use
> per-config locations in multi-config generators.  However, our
> multi-config generators don't all fully support per-config sources yet.
> Your new generator should do so if possible. See GetAllConfigSources
> in cmGeneratorTarget.
> 
> Sebastian, FYI comic fans is working on a new multi-config generator.

It might be better to whitelist generators in
AUTOGEN to use per-config sources than blacklisting them.
I'll have another look at it later.

-Sebastian
-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers


Re: [cmake-developers] AUTOGEN per-config sources

2017-07-17 Thread Brad King
On 07/15/2017 08:37 PM, comic fans wrote:
> recent commits shows that QtAutogen is moving to create suffixed files
> and folders for different config, but with this commit,
> my generator test failed on Qt5Autogen as following:
> 
> Target "rccDepends" has source files which vary by configuration.  This is
>  not supported by the "Fastbuild" generator.
> 
>  Config "Debug":
> 
>/working/CMake/Tests/QtAutogen/rccDepends/main.cpp
>/working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/res1.qrc
>/working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/res2.qrc
>
> /working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/rccDepends_autogen/EJRQKI7XPS/qrc_res1_Debug.cpp
>
> /working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/rccDepends_autogen/EJRQKI7XPS/qrc_res2_Debug.cpp
> 
>  Config "Release":
> 
>/working/CMake/Tests/QtAutogen/rccDepends/main.cpp
>/working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/res1.qrc
>/working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/res2.qrc
>
> /working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/rccDepends_autogen/EJRQKI7XPS/qrc_res1_Release.cpp
>
> /working/CMake/fastbuild/Tests/Qt5Autogen/rccDepends/rccDepends_autogen/EJRQKI7XPS/qrc_res2_Release.cpp
> 
> How should I make my generator compatible with this ? or this feature
> didn't complete ,I should wait it complete ?

The goal for Qt AUTOGEN features is to have the generated files use
per-config locations in multi-config generators.  However, our
multi-config generators don't all fully support per-config sources yet.
Your new generator should do so if possible. See GetAllConfigSources
in cmGeneratorTarget.

Sebastian, FYI comic fans is working on a new multi-config generator.

-Brad

-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers