<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Dec 19, 2012 at 2:28 AM, Friedrich W. H. Kossebau <span dir="ltr"><<a href="mailto:kossebau@kde.org" target="_blank">kossebau@kde.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Am Dienstag, 18. Dezember 2012, 18:07:03 schrieb Jeremy Whiting:<br>
<div class="im">> Albert,<br>
><br>
> I got all of kdesdk subfolders to build standalone using the attached<br>
> (huge) patch.<br>
<br>
</div>Good work.<br>
<br>
Hm, maybe not completely, configuring (of complete kdesdk, trunk, with out-of-<br>
source build) fails for me with:<br>
--- 8< ---<br>
CMake Error at kapptemplate/CMakeLists.txt:17 (include):<br>
  include could not find load file:<br>
<br>
    KAppTemplateMacro<br>
<br>
<br>
CMake Error at kapptemplate/templates/C++/CMakeLists.txt:17<br>
(kapptemplate_add_app_templates):<br>
  Unknown CMake command "kapptemplate_add_app_templates".<br>
--- 8< ---<br>
<br>
Did not investigate more, no time ATM, sorry.<br></blockquote><div><br></div><div>That's very weird. it works fine here, and I just checked the file kapptemplate/cmake/macros/KAppTemplateMacro.cmake is in the svn commit, but isn't in the patch file somehow :/  Ok, here's a new patch created with svn diff --show-copies-as-adds which includes it.<br>
<br></div><div>thanks,<br>Jeremy<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
> I want to get someone besides myself to take a look before I<br>
> commit it though, but here it makes it so kdesdk can be built as one<br>
> module, and also each subfolder (except docs) can be built by itself.  I<br>
> removed quite a bit from the top level CMakeLists.txt so it's just a bunch<br>
> of add_subdirectory calls.<br>
<br>
</div>Though that bit was in to support people who just have a few of the kdesk<br>
submodules checked out and not all of them. Changing this would break things<br>
for them.<br>
<br>
So I propose to leave kdesdk_optional_add_subdirectory in for now and use that<br>
instead of add_subdirectory, does not really hurt, does it? :)<br></blockquote><div><br></div><div>Ah, I see, do we really expect people to have a checkout of part of kdesvn? and if so couldn't they just build each part individually that way?  Also it's pretty easy to comment out lines of stuff you don't want to build, I remember doing that quite a bit when I was building kanagram over and over when working on it years ago.<br>
 <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
> I also took out pimlibs detection and couldn't<br>
> find any subdir that needed/used it.  That was either in there from<br>
> something that has since been removed or is an optional dependency for<br>
> something.  If anyone knows which we may need to tweak this before<br>
> committing.<br>
<br>
</div>No idea, cannot remember to have seen something.<br>
<br>
Cheers<br>
<span class="HOEnZb"><font color="#888888">Friedrich<br>
</font></span></blockquote></div><br></div></div>