Review Request 128012: Check for an existing out-of-source build dir

[Kevin Funk]

> On June 2, 2016, 9:26 p.m., Milian Wolff wrote:
> > project/helper.cpp, line 221
> > <https://git.reviewboard.kde.org/r/128012/diff/3/?file=465463#file465463line221>
> >
> >     I feel really bad with this... Yes - it's powerful, but it's also overkill, no? Who would ever write such a script?
> 
> Kevin Funk wrote:
>     I also think the custom script is overkill. We should just default to sane default paths.
>     
>     +1 for discarding this idea. Sorry, Rene.
> 
> Alex Richardson wrote:
>     I would use this feature but I'm not sure running a custom script is the best solution (it is however quite easy to implement). No one would ever use it because it is very hard to discover. Maybe there should be an option in the settings menu.
>     
>     I have the following structure for my projects: source dirs are in /data/sources/ and build is in /build/ (separate partition because I don't have enough space on my SSD). It would be quite nice if I wouldn't have to manually set the build directory for every new project that I open.
>     
>     If I had known that kdevelop replaces /src/ with /build/ (as I found out by reading this diff) I could have come up with a different hierachy that wouldn't require running a custom script.

Yes, I don't think this is super discoverable as is (using an external script).

I think what *could work* is a line edit in the settings where can specify the translation from source directory to build directory (via string having the project name as placeholder). E.g. it would be "/build/${projectname}" in your case, and would default to something along "${projectpath/\/src/\/build}" (undecided about the syntax, can't think of anything more appropriate right now than Bash's string replacement syntax).

Anyway, I don't think it needs to be / or should be an external script (way to complex).


- Kevin


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


On May 25, 2016, 8:24 p.m., René J.V. Bertin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128012/
> -----------------------------------------------------------
> 
> (Updated May 25, 2016, 8:24 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> When importing a new project, check if it already has a pre-existing out-of-source build directory, and propose that as the build directory in `KDevelop::proposedBuildFolder()`.
> This is comparable to proposing to put the build folder alongside an existing `src` directory (but has priority over that rule).
> 
> 
> Diffs
> -----
> 
>   project/helper.cpp 6df9f90 
> 
> Diff: https://git.reviewboard.kde.org/r/128012/diff/
> 
> 
> Testing
> -------
> 
> Builds and behaves as expected.
> 
> 
> Thanks,
> 
> René J.V. Bertin
> 
>

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