Bugzilla Bug 94465

Hi

RRDtool 1.2.9 has been released, with a few bugfixes ("make tcl bindings compile
again") and some new features ("consider the users locale settings when
rendering text").

The full announcement is here : http://lists.ee.ethz.ch/rrd-announce/msg00081.html

It's building & running fine by copying the 1.2.8 ebuild.

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

in cvs

Well, I should have tested it more before submiting the ebuild :-(

It seems that the new locales management breaks something :
http://forums.cacti.net/viewtopic.php?t=7824

(In reply to comment #2)
> Well, I should have tested it more before submiting the ebuild :-(
> It seems that the new locales management breaks something :
> http://forums.cacti.net/viewtopic.php?t=7824

This is confirmed on my gentoo system: http://david.port80.se/fault/
I've posted this on the rrd-users mailing-list but I haven't been able to 
resolve this.

/David 

Hi

Tobi sent me a patch yesterday, and it seems to be included in the latest
RRDtool snapshot
(http://people.ee.ethz.ch/~oetiker/webtools/rrdtool/pub/beta/rrdtool-svn-snap.tar.gz)
 :
r638 | oetiker | 2005-06-13 15:20:45 +0200 | 2 lines

wide chars should better be unsigned

Here's the diff :

--- rrd_gfx.c.old       2005-06-13 15:03:04.000000000 +0200
+++ rrd_gfx.c   2005-06-14 09:51:02.000000000 +0200
@@ -384,7 +384,11 @@
   wchar_t      *cstr;
   size_t       clen = strlen(text)+1;
   cstr = malloc(sizeof(wchar_t) * clen); /* yes we are allocating probably too
much here, I know */
-  string->count=mbstowcs(cstr,text,clen);
+  /* conversion did not work, so lets fall back to just use what we got */
+  string->count=clen-1;
+  for(n=0;text[n] != '\0';n++){
+       cstr[n]=(unsigned char)text[n];
+  }
   if ( string->count == -1){
        string->count=mbstowcs(cstr,"Enc-Err",6);
   }

Let's wait for 1.2.10 ;-)