<br><br><div class="gmail_quote">On Tue, Aug 4, 2009 at 9:18 AM, 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 class="im">On 04.08.09 02:01:12, Aleix Pol wrote:<br>
> On Tue, Aug 4, 2009 at 1:23 AM, Andreas Pakulat <<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>> wrote:<br>
><br>
> > On 03.08.09 22:57:29, Aleix Pol Gonzalez wrote:<br>
> > > SVN commit 1006580 by apol:<br>
> > ><br>
> > > Properly look for Kompare, the CMake way.<br>
> ><br>
> > Why? IMHO its just unnecessary overhead compared to what "finding<br>
> > kompare" actually means. All we need is find 1 single header file.<br>
><br>
> Yes, and in CMake we find using find_package.<br>
<br>
</div>find_package is usually used if you have to do a couple of things to<br>
find a given package. Including finding the headers, the libraries and<br>
setting up all the variables.<br>
<br>
I don't really see any benefit of having a FindKompare.cmake.<br>
<div class="im"><br>
> > > +                        "<a href="http://www.caffeinated.me.uk/kompare/" target="_blank">http://www.caffeinated.me.uk/kompare/</a>" FALSE<br>
> > ""<br>
> > > +                        "Required for difference checking")<br>
> ><br>
> > This website is completely outdated, if we put a webpage here it should<br>
> > at least point to the right sources and list the current developers.<br>
><br>
><br>
> Ok, I can change to: <a href="http://www.google.es/search?q=kompare" target="_blank">http://www.google.es/search?q=kompare</a><br>
<br>
</div>How about asking the maintainer what the official address is and if<br>
there's none just put the websvn address from kdesdk/kompare there?<br>
<div class="im"><br>
> > > +configure_file( ${CMAKE_CURRENT_SOURCE_DIR}/config-kdevplatform.h.cmake<br>
> > > +                ${CMAKE_CURRENT_BINARY_DIR}/config-kdevplatform.h )<br>
> > > +<br>
> ><br>
> > Even more code to execute.<br>
><br>
> I wonder what you mean. Every single cmake project use configure file for<br>
> conditional compilation inside files. I don't think is that much overhead<br>
> anyway. If cmake parsing (either cmake's or our's) is not fast enough is not<br>
> because of this (actually we don't do anything in configure_file).<br>
> I don't think it's superfluous anyway.<br>
<br>
</div>Using a configure-file is good if you have lots of HAVE_XXX in various<br>
places/find-modules. But we only have 1 in there, its really, really<br>
easy to just have add_definitions( -DHAVE_KOMPARE ) inside the<br>
language/CMakeLists.txt and the configure_file doesn't add any value to<br>
this - hence its superfluous.</blockquote><div>Then let's use some of the other files we're already configuring.  Is it so hard to think positively? <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>
> > >  include (KDE4Defaults)<br>
> > >  include (MacroWriteBasicCMakeVersionFile)<br>
> > >  include (MacroLibrary)<br>
> ><br>
> > Why is the above stuff before these? It should be after it.<br>
><br>
> I wanted to add it with the other find_packages. If you prefer to have it<br>
> below I'll move it. It is not a problem.<br>
<br>
</div>Actually, thinking about it, the find-kompare stuff should be in<br>
language/CMakeLists.txt so its clear which part of kdevplatform actually<br>
uses it. <br></blockquote><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>
You will be attacked by a beast who has the body of a wolf, the tail of<br>
a lion, and the face of Donald Duck.<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>The whole idea of moving it was to be able to use it outside the changes widget, that's why I think it's necessary to have the information in a config file.<br>
<br>Aleix<br>