Review Request 119209: fix build for applications

Giorgos Tsiapaliokas giorgos.tsiapaliokas at kde.org
Thu Jul 10 17:09:40 UTC 2014



> On July 10, 2014, 2:37 p.m., Kevin Funk wrote:
> > Ship It!
> 
> Kevin Funk wrote:
>     That's not the only issue right now. We have do a bulk commit to fix all of the public headers along the lines of your patch.

> We have do a bulk commit to fix all of the public headers along the lines of your patch.

Until now this is the only one that I came across. I would be happy to help with the porting of those.


- Giorgos


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


On July 10, 2014, 2:20 p.m., Giorgos Tsiapaliokas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119209/
> -----------------------------------------------------------
> 
> (Updated July 10, 2014, 2:20 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> fix build for applications
> 
> Applications which depend on kdevplatformvcs weren't able to build
> because DvcsJob wasn't able to find "vcsexport.h".
> 
> 
> Diffs
> -----
> 
>   vcs/dvcs/dvcsjob.h f0d5aeecf8c20b5cd01a86c40d6184db6370301e 
> 
> Diff: https://git.reviewboard.kde.org/r/119209/diff/
> 
> 
> Testing
> -------
> 
> kdevplatform, kdevelop and plasmate builds with this patch.
> 
> 
> Thanks,
> 
> Giorgos Tsiapaliokas
> 
>

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


More information about the KDevelop-devel mailing list