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 Aug 28 08:12:39 2012  
 
Should Start On: Mon Aug 27 12:00:00 2012Should be Finished on: Wed Sep 26 15:00:00 2012
Category: NonePriority: 7 - High
Status: Ready For TestPrivacy: Public
Percent Complete: 100%Assigned to: Alexandre VINCENT <asmodehn>
Open/Closed: OpenEffort: 0.00

Wed Aug 29 14:26:37 2012, 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 Aug 28 08:12:39 2012, 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 Aug 16 06:29:28 2016cazfiDependencies-=>task #7966 is dependent
    Wed Aug 29 14:26:37 2012asmodehnStatusNone=>Ready For Test
      Percent Complete0%=>100%
    Wed Aug 29 09:43:30 2012asmodehnShould be Finished onMon Aug 27 12:00:00 2012=>Wed Sep 26 15:00:00 2012
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup