Review Request 124201: Extend builder API to be able to install into a suggested prefix

[Aleix Pol Gonzalez]

> On June 29, 2015, 4:07 p.m., Milian Wolff wrote:
> > project/interfaces/iprojectbuilder.h, line 54
> > <https://git.reviewboard.kde.org/r/124201/diff/1/?file=381668#file381668line54>
> >
> >     mention the specificPrefix?
> >     
> >     also, could/should we merge it with the above install method and give a default arg to `const QUrl& specificPrefix = {}`?
> >     
> >     style: put `&` next to typename.

Mh, I thought about it, I considered it maybe was a good idea to keep them separate as it's something less common, but maybe you're right and we can just have the one.


- Aleix


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


On June 28, 2015, 8:37 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124201/
> -----------------------------------------------------------
> 
> (Updated June 28, 2015, 8:37 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> Due to some work stuff I'm doing, I realized it would be really cool if I could automatize some package generation (for the curious: kde:scratch/apol/kdev-debdeploy).
> 
> What we're trying to achieve, is exactly what make's DESTDIR coding convention does:
> https://www.gnu.org/prep/standards/html_node/DESTDIR.html
> 
> See KDevelop patch, which is where things actually change: https://git.reviewboard.kde.org/r/124202/
> 
> 
> Diffs
> -----
> 
>   project/interfaces/iprojectbuilder.h 9d65ff6 
> 
> Diff: https://git.reviewboard.kde.org/r/124201/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

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