Review Request 117835: Now IDM (Includes/Defines manager) provides all types of includes/defines, not just user-specified. (kdevplatform)

[Milian Wolff]

> On April 29, 2014, 12:05 p.m., Aleix Pol Gonzalez wrote:
> > I think this change makes the development of project managers more complex for no reasons.
> > 
> > I don't think project managers should be aware of the includes you want to bypass to the language support.
> 
> Sergey Kalinichev wrote:
>     >I don't think project managers should be aware of the includes you want to bypass to the language support.
>     
>     Huh? I think I lost the thread... What are you talking about?
> 
> Aleix Pol Gonzalez wrote:
>     I'm saying that I don't think an implementation of a project manager should need to be aware of IDefinesAndIncludesManager. You can have the implementations of IDefinesAndIncludesManager to access the IBuildSystemManager interface or whatever the user defined.
>     
>     To put it differently, what's the point of implementing a buildsystem manager without includes and defines?

Aleix, I'm with Sergey here. At least two of our project managers already don't implement this interface. With the new plugin to define custom include paths/defines, it might be even more eventually.

Then, thinking about the future, we need some kind of interface for other plugins to implement to provide more defines/include paths. Esp. for cross compile toolchains, we must be able to define different compilers to grab the default includes/defines from etc. pp. As such, I think the approach given here goes into the right direction.

And, furthermore, what differences does it make from a IBSM pov? It could just implement the new interface and register itself to the IDAIM, no? I don't think its making things overly complex compared to before?


- Milian


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


On April 28, 2014, 11:03 a.m., Sergey Kalinichev wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117835/
> -----------------------------------------------------------
> 
> (Updated April 28, 2014, 11:03 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> The changes include extending IDM interface and removing i/d notion from IBuildSystemManager interface.
> 
> 
> Diffs
> -----
> 
>   language/interfaces/idefinesandincludesmanager.h ab42444 
>   project/interfaces/ibuildsystemmanager.h 241b696 
> 
> Diff: https://git.reviewboard.kde.org/r/117835/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergey Kalinichev
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140430/0dfd6496/attachment.html>