Marco Nenciarini a écrit :
On Tue, Jun 05, 2007 at 12:38:19PM +0200, Dave Neary wrote:
Hi,

Aurélien Gâteau wrote:
Marco Nenciarini a écrit :
I don't known if it is usevull, but the provided workarround does not
works[1] :-/
Argh... it seems coredumper is unconditionally linked in owutil. Will
try to have a look at it, but I am particularly squeezed these days.
Marco or Andreas, do you have any spare cycles to help remove this
dependency?


I'm actually working on it.

I cooked together a crude hack for this, see attached patch. You might find it handy. If it's ok, all it's remaining is to move the CMake option from the wengophone to the owutil dir.

Aurélien
Index: libs/owutil/memorydump/src/unix/UNIXMemoryDump.cpp
===================================================================
--- libs/owutil/memorydump/src/unix/UNIXMemoryDump.cpp	(révision 11334)
+++ libs/owutil/memorydump/src/unix/UNIXMemoryDump.cpp	(copie de travail)
@@ -19,6 +19,7 @@
 
 #include <memorydump/UNIXMemoryDump.h>
 
+#ifdef ENABLE_CRASHREPORT
 #include <google/coredumper.h>
 
 #include <util/Logger.h>
@@ -61,4 +62,16 @@
 		);
 	}
 }
+#else
+UNIXMemoryDump::UNIXMemoryDump(const std::string & applicationName, const std::string & revision)
+	: MemoryDump(applicationName, revision) {
+}
 
+UNIXMemoryDump::~UNIXMemoryDump() {
+}
+
+void UNIXMemoryDump::doJob() {
+}
+
+#endif /* ENABLE_CRASHREPORT */
+
Index: libs/owutil/memorydump/CMakeLists.txt
===================================================================
--- libs/owutil/memorydump/CMakeLists.txt	(révision 11334)
+++ libs/owutil/memorydump/CMakeLists.txt	(copie de travail)
@@ -23,9 +23,11 @@
 			src/mac/MACMemoryDump.cpp
 		)
 	else (APPLE)
-		ow_use_public_libraries(
-			coredumper
-		)
+		if (ENABLE_CRASHREPORT)
+			ow_use_public_libraries(
+				coredumper
+			)
+		endif (ENABLE_CRASHREPORT)
 		ow_add_sources(
 			src/unix/UNIXMemoryDump.cpp
 		)
_______________________________________________
Wengophone-devel mailing list
[email protected]
http://dev.openwengo.com/mailman/listinfo/wengophone-devel

Reply via email to