From: Jens Thiele <jens.thiele@intra2net.com>
Date: Tue, 5 Dec 2006 16:58:38 +0000 (+0000)
Subject: updated TODOs
X-Git-Tag: v0.2~42
X-Git-Url: http://developer.intra2net.com/git/?p=libt2n;a=commitdiff_plain;h=373558f707dd668486de943dadfb4efe492b07af;hp=b522255ce25f1fc10826b08ffa0228836d79ca0b

updated TODOs
---

diff --git a/codegen/README b/codegen/README
index f5ef756..c0a1598 100644
--- a/codegen/README
+++ b/codegen/README
@@ -1 +1,3 @@
 code generator using gccxml and libxml++
+
+for example usage take a look at the example-codegen directory.
diff --git a/codegen/TODO b/codegen/TODO
index fb1591e..dd9f296 100644
--- a/codegen/TODO
+++ b/codegen/TODO
@@ -11,13 +11,9 @@ open questions:
   the variables which must be set:
 	LIBT2N_CODEGEN="\$(top_builddir)/codegen/libt2n-codegen"
 	LIBT2N_CLIENT_PCTEMPLATE="\$(top_srcdir)/codegen/clientlib.pc.in"
-	LIBT2N_CODEGEN_MAKESNIPPET="include \$(top_srcdir)/codegen/codegen.make"
-
-  we store the variables in the .pc file of libt2n
-  maybe wrap the calls to pkg-config in a m4 macro
-  (=> projects using libt2n-codegen need only one line in configure.in)
-
+  => we store the variables in the .pc file of libt2n
 - it would really be much nicer if the client lib includes would not depend upon boost serialization
   i thought a solution would be to provide this optionally by wrapping the corresponding includes in a #ifdef
   but this does not work since command.hxx is included which depends on boost serialization headers anyway
   => we do not provide this for now
+- naming scheme?!
diff --git a/example-codegen/README b/example-codegen/README
index 2fe7e02..ca3ab46 100644
--- a/example-codegen/README
+++ b/example-codegen/README
@@ -16,5 +16,3 @@ cd example-codegen
 # -M pathtolibt2n.m4
 autoreconf -f -i
 ./configure && make && make distcheck
-
-tar xzvf libt2n-0.1.tar.gz && mv libt2n-0.1/example-codegen . && cd example-codegen && autoreconf -f -i -M /tmp/jens-delme/usr/share/aclocal && ./configure && dmake && dmake distcheck
diff --git a/example-codegen/TODO b/example-codegen/TODO
index 39fc78d..73fe79b 100644
--- a/example-codegen/TODO
+++ b/example-codegen/TODO
@@ -14,9 +14,16 @@ now we use the first solution
   for codegen - but not overwriting output files if they are the same)
 - howto include the part after "# always the same:" in all makefiles (using the codegenerator)
   (we now use make's include)
-- lib name should be group name + suffix/prefix
+- lib name should be group name + suffix/prefix ?!
+  user should have complete control => no default suffix/perfix
+  headers should be installed in pkgincludedir? maybe yes on the other hand
+  the user can pass the directory to configure via --includedir
 - the example should not get installed (but still it should show how to build a lib that gets installed :-(
   (overwriting install: doesn't work)
+  => perhaps split libt2n package into 3 packages?
+	libt2n
+	libt2n-example-codegen
+	libt2n-example-libusage
 - at the moment make dist from within a clean source will not work
   (this is related to the nodist problem / old automake version)
   this is caused by BUILT_SOURCES not working for make dist(check?)
@@ -29,6 +36,3 @@ now we use the first solution
   => how to add dependency on BUILT_SOURCES for dist?
   the real problem is that make dist should not depend on BUILT_SOURCES
   unfortunately we can't fix this as long as we use a old automake version
-- add (standalone) example using generated client library
-  (=> depends on installed example)
-  example consists of: the same client.cpp but provides a configure.in and Makefile.am