<br><br><div class="gmail_quote">On Thu, Jul 9, 2009 at 6:47 PM, Andreas Pakulat <span dir="ltr"><<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="h5">On 09.07.09 16:54:36, Aleix Pol wrote:<br>
> On Wed, Jul 8, 2009 at 9:32 PM, Andreas Pakulat <<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>> wrote:<br>
> > On 08.07.09 18:17:02, Aleix Pol Gonzalez wrote:<br>
> > > SVN commit 993467 by apol:<br>
> > ><br>
> > > A little bit of refactoring and reorganization on the buildsets.<br>
> > > Simplifies the buildset creation a lot.<br>
> > ><br>
> > > If it does not fix the bug, we will have to create some kind of<br>
> > "restructuring" state for the project or just add the new tree to the<br>
> > project when it's ready.<br>
> ><br>
> > Ok, took the time now to do a complete review. I still don't like this<br>
> > commit, it mixes too many things within the same commit.<br>
> ><br>
> > > ---<br>
> > trunk/KDE/kdevplatform/plugins/projectmanagerview/builditembuilderjob.cpp<br>
> > #993466:993467<br>
> > > @@ -29,7 +29,9 @@<br>
> > >  {<br>
> > >      foreach( const BuildItem &item, items )<br>
> > >      {<br>
> > > -        addItem( t, item.findItem() );<br>
> > > +        KDevelop::ProjectBaseItem *it=item.findItem();<br>
> > > +        if(it)<br>
> > > +            addItem( t, it );<br>
> > >      }<br>
> ><br>
> > I don't think this is the right fix for the problem. If an item from the<br>
> > buildset is currently being reloaded, we shouldn't build the buildset at<br>
> > all, because we cannot know what changes are being done to the cmake files<br>
> > (that are the cause of the reload). We should just disable building as long<br>
> > as the reloading takes if a buildset is active.<br>
><br>
> agree. but we still don't want it to crash until it's done.<br>
<br>
</div></div>Well, with the crash we were at least reminded of the problem, now its easy<br>
to forget about it. Besides I actually did plan to look into the crash<br>
today/tomorrow ever since the last additional info was posted :)</blockquote><div> </div><div>Then add a #warning. I don't think having a crash is good for anything.<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<div class="im"><br>
> > >  BuildItem::BuildItem( const BuildItem& rhs )<br>
> > >  {<br>
> > > -    m_itemName = rhs.itemName();<br>
> > > -    m_projectName = rhs.projectName();<br>
> ><br>
> > This is not going to work in the long term. We have open reports from<br>
> > people that would like to put in arbitrary items into the buildset that are<br>
> > not in the project tree. Which is actually needed for proper<br>
> > custom-makefile support (unless someone writes a makefile parser that<br>
> > produces meaningful project tree with all targets).<br>
><br>
> The project is not needed, it's in the path.<br>
<br>
</div>Ok, so I'd get<br>
<br>
foobar | myproject<br>
<br>
In the buildset. That should work.</blockquote><div><br>It's something like<br>target ! path/to/the/target<br>folder ! path/to/the/folder<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<div class="im"><br>
> > We need at least an "item name" to indicate the custom target.<br>
> ><br>
> We can have it in the path some way. Or when we need it, we add it. I don't<br>
> think it's a good idea to have it specified before having it.<br>
<br>
</div>I was about to say that allowing the user to add arbitrary items to the<br>
project tree (which would be needed to have this available as path) opens<br>
up a can of worms. While I still think that way, it also is a nice way of<br>
integrating custom, unkown targets and have the custom-targets support<br>
instantly not only in buildsets but also as possible dependency for<br>
execution.<br><br>
So, ok, you convinced me :)<br>
</blockquote><div>I think we can go far with the setup we have now. When we actually  know for sure what we need, we can extend it.<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
Andreas<br>
<font color="#888888"><br>
--<br>
Try the Moo Shu Pork.  It is especially good today.<br>
</font><div><div></div><div class="h5"><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>
</div></div></blockquote></div><br>