tags 493942 + patch
thanks

Hi there!

On Mon, 11 Aug 2008 14:55:02 +0200, Neil Williams wrote:
> On Mon, 2008-08-11 at 16:37 +0800, Wen-Yen Chuang wrote:
>> This is not an RC bug. I plan to fix it after release of Lenny.
>
> It does make the package unusable by all except a particular number of
> devices
[...]
> I'd support making this bug 'serious'.

If not 'serious' (to which I agree), it's at least 'important'.

>> However, if my sponsor want to see it fixed for Lenny,
>> I will be happy to do it.
>
> As the sponsor of previous uploads, I'd support fixing it for Lenny.

ACK.

>> I have not tested the patch on Sharp Zaurus yet.
>> My SLC-3200 is not on my hand currently.
>
> Please test on the Sharp Zaurus - I'm at DebConf currently (without my
> main upload key) but I can arrange for a sponsored upload either here
> or when I get home in a week.

I'd say that we should upload the fix ASAP, not only because I'm
interested in it for the Openmoko FreeRunner.  About the latter, the
patched tslib package [1] has been successfully used by different
people.

FWIW, if needed I can upload either an NMU (patch attached [2]) or I can
sponsor here at DebConf.

Thx, bye,
Gismo / Luca

Footnotes: 
[1] http://pkg-fso.alioth.debian.org/debian/pool/main/x/xf86-input-tslib/
[2] the patch is the less intrusive as possible, relying on quilt to
    manage the upstream changes

--8<---------------cut here---------------start------------->8---
diff -Nru xf86-input-tslib-0.0.4/debian/changelog 
xf86-input-tslib-0.0.4/debian/changelog
--- xf86-input-tslib-0.0.4/debian/changelog     2008-08-09 15:55:44.000000000 
+0200
+++ xf86-input-tslib-0.0.4/debian/changelog     2008-08-09 15:55:44.000000000 
+0200
@@ -1,3 +1,22 @@
+xf86-input-tslib (0.0.4-5.1) unstable; urgency=low
+
+  * NMU to fix important bug.
+
+  * debian/control:
+    + depend on quilt to manage Debian patches.
+
+  * debian/patches/series: new file.
+  * debian/patches/01_fix-wrong-value-range-for-the-axises.diff:
+    - use touchscreen width and height from the option instead of
+      hardcoding it in the sources, thanks to Sascha Hauer
+      <[EMAIL PROTECTED]> (Closes: #493942).
+
+  * debian/rules:
+    - adapt to quilt, using the debian/xsfbs/xsfbs.mk targets instead
+      of the usual quilt ones.
+
+ -- Luca Capello <[EMAIL PROTECTED]>  Sat, 09 Aug 2008 15:47:12 +0200
+
 xf86-input-tslib (0.0.4-5) unstable; urgency=low
 
   * Add Replaces: xf86-input-tslib to unbreak upgrades
diff -Nru xf86-input-tslib-0.0.4/debian/control 
xf86-input-tslib-0.0.4/debian/control
--- xf86-input-tslib-0.0.4/debian/control       2008-08-09 15:55:44.000000000 
+0200
+++ xf86-input-tslib-0.0.4/debian/control       2008-08-09 15:55:44.000000000 
+0200
@@ -5,7 +5,8 @@
 Build-Depends: debhelper (>= 5),
                xserver-xorg-dev (>= 2:1.4),
                pkg-config,
-               libts-dev (>= 1.0-4)
+               libts-dev (>= 1.0-4),
+               quilt (>= 0.40)
 Standards-Version: 3.7.3
 
 Package: xserver-xorg-input-tslib
diff -Nru 
xf86-input-tslib-0.0.4/debian/patches/01_fix-wrong-value-range-for-the-axises.diff
 
xf86-input-tslib-0.0.4/debian/patches/01_fix-wrong-value-range-for-the-axises.diff
--- 
xf86-input-tslib-0.0.4/debian/patches/01_fix-wrong-value-range-for-the-axises.diff
  1970-01-01 01:00:00.000000000 +0100
+++ 
xf86-input-tslib-0.0.4/debian/patches/01_fix-wrong-value-range-for-the-axises.diff
  2008-08-09 15:55:44.000000000 +0200
@@ -0,0 +1,42 @@
+Index: src/tslib.c
+===================================================================
+--- a/src/tslib.c      (revision 28)
++++ b/src/tslib.c      (working copy)
+@@ -184,9 +184,11 @@
+       InputInfoPtr pInfo;
+       unsigned char map[MAXBUTTONS + 1];
+       int i;
++      struct ts_priv *priv;
+ 
+       ErrorF("%s\n", __FUNCTION__);
+       pInfo = device->public.devicePrivate;
++      priv = pInfo->private;
+ 
+       switch (what) {
+       case DEVICE_INIT:
+@@ -222,15 +224,17 @@
+                       return !Success;
+               }
+ 
+-              InitValuatorAxisStruct(device, 0, 0, /* min val */ 1023,        
/* max val */
+-                                             1024,    /* resolution */
+-                                             0,       /* min_res */
+-                                             1024);   /* max_res */
++              InitValuatorAxisStruct(device, 0, 0,            /* min val */
++                                             priv->width - 1, /* max val */
++                                             priv->width,     /* resolution */
++                                             0,               /* min_res */
++                                             priv->width);    /* max_res */
+ 
+-              InitValuatorAxisStruct(device, 1, 0, /* min val */ 1023,        
/* max val */
+-                                             1024,    /* resolution */
+-                                             0,       /* min_res */
+-                                             1024);   /* max_res */
++              InitValuatorAxisStruct(device, 1, 0,            /* min val */
++                                             priv->height - 1,/* max val */
++                                             priv->height,    /* resolution */
++                                             0,               /* min_res */
++                                             priv->height);   /* max_res */
+ 
+               if (InitProximityClassDeviceStruct (device) == FALSE) {
+                       ErrorF ("Unable to allocate EVTouch touchscreen 
ProximityClassDeviceStruct\n");
diff -Nru xf86-input-tslib-0.0.4/debian/patches/series 
xf86-input-tslib-0.0.4/debian/patches/series
--- xf86-input-tslib-0.0.4/debian/patches/series        1970-01-01 
01:00:00.000000000 +0100
+++ xf86-input-tslib-0.0.4/debian/patches/series        2008-08-09 
15:55:44.000000000 +0200
@@ -0,0 +1 @@
+01_fix-wrong-value-range-for-the-axises.diff
diff -Nru xf86-input-tslib-0.0.4/debian/rules 
xf86-input-tslib-0.0.4/debian/rules
--- xf86-input-tslib-0.0.4/debian/rules 2008-08-09 15:55:44.000000000 +0200
+++ xf86-input-tslib-0.0.4/debian/rules 2008-08-09 15:55:44.000000000 +0200
@@ -3,6 +3,7 @@
 # Uncomment this to turn on verbose mode.
 #export DH_VERBOSE=1
 
+
 include debian/xsfbs/xsfbs.mk
 
 DEB_HOST_GNU_TYPE   ?= $(shell dpkg-architecture -qDEB_HOST_GNU_TYPE)
@@ -20,7 +21,7 @@
 
 build: build-stamp
 
-build-stamp:  config.status
+build-stamp: patch config.status
        dh_testdir
        $(MAKE)
        touch $@
--8<---------------cut here---------------end--------------->8---

Attachment: pgpGgvTe5s6WM.pgp
Description: PGP signature

Reply via email to