RE: Some bugs found when compiling for maemo

[<Dirk-Jan Binnema nokia com>]

Index: libtinymail-camel/camel-lite/libedataserver/e-iconv.c
===================================================================
--- libtinymail-camel/camel-lite/libedataserver/e-iconv.c   (revision 1573)
+++ libtinymail-camel/camel-lite/libedataserver/e-iconv.c   (working copy)
@@ -26,6 +26,10 @@
 #include <config.h>
 #endif

+#ifndef ICONV_10646
+#define ICONV_10646 "iso-10646"
+#endif /*ICONV_10646*/
+
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>


Works for me at least. The problem is that iconv-detect does _not_
detect the ICON_10646 on maemo, but the camel code assumes that
it does.

Best wishes,
Dirk.



>-----Original Message-----
>From: tinymail-devel-list-bounces gnome org 
>[mailto:tinymail-devel-list-bounces gnome org] On Behalf Of 
>ext Sergio Villar Senin
>Sent: Monday, February 12, 2007 10:03
>To: tinymail-devel-list gnome org
>Subject: Re: Some bugs found when compiling for maemo
>
>Philip Van Hoof wrote:
>> On Sat, 2007-02-10 at 18:13 +0200, Tomi Pieviläinen wrote:
>>> Hi,
>>>
>>> Found the following small bugs when compiling tinymail for maemo:
>>>
>>> Index: libtinymail-camel/camel-lite/libedataserver/e-iconv.c
>>> ===================================================================
>>> --- libtinymail-camel/camel-lite/libedataserver/e-iconv.c   
>(revision 1558)
>>> +++ libtinymail-camel/camel-lite/libedataserver/e-iconv.c   
>(working copy)
>>> @@ -381,7 +381,7 @@
>>>         
>>>         if (iso == 10646) {
>>>             /* they all become ICONV_10646 */
>>> -           ret = g_strdup (ICONV_10646);
>>> +           ret = g_strdup ("ICONV_10646");
>> 
>> 
>> Euh, no. That define should have been set in iconv-detect.h which is 
>> generated during the ./configure
>
>You're right, it should be but it isn't :-). I mean, I think 
>there is something wrong in the configuration phase because I 
>never got that definition in the header file.
>
>Br
>_______________________________________________
>tinymail-devel-list mailing list
>tinymail-devel-list gnome org
>http://mail.gnome.org/mailman/listinfo/tinymail-devel-list
>