Your message dated Sun, 03 Sep 2006 18:47:45 -0700
with message-id <[EMAIL PROTECTED]>
and subject line Bug#355458: fixed in lcdproc 0.5.0-1
has caused the attached Bug report to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere.  Please contact me immediately.)

Debian bug tracking system administrator
(administrator, Debian Bugs database)

--- Begin Message ---
Package: lcdproc
Version: 0.4.5-1
Severity: normal

LCDd can handle widgets on a frame. So one can create a frame 
that is bigger than the actual size of the LCD. LCDd cares about
scrolling the frame withing the display portion. To add a widget
to a frame you have to send "widget_add ... in <frameid>" to the 
server.

The problem is that on the server side in client_functions.c on
line 827 there is a test for "in" - but in the wrong way.

   if (strcmp(p, "in")) {
   
should read
   
   if (0==strcmp(p, "in")) {
   
This means the "in" option is not recognized. In lcdproc
(see mem.c and disk.c) they tried a workaround by using "-in"
instead "in" but this also does not work becaus the "-" char 
is stripped off using a "if (*p=='-') p++;" a few lines above.

diff file would go like:

--- lcdproc-0.4.5/server/client_functions.c_ORG   2004-05-19
13:27:58.000000000 +0200
+++ lcdproc-0.4.5-1/server/client_functions.c    2006-03-04
18:16:54.000000000 +0100
@@ -824,7 +824,7 @@
                        p++;
                        
                                        /* Handle the "in" flag to place 
widgets in a
container...*/
-               if (strcmp (p, "in")) {
+               if (0==strcmp (p, "in")) {
                        if (argc < 6) {
                                sock_send_string (c->sock, "huh? Specify a
frame to place widget in\n");
                                return 0;
                                                                                
                                                
Please note that if LCDd gets fixed do not forget to 
change "-in" to "in" in 
/clients/lcdproc/disk.c and 
clients/lcdproc/mem.c.
also.

Greetings
Andreas

-- System Information:
Debian Release: testing/unstable
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/bash
Kernel: Linux 2.6.15
Locale: LANG=en_US, LC_CTYPE=en_US (charmap=ISO-8859-1)

Versions of packages lcdproc depends on:
ii  debconf     1.4.70     Debian configuration management sy
ii  libc6       2.3.5-13   GNU C Library: Shared libraries an
ii  libncurses5 5.5-1      Shared libraries for terminal hand
ii  libsvga1    1:1.4.3-22 console SVGA display libraries

lcdproc recommends no packages.

-- no debconf information

-- 
Echte DSL-Flatrate dauerhaft für 0,- Euro*!
"Feel free" mit GMX DSL! http://www.gmx.net/de/go/dsl


--- End Message ---
--- Begin Message ---
Source: lcdproc
Source-Version: 0.5.0-1

We believe that the bug you reported is fixed in the latest version of
lcdproc, which is due to be installed in the Debian FTP archive:

lcdproc_0.5.0-1.diff.gz
  to pool/main/l/lcdproc/lcdproc_0.5.0-1.diff.gz
lcdproc_0.5.0-1.dsc
  to pool/main/l/lcdproc/lcdproc_0.5.0-1.dsc
lcdproc_0.5.0-1_amd64.deb
  to pool/main/l/lcdproc/lcdproc_0.5.0-1_amd64.deb
lcdproc_0.5.0-1_i386.deb
  to pool/main/l/lcdproc/lcdproc_0.5.0-1_i386.deb
lcdproc_0.5.0-1_sparc.deb
  to pool/main/l/lcdproc/lcdproc_0.5.0-1_sparc.deb
lcdproc_0.5.0.orig.tar.gz
  to pool/main/l/lcdproc/lcdproc_0.5.0.orig.tar.gz



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [EMAIL PROTECTED],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Jose Luis Tallon <[EMAIL PROTECTED]> (supplier of updated lcdproc package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [EMAIL PROTECTED])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Sun, 23 Jul 2006 20:23:48 +0200
Source: lcdproc
Binary: lcdproc
Architecture: source i386 sparc amd64
Version: 0.5.0-1
Distribution: unstable
Urgency: low
Maintainer: Jose Luis Tallon <[EMAIL PROTECTED]>
Changed-By: Jose Luis Tallon <[EMAIL PROTECTED]>
Description: 
 lcdproc    - LCD display driver daemon and clients
Closes: 331885 355458 355460 365436 365496 367945 376449
Changes: 
 lcdproc (0.5.0-1) unstable; urgency=low
 .
   * New upstream version (Closes: #367945)
     - New maintainer. Thank you for your previous work, Jon!
     - Upstream added suport for 'imon' devices (Closes: #365496)
     - Upstream fixed descriptor leak (Closes: #355460)
     - Upstream fixed placing widgets in frame (Closes: #355458)
 .
   * Packaging
     - Depend on debconf-2.0; Allow transition (Closes: #331885)
     - Remove dependency on automake (Closes: #376449)
     - Include missing INSTALL instructions (Closes: #365436)
     - Changed most "by hand" installation steps into debhelper-based ones
     - Updated to 3.7.2 standards version
Files: 
 25e2543034b524b7409661958a5b7b87 659 utils extra lcdproc_0.5.0-1.dsc
 4b67e421c19063fa322611a849ab5b80 711118 utils extra lcdproc_0.5.0.orig.tar.gz
 4acfd3f0660d83de024cc5f1cb78dd7f 12554 utils extra lcdproc_0.5.0-1.diff.gz
 b666166dfd6cf5fda800d1f314f94079 272230 utils extra lcdproc_0.5.0-1_i386.deb
 c760d2cda4653de48465ca12edf42829 305142 utils extra lcdproc_0.5.0-1_amd64.deb
 94526884ab75bf786994394475874112 240790 utils extra lcdproc_0.5.0-1_sparc.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.3 (GNU/Linux)

iD8DBQFE+4KkipBneRiAKDwRAnv/AJ9wGUyhRSlrm98uibadmkSpyIl4KQCeJIR8
QunIY557muF+aE6VdDzJgok=
=hNWe
-----END PGP SIGNATURE-----


--- End Message ---

Reply via email to