Review Request 120190: port away from KStandardDirs

Laurent Navet laurent.navet+kde at gmail.com
Sat Sep 13 21:03:13 UTC 2014



> On sep. 13, 2014, 10:47 après-midi, Sven Brauch wrote:
> > Very nice, thanks a lot!
> > 
> > When porting the snippets stuff in kate, I noticed that GenericDataLocation doesn't seem to be what you usually want ... did you test where the data actually ends up?
> > 
> > Do all the tests still pass after you applied this patch?
> 
> Sven Brauch wrote:
>     ... I forgot, the thing I used instead in kate was DataLocation instead of GenericDataLocation.

no, didn't test this, because tests were falling before, and for now, even kdevelop don't start properly, I need to fix my setup first


- Laurent


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


On sep. 13, 2014, 10:37 après-midi, Laurent Navet wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120190/
> -----------------------------------------------------------
> 
> (Updated sep. 13, 2014, 10:37 après-midi)
> 
> 
> Review request for KDevelop and Sven Brauch.
> 
> 
> Repository: kdev-python
> 
> 
> Description
> -------
> 
> replace deprecated KStandardDirs by QStandardPaths
> 
> 
> Diffs
> -----
> 
>   pythonlanguagesupport.cpp 397bd41 
>   codecompletion/context.cpp cd88efd 
>   codecompletion/tests/pycompletiontest.cpp d55430f 
>   debugger/debugjob.cpp 504971f 
>   debugger/debugsession.cpp bbcafe9 
>   docfilekcm/docfilemanagerwidget.cpp 88ab17a 
>   docfilekcm/docfilewizard.cpp da104c5 
>   duchain/assistants/missingincludeassistant.cpp 0b53a74 
>   duchain/contextbuilder.cpp 99c4d9c 
>   duchain/correctionhelper.cpp 04eb329 
>   duchain/helpers.cpp 7676be6 
>   duchain/tests/duchainbench.cpp 4c717e9 
>   duchain/tests/pyduchaintest.cpp c587442 
> 
> Diff: https://git.reviewboard.kde.org/r/120190/diff/
> 
> 
> Testing
> -------
> 
> just build
> 
> 
> Thanks,
> 
> Laurent Navet
> 
>

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


More information about the KDevelop-devel mailing list