*** tpb has joined #freeorion | 00:00 | |
*** Stranger has joined #freeorion | 02:05 | |
*** O01eg has joined #freeorion | 02:05 | |
*** GeofftheMedio has joined #freeorion | 02:05 | |
*** STalKer-X has joined #freeorion | 02:05 | |
*** aanderse has joined #freeorion | 02:05 | |
*** custos-mundi has joined #freeorion | 02:05 | |
*** theTroy has joined #freeorion | 02:05 | |
*** planetmaker has joined #freeorion | 02:05 | |
*** Nagilum_ has joined #freeorion | 02:05 | |
*** VargaD has joined #freeorion | 02:05 | |
*** ioo has joined #freeorion | 02:05 | |
*** STalKer-X_i has joined #freeorion | 03:37 | |
*** STalKer-X has quit IRC | 03:37 | |
*** Quix0r has joined #freeorion | 07:57 | |
*** O01eg has quit IRC | 08:36 | |
*** frkarl has joined #freeorion | 09:20 | |
*** Stranger has quit IRC | 09:23 | |
*** i4c has joined #freeorion | 10:09 | |
Nagilum_ | CMake Error at GG/cmake/Util.cmake:127 (add_library): Cannot find source file: | 10:10 |
---|---|---|
Nagilum_ | OpenSteer/Obstacle.cpp | 10:10 |
adrian_broher | didn't had time yet to fix this. | 10:17 |
adrian_broher | I'm still compiling for max | 10:17 |
adrian_broher | mac | 10:17 |
GeofftheMedio | in root FreeOrion CMakeLists.txt | 10:23 |
GeofftheMedio | change OpenSteer/ to OpenSteer/source/ | 10:23 |
GeofftheMedio | on lines 146-148 | 10:23 |
Nagilum_ | k, same applies to PagedGeometry | 10:28 |
Nagilum_ | [ 79%] Building CXX object client/AI/CMakeFiles/freeorionca.dir/__/__/util/VarText.cpp.o | 10:45 |
Nagilum_ | /root/freeorion/src/freeorion-build/UI/CombatWnd.cpp:34:23: fatal error: BatchPage.h: No such file or directory #include "BatchPage.h" | 10:45 |
adrian_broher | <> instead of "" | 10:46 |
*** don_rzl has quit IRC | 10:47 | |
GeofftheMedio | what difference would that make? | 10:49 |
Nagilum_ | different search path iirc | 10:49 |
adrian_broher | <> includes from the compiler search pathes. | 10:50 |
GeofftheMedio | not just order pathes are searched? | 10:50 |
adrian_broher | "" includes from the directory the file is located | 10:51 |
Nagilum_ | doesn't help in this case though | 10:51 |
GeofftheMedio | what I'm reading says both do both, but just in a different order | 10:51 |
*** EXetoC has joined #freeorion | 10:51 | |
GeofftheMedio | line 121 of CMakeLists.txt | 10:52 |
adrian_broher | -include_directories(${CMAKE_SOURCE_DIR} /PagedGeometry/include) | 10:52 |
adrian_broher | +include_directories(${CMAKE_SOURCE_DIR}/PagedGeometry/include) | 10:52 |
GeofftheMedio | add PagedGeometry/include | 10:52 |
GeofftheMedio | ... yes | 10:52 |
adrian_broher | take a look at the space | 10:52 |
adrian_broher | that shouldn't be there | 10:52 |
GeofftheMedio | nmind my suggestion wrong cmakelists | 10:53 |
GeofftheMedio | or the first / shouldn't be there | 10:53 |
GeofftheMedio | to be consistent with other uses of the include_directories function | 10:53 |
GeofftheMedio | so in the /client/human/CMakeLists.txt line 64 remove the first / | 10:54 |
Nagilum_ | yep, that looks better | 10:55 |
Nagilum_ | oh wait | 10:56 |
Nagilum_ | no, still not found | 10:56 |
GeofftheMedio | make CMAKE_SOURCE_DIR is different here since it's a different CMakeLists.txt? | 10:57 |
GeofftheMedio | add ../../ on the front of the path? | 10:57 |
adrian_broher | CMAKE_SOURCE_DIR equals to what you're giving cmake as parameter | 10:57 |
GeofftheMedio | that doesn't help | 10:58 |
adrian_broher | resolved as canonical path. | 10:58 |
Nagilum_ | GeofftheMedio: let me try that, that should work imho | 10:58 |
GeofftheMedio | nor does that | 10:58 |
*** don_rzl has joined #freeorion | 11:00 | |
Nagilum_ | #include "../../PagedGeometry/include/BatchPage.h" | 11:00 |
GeofftheMedio | ? | 11:01 |
GeofftheMedio | I meant a_h's explanation meant nothing to me | 11:01 |
Nagilum_ | ah .-) | 11:01 |
adrian_broher | ah geeze | 11:02 |
adrian_broher | wait a second, have a patch | 11:02 |
Nagilum_ | if I had a server I would set up jenkins, then anyone interested would be notified automatically when something broke.. | 11:14 |
GeofftheMedio | breaking was expected in this case | 11:15 |
Nagilum_ | i see | 11:15 |
adrian_broher | http://www.freeorion.org/forum/viewtopic.php?f=9&t=7367&p=60240#p60240 | 11:20 |
tpb | <http://ln-s.net/+xhV> (at www.freeorion.org) | 11:21 |
adrian_broher | Nagilum_, GeofftheMedio ^ | 11:21 |
GeofftheMedio | what's the deal with the include_directories line? | 11:23 |
adrian_broher | it's two entries you specify in your line. | 11:24 |
GeofftheMedio | I mean why "${CMAKE_SOURCE_DIR}/PagedGeometry/include" and not "${CMAKE_SOURCE_DIR} PagedGeometry/include" | 11:25 |
adrian_broher | your version adds two include pathes, 1. whatever CMAKE_SOURCE_DIR evaluates to, and the absolute path '/PagedGeometry/include' | 11:25 |
adrian_broher | because the second one doesn't allow out of source builds. | 11:26 |
adrian_broher | http://www.cmake.org/pipermail/cmake/2009-November/033051.html# | 11:27 |
tpb | <http://ln-s.net/+xhg> (at www.cmake.org) | 11:27 |
GeofftheMedio | what's going on with the root FreeOrion CMakeLists.txt then? | 11:27 |
GeofftheMedio | absolute paths to /log4cpp and /OpenSteer/include ? | 11:27 |
adrian_broher | there is no leading / | 11:30 |
adrian_broher | still, relative pathes call for trouble. That's why CMAKE_SOURCE_DIR exists in the first place. | 11:31 |
GeofftheMedio | there is no leading / in my "and not" case above | 11:31 |
adrian_broher | http://www.cmake.org/Wiki/CMake_FAQ#Why_does_CMake_use_full_paths.2C_or_can_I_copy_my_build_tree.3F | 11:31 |
tpb | <http://ln-s.net/+xho> (at www.cmake.org) | 11:31 |
adrian_broher | The developers themself suggest to use absolute pathes. | 11:32 |
GeofftheMedio | so should the root FreeOrion CMakeLists.txt be changed then? | 11:33 |
adrian_broher | probably yes. | 11:33 |
Nagilum_ | that patch doesn't apply | 11:33 |
adrian_broher | it's right from master | 11:33 |
adrian_broher | or tip or head | 11:34 |
GeofftheMedio | git / svn ? | 11:34 |
adrian_broher | don't know the svn terminology | 11:34 |
GeofftheMedio | you did a diff -git | 11:34 |
GeofftheMedio | should do an svn diff | 11:35 |
Nagilum_ | http://pastebin.com/y1USnS6r | 11:35 |
tpb | Title: [root@arch64 freeorion]# svn revert -R . Reverted 'CMakeLists.txt' [root@arch6 - Pastebin.com (at pastebin.com) | 11:35 |
Nagilum_ | fett, aber die StromRechnung... | 11:42 |
Nagilum_ | ups | 11:42 |
*** EXetoC has quit IRC | 11:48 | |
adrian_broher | Nagilum_, | 11:51 |
adrian_broher | patch -Np0 < ../cmake-fix.patch | 11:51 |
adrian_broher | works as expected | 11:51 |
*** i4c has quit IRC | 12:06 | |
*** EXetoC has joined #freeorion | 12:07 | |
Nagilum_ | adrian_broher: ah, yes | 12:11 |
*** i4c has joined #freeorion | 12:18 | |
Nagilum_ | build succeeds with the patch :) | 12:27 |
adrian_broher | wunderbar | 12:52 |
adrian_broher | Nagilum_, fyi: p0 ist zwingend notwendig, da ansonsten die pfade im patch zum basename zusammengekürzt werden. | 13:02 |
Nagilum_ | ah! | 14:34 |
*** jinxdone has quit IRC | 14:39 | |
STalKer-X_i | kompliziert! | 17:01 |
Nagilum_ | nope, I was just lazy | 17:08 |
Nagilum_ | too lazy to look at the patch and the patch manpage | 17:09 |
STalKer-X_i | ewww! | 17:15 |
STalKer-X_i | you still need like... 20GB to compile FO? :D | 17:16 |
Nagilum_ | no | 17:17 |
STalKer-X_i | just 5, right? | 17:17 |
Nagilum_ | you need 20GB ram if you compile with gcc4.8 and -j16 | 17:17 |
STalKer-X_i | gcc is the devil | 17:18 |
Nagilum_ | my buildroot is 3.2GB, but of course thats bare minimum including the whole toolchain | 17:19 |
STalKer-X_i | sounds like stuff, EA would do. | 17:20 |
Nagilum_ | EA? | 17:20 |
STalKer-X_i | Electronic Arts :-) | 17:22 |
Nagilum_ | no idea about their games, I'm not a gamer, last time I saw one of their games was on the Amiga :> | 17:27 |
STalKer-X_i | old man. | 17:39 |
Nagilum_ | https://cakebox.homeunix.net/~nagilum/build.log rev5951 has a few warnings that might be interesting.. | 18:27 |
STalKer-X_i | should i accept the certificate? :p | 18:55 |
Nagilum_ | SHA1 Fingerprint=68:BE:F4:74:81:07:07:CE:A7:E8:03:6A:7A:66:E6:22:86:3B:A8:FF | 18:58 |
Nagilum_ | if thats what your browser tells you, you should be fine | 18:58 |
STalKer-X_i | yay | 19:00 |
*** i4c has quit IRC | 19:03 | |
*** Xenux has joined #freeorion | 19:26 | |
*** Xenux has quit IRC | 19:28 | |
*** Xenux has joined #freeorion | 19:28 | |
*** dansan has joined #freeorion | 20:34 | |
*** Stranger has joined #freeorion | 20:57 | |
*** i4c has joined #freeorion | 22:00 | |
GeofftheMedio | Naglium_: VarText.cpp:113:21: warning: variable 'log_id' set but not used .... that variable is used just below? | 22:33 |
Nagilum_ | not really | 22:40 |
GeofftheMedio | int log_id = -1; // should not be a valid log id, for default | 22:41 |
GeofftheMedio | try { | 22:41 |
GeofftheMedio | log_id = boost::lexical_cast<int>(token_elem.Attribute("value")); | 22:41 |
GeofftheMedio | } | 22:41 |
Nagilum_ | it's assigned a value but thats not used anywhere if I'm seing this right | 22:41 |
GeofftheMedio | since when are you not supposed to assign initial values if they might / do get changed later? | 22:41 |
Nagilum_ | its just telling you that its not used so you might either want to use it (for error reporting or whatever) or delete it | 22:43 |
Nagilum_ | at least thats what I think this warning is supposed to express | 22:43 |
*** jinxdone has joined #freeorion | 22:45 | |
GeofftheMedio | no luck getting into the sourceforge account? | 23:38 |
Nagilum_ | never heard from them :( | 23:39 |
Nagilum_ | but let me try again | 23:39 |
*** Xenux has quit IRC | 23:43 | |
*** GeofftheMedio has quit IRC | 23:43 | |
*** GeofftheMedio has joined #freeorion | 23:44 |
Generated by irclog2html.py 2.5 by Marius Gedminas - find it at mg.pov.lt!