<div class="gmail_quote">On Tue, Sep 21, 2010 at 11:03 PM, Milian Wolff <span dir="ltr"><<a href="mailto:mail@milianw.de">mail@milianw.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hey guys,<br>
<br>
I need your feedback on this. Apparently some people are using my external<br>
scripts plugin to run some magic build scripts, like e.g.:<br>
<br>
<a href="https://bugs.kde.org/show_bug.cgi?id=251983" target="_blank">https://bugs.kde.org/show_bug.cgi?id=251983</a><br>
<br>
This is great since that was definitely one of the reasons I wrote that plugin.<br>
Anyways, people are not yet satisfied and want (rightly so!) the features we<br>
give them for plain make build jobs, esp. the highlighting of errors/warnings<br>
and the ability to jump to those positions inside the code.<br>
<br>
I want to give that to the users of custom buildsystems as well, currently I'd<br>
imagine a checkbox in the configuration of an external script, which you can<br>
tick to enable this "error detection".<br>
<br>
What I want to prevent is duplicated code. The code for MakeJobs works fine it<br>
seems, so I'd be interested in reusing that. The real question is: how,<br>
without making it a API mess that cannot stay BC.<br>
<br>
Right now I think the best way would be to put the code in<br>
MakeOutputModel::addLines , that decides whether a line has an error or<br>
action, into a library so one could reuse it. But I'm not sure whether<br>
returning a FilteredItem from this function is fine, considering BC. I could<br>
make it a proper class with dptr if it's preferred, or is using that struct<br>
there OK?<br>
<br>
Furthermore I'd need MakeOutputDelegate and quite some code currently found in<br>
MakeOutputModel, like activate, next/prevHighlightIndex, urlForFile...<br>
<br>
So should I maybe export the whole MakeOuputModel and rebase my own model on<br>
that somehow? I.e. put MakeOutputModel into kdevplatform/outputview?<br>
<br>
Too many maybes, whats and unsures for me. Please give me some help :)<br>
<br>
Bye<br>
<font color="#888888">--<br>
Milian Wolff<br>
<a href="mailto:mail@milianw.de">mail@milianw.de</a><br>
<a href="http://milianw.de" target="_blank">http://milianw.de</a><br>
</font><br>--<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
<br></blockquote></div><br><div>Maybe it would be easier to add support for custom targets? I think that our makebuilder should support all kinds of interaction a developer might need against make. No?</div><div><br></div>
<div>If you used the MakeOutputModel, how would you let the user tell if he's using make command or not? isn't that workaround-ish?</div><div><br></div><div>Aleix</div>