<table><tr><td style="">rjvbb added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D17794">View Revision</a></tr></table><br /><div><div><p>Continuing from a few thoughts I launched on the original <a href="https://phabricator.kde.org/T10209" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T10209</a>, mainly aimed at keeping the project configuration dialog's left side-bar as unencumbered as possible. I think the current plugin could be merged into the customdefinesandincludes plugin because it provides a programmatic way to add include paths and/or defines.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>blackwarthog added a comment.</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">> You'll get more feedback on this when you present your patch via a differential (aka "patch") review.

Already https://phabricator.kde.org/D17794
Sorry, it was my misunderstanding. May i suggest to add link to these instructions <https://community.kde.org/Infrastructure/Phabricator> to this page <https://www.kdevelop.org/contribute-kdevelop> :)

'custom-definesandincludes' provides generic functionality for injection of cuctom includes and defines into build and indexing sequence. This functionality uses by CMake, Make and other build plugins.</pre></div></blockquote>

<p>Are you certain that information is used for anything other than parsing? That would mean the heading in the side bar (<tt style="background: #ebebeb; font-size: 13px;">Language Support</tt>) isn't a very happy choice.</p>

<p>OTOH, if you're right, certain of the build managers could benefit just like the parser from relevant information injected through pkgconfig, meaning this is actually an argument in favour of patching the customdefines* plugin rather than adding another one.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Pkgconfig functionality is more specialized and not always applicable for MacOS and windows, required to run third-party application etc. So, i think it is not good idea to merge these plugins into one.</p></blockquote>

<p>A project either requires pkgconfig or it doesn't, and the user may or may not need (or want) to rely on pkgconfig to add custom include paths. I don't think the platform used enters into the equation. But if pkgconfig is by definition NOT available on a given target platform the corresponding code can be omitted from the build.<br />
It's relevant to know here if KDevelop can be built without pkgconfig. If not, pkgconfig is by definition available for all platforms where the IDE can run.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">May be we need to export some new interface from 'custom-definesandincludes' to allow for other plugins to put addiional sub-pages into 'Language support' page.</pre></div></blockquote>

<p>That probably requires more of an overhaul than anyone feels like implementing...</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17794">https://phabricator.kde.org/D17794</a></div></div><br /><div><strong>To: </strong>blackwarthog, KDevelop<br /><strong>Cc: </strong>rjvbb, arrowd, kdevelop-devel, KDevelop, glebaccon, hase, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight<br /></div>