Review Request 119968: add compiler sanitizers support

[Mathieu Tarral]

> On août 28, 2014, 10:57 matin, Albert Astals Cid wrote:
> > How would i compile karchive with this? does karchive need any modification?

I don't know the build process for Karchive, neither for KDE in general,
but you should have to modify your buildsystem or any of your flags.
The goal of this module is to handle errors, misconfigurations and
right compiler flags internaly.

It aims to be as simple as :
include (ECMEnableSanitizers)

and then :
cmake -DCMAKE_ENABLE_SANITIZERS ..
make karchive ?

Please tell me more about the way you configure and build karchive
so we can test this module integration.


- Mathieu


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119968/#review65411
-----------------------------------------------------------


On août 28, 2014, 10:33 matin, Mathieu Tarral wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119968/
> -----------------------------------------------------------
> 
> (Updated août 28, 2014, 10:33 matin)
> 
> 
> Review request for Build System and Extra Cmake Modules.
> 
> 
> Repository: extra-cmake-modules
> 
> 
> Description
> -------
> 
> This module uses a cached variable to hold a list of compiler sanitizers
> to enable.
> It parses that list, checks the sanitizer desired options, then checks the compiler
> support and set the appropriate flags.
> 
> 
> Diffs
> -----
> 
>   modules/ECMEnableSanitizers.cmake PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/119968/diff/
> 
> 
> Testing
> -------
> 
> cmake -DECM_ENABLE_SANITIZERS=address;undefined
> make VERBOSE=1
> 
> 
> Thanks,
> 
> Mathieu Tarral
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20140828/4a547c1d/attachment.html>