Review Request 108690: Use an actual directory as the working dir for the rev-list job.

Commit Hook null at kde.org
Thu Jan 31 21:57:32 UTC 2013


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


This review has been submitted with commit 837ae2715a37eda62fb0b89d9bc62142016ae259 by eli mackenzie to branch 1.5.

- Commit Hook


On Jan. 31, 2013, 8:01 p.m., Eli MacKenzie wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108690/
> -----------------------------------------------------------
> 
> (Updated Jan. 31, 2013, 8:01 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> I was unable to open the Revision Graph, KDevelop would freeze every time. With the freeze fixed, there was still no contents. I found that the git plugin was trying to use the full path to a file as a directory, which of course QDir did not agree was a directory. I was about to implement a method like urlDir, but found QFileInfo was already included... which lead to this simple patch. 
> 
> 
> Diffs
> -----
> 
>   plugins/git/gitplugin.cpp f1f9baf6ff28ec055155e67f7cb0996c11356c22 
> 
> Diff: http://git.reviewboard.kde.org/r/108690/diff/
> 
> 
> Testing
> -------
> 
> I'm able to open the revision graph from the context menu now.
> 
> 
> Thanks,
> 
> Eli MacKenzie
> 
>

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


More information about the KDevelop-devel mailing list