Review Request: Fix MakeOutput items for cmake builds with relative paths

Milian Wolff mail at milianw.de
Mon Feb 28 21:03:22 UTC 2011


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

Ship it!


Why are you using a QLinkedList? And there is a problem with i18n usage imo:

a) potential_cd -> that does not sound like a string that should be translated, and i18n is only used in one of the two places that is used
b) all new i18n strings are missing context, they probably need to be exactly the same as outputted by cmake, or?


projectbuilders/makebuilder/makeoutputmodel.cpp
<http://git.reviewboard.kde.org/r/100745/#comment1416>

    why i18n here but not above/below?


- Milian


On Feb. 27, 2011, 4:03 p.m., Manuel Massing wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100745/
> -----------------------------------------------------------
> 
> (Updated Feb. 27, 2011, 4:03 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Summary
> -------
> 
> Fixes MakeOutput items for cmake builds which use relative paths 
> (i.e. cmake projects with CMAKE_USE_RELATIVE_PATHS set to on).
> 
> Previously, errors generated by source files with relativ paths
> were not clickable because MakeOutputModel does not know about
> the directories they are built in.
> 
> Because cmake is not verbose enough to indicate directory changes,
> observes where object files are being built and use these paths
> as search path for urlToFile.
> 
> 
> Diffs
> -----
> 
>   projectbuilders/makebuilder/makeoutputmodel.h 5ac45535a0d6d5496c48631d281e322c7b1b792e 
>   projectbuilders/makebuilder/makeoutputmodel.cpp 09bb72e4621e8033666d6c0c36c1fc34834754a9 
>   projectbuilders/makebuilder/outputfilters.cpp 3b6b806d73375daf85831f734153a5bf4e2293d6 
> 
> Diff: http://git.reviewboard.kde.org/r/100745/diff
> 
> 
> Testing
> -------
> 
> Tested on cmake project with SET(CMAKE_USE_RELATIVE_PATHS ON).
> 
> 
> Thanks,
> 
> Manuel
> 
>

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


More information about the KDevelop-devel mailing list