Review Request 111130: [outputview/tests] Fix build when PROJECTS_SOURCE_DIR contains spaces

Aleix Pol Gonzalez aleixpol at kde.org
Wed Jun 19 17:17:00 UTC 2013



> On June 19, 2013, 4:53 p.m., Aleix Pol Gonzalez wrote:
> > Don't you get the same error in other parts of KDevelop? we're using this trick in many places...
> 
> Andrea Scarpino wrote:
>     yep...
>     
>     project/tests/CMakeLists.txt:#add_definitions( -DBUILD_DIR="\\"${KDevPlatform_BINARY_DIR}\\"" )
>     shell/tests/CMakeLists.txt:#add_definitions( -DBUILD_DIR="\\"${KDevPlatform_BINARY_DIR}\\"" )
>     
>     KDevelop:
>     debuggers/gdb/printers/tests/CMakeLists.txt:add_definitions("-DPRINTER_BIN_DIR=\\\"${CMAKE_CURRENT_BINARY_DIR}\\\"")
>     documentation/qthelp/tests/CMakeLists.txt:add_definitions( "-DQTHELP_FILES=\\\"${CMAKE_CURRENT_SOURCE_DIR}/documentation\\\"" )
>     projectmanagers/custom-buildsystem/tests/CMakeLists.txt:add_definitions( "-DPROJECTS_SOURCE_DIR=\\\"${CMAKE_CURRENT_SOURCE_DIR}/projects\\\"" )

Why is it that you didn't hit it on those?

Can you propose a patch that fixes all of them? :D


- Aleix


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111130/#review34685
-----------------------------------------------------------


On June 19, 2013, 4:38 p.m., Andrea Scarpino wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111130/
> -----------------------------------------------------------
> 
> (Updated June 19, 2013, 4:38 p.m.)
> 
> 
> Review request for KDevelop and Morten Volden.
> 
> 
> Description
> -------
> 
> - Fix build when PROJECTS_SOURCE_DIR contains spaces
> - Define PROJECTS_SOURCE_DIR as constant, not as a compiler flag.
> 
> 
> Diffs
> -----
> 
>   outputview/tests/CMakeLists.txt 2be5f82 
>   outputview/tests/outputview-tests.h.cmake PRE-CREATION 
>   outputview/tests/testlinebuilderfunctions.h 653e484 
> 
> Diff: http://git.reviewboard.kde.org/r/111130/diff/
> 
> 
> Testing
> -------
> 
> Builds.
> 
> 
> Thanks,
> 
> Andrea Scarpino
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20130619/f935b2a8/attachment.html>


More information about the KDevelop-devel mailing list