taskWkCMake - Tasks: task #7614, WkCMake to have per target compile...

Show feedback again

You are not allowed to post comments on this tracker with your current authentification level.

task #7614: WkCMake to have per target compile / linkage options

Submitted by:  Alexandre VINCENT <asmodehn>
Submitted on:  Tue 28 Aug 2012 08:12:39 AM UTC  
Should Start On: Mon 27 Aug 2012 12:00:00 PM UTCShould be Finished on: Wed 26 Sep 2012 03:00:00 PM UTC
Category: NonePriority: 7 - High
Status: Ready For TestPrivacy: Public
Percent Complete: 100%Assigned to: Alexandre VINCENT <asmodehn>
Open/Closed: OpenEffort: 0.00

Wed 29 Aug 2012 02:26:37 PM UTC, comment #1:

This has been implemented in revision 189.
CMake command interface hasnt changed and this should be backward compatible without any issues.
However CMake 2.6 might not be supported anymore...
Thorough tests on all projects using WkCMake are needed.

Alexandre VINCENT <asmodehn>In charge of this item.
Tue 28 Aug 2012 08:12:39 AM UTC, original submission:

Currently wkcmake is overriding compile settings setup by CMake.
1) CMake has improved a lot since the beginning of WkCMake, and overriding the compile and linkage option might not be necessary anymore
2) We may want different option of compilation for different projects ( even if in the same wkcmake run - should be done with add_subdirectory ).

Since we have only one main target per project, we need to use per target compilation option.
Something like :
set_target_properties(myexe_target PROPERTIES COMPILE_FLAGS "-Wall")

This way we can also avoid overriding global cmake options, and keep reasonable defaults for other targets.

Alexandre VINCENT <asmodehn>In charge of this item.


No files currently attached


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by asmodehn (Submitted the item)

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.


    Error: not logged in



    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 16 Aug 2016 06:29:28 AM UTCcazfiDependencies-=>task #7966 is dependent
    Wed 29 Aug 2012 02:26:37 PM UTCasmodehnStatusNone=>Ready For Test
      Percent Complete0%=>100%
    Wed 29 Aug 2012 09:43:30 AM UTCasmodehnShould be Finished onMon 27 Aug 2012 12:00:00 PM UTC=>Wed 26 Sep 2012 03:00:00 PM UTC
    Show feedback again

    Back to the top

    Powered by Savane 3.1-cleanup