[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

X Strike Force XFree86 SVN commit: r1987 - in branches/ubuntu/debian: . patches



Author: fabbione
Date: 2004-10-30 00:33:16 -0500 (Sat, 30 Oct 2004)
New Revision: 1987

Modified:
   branches/ubuntu/debian/CHANGESETS
   branches/ubuntu/debian/TODO
   branches/ubuntu/debian/changelog
   branches/ubuntu/debian/patches/000_stolen_from_HEAD.diff
   branches/ubuntu/debian/patches/000_stolen_from_HEAD_xkb_data.diff
   branches/ubuntu/debian/patches/087_xkb_fix_altwin.diff
   branches/ubuntu/debian/xserver-xfree86.config.in
   branches/ubuntu/debian/xserver-xfree86.postinst.in
Log:
Import 4.3.0.dfsg.1-6ubuntu11 release.


Modified: branches/ubuntu/debian/CHANGESETS
===================================================================
--- branches/ubuntu/debian/CHANGESETS	2004-10-30 05:31:48 UTC (rev 1986)
+++ branches/ubuntu/debian/CHANGESETS	2004-10-30 05:33:16 UTC (rev 1987)
@@ -14,7 +14,7 @@
 
 Miscellaneous cosmetic fixes.
     1607, 1608, 1624, 1626, 1627, 1636, 1651, 1654, 1663, 1664, 1675, 1685,
-    1690, 1713, 1715, 1716, 1718, 1736, 1737, 1743, 1761
+    1690, 1713, 1715, 1716, 1718, 1736, 1737, 1743, 1761, 1768, 1774, 1776
 
 Grab latest version of XTerm (#191) from Thomas Dickey's website.
     1609
@@ -481,16 +481,29 @@
   the Debian Policy Manual).
     1753, 1757, 1760, 1763
 
-* Add french and danish translation of "Generic Monitor" to
-  xserver-xfree86.config.in. Thanks to Christian Perrier and Anders Dybdal.
+* Add French and Danish translation of "Generic Monitor" to
+  xserver-xfree86.config.in.  Thanks to Christian Perrier and Anders Dybdal.
     1762
 
-* Add danish translation of "Generic Video Card" to 
-  xserver-xfree86.config.in. Thanks to Anders Dybdal.
+* Add Danish translation of "Generic Video Card" to
+  xserver-xfree86.config.in.  Thanks to Anders Dybdal.
     1764
 
-* Fix validate_monitor_frequency_db_input() regexp to match for proper
-  values.
-    1766      
+Make xserver-xfree86 config script's validate_monitor_frequency_db_input()
+function more strict, so it doesn't regard open-ended ranges like "40-" as
+valid.
+    1766, 1769, 1772
 
+Grab from XFree86 CVS (2003-12-18) several fixes for keyboard issues:
++ Fix for XKB map 'altwin' to avoid one keysym to more than one modifier
+  mapping (Ivan Pascal).  This really fixes the Alt-Tab issue.
+  (Closes: #256706)
++ Fix xmodmap's output of the modifiers map to not display BadKey when
+  the first column keysym is empty (Ivan Pascal).
++ The "modifier_add none" construct was introduced by Ivan Pascal some
+  times ago, but it was not documented in the changelog and a piece was
+  missing to enable this feature.  No keymaps currently use this feature,
+  but users may find it useful with broken keymaps.
+    1773
+
 vim:set ai et sts=4 sw=4 tw=80:

Modified: branches/ubuntu/debian/TODO
===================================================================
--- branches/ubuntu/debian/TODO	2004-10-30 05:31:48 UTC (rev 1986)
+++ branches/ubuntu/debian/TODO	2004-10-30 05:33:16 UTC (rev 1987)
@@ -47,22 +47,10 @@
     done about this [BR]
   + #257515: xserver-xfree86: [debconf] keyboard layout template should tell U.K.
      users to use 'gb' [BR]
+  + Use /proc/hardware on m68k architecture to set a reasonable default mouse
+    port.  See <URL: http://lists.debian.org/debian-68k/2004/08/msg00392.html>.
 * Add FAQ entry describing Debian's plans in the X department.
-* #256706: keep swatting away at the keyboard modifer issue; Ivan Pascal told
-  BR:
-    There is not modifier_del instruction in 'XKB language' but ...
-    I made a simple change for xkbcomp and now it understands an instruction
-    like
 
-       modifier_add none { <LALT> };
-    It removes the modifier binding from the specified key.
-    I'm sorry I forget to mention this patch in CHANGELOG but it was a part
-    of 667 change made at Dec 18 14:14:37 2003 (i.e. it is included in 4.4.0).
-    If your xkbcomp is later than that version it understands such instruction.
-  So I'm thinking the fix is to apply the above, then modify the symbol maps
-  such that (in most cases) any keys that get modifier mappings are cleared
-  first.  This solution will absolutely positively need testing.
-
 4.3.0.dfsg.1-8
 --------------
 

Modified: branches/ubuntu/debian/changelog
===================================================================
--- branches/ubuntu/debian/changelog	2004-10-30 05:31:48 UTC (rev 1986)
+++ branches/ubuntu/debian/changelog	2004-10-30 05:33:16 UTC (rev 1987)
@@ -1,3 +1,10 @@
+xfree86 (4.3.0.dfsg.1-6ubuntu11) warty; urgency=low
+
+  * Resync changes with Debian r1777.
+  * Kill debian/patches/914_novtswitch_option.diff.
+
+ -- Fabio M. Di Nitto <fabbione@fabbione.net>  Tue, 31 Aug 2004 06:42:51 +0200
+
 xfree86 (4.3.0.dfsg.1-6ubuntu10) warty; urgency=low
 
   * Resync changes with Debian r1766.
@@ -558,6 +565,10 @@
       redirect them, which is precisely what we don't want (see section 6.5 of
       the Debian Policy Manual).
 
+  * Make xserver-xfree86 config script's validate_monitor_frequency_db_input()
+    function more strict, so it doesn't regard open-ended ranges like "40-" as
+    valid.
+
   Changes by Robert Millan:
 
   * Get glxinfo to link properly with g++ (instead of adding -lstdc++).
@@ -571,17 +582,27 @@
     autodetection would cause the script to think it had detected one X server
     or XFree86 video driver instead ($(echo "" | wc -l) is "1", not "0").
 
-  * Add french and danish translation of "Generic Monitor" to
-    xserver-xfree86.config.in. Thanks to Christian Perrier and Anders Dybdal.
+  * Add French and Danish translation of "Generic Monitor" to
+    xserver-xfree86.config.in.  Thanks to Christian Perrier and Anders Dybdal.
 
-  * Add danish translation of "Generic Video Card" to
-    xserver-xfree86.config.in. Thanks to Anders Dybdal.
+  * Add Danish translation of "Generic Video Card" to
+    xserver-xfree86.config.in.  Thanks to Anders Dybdal.
 
-  * Fix validate_monitor_frequency_db_input() regexp to match for proper
-    values.
+  Changes by Denis Barbier:
 
- -- Branden Robinson <branden@debian.org>  Wed, 25 Aug 2004 11:41:15 -0500
+  * Grab from XFree86 CVS (no later than 2003-12-18) several fixes to XKB:
+    + Fix for XKB map 'altwin' to avoid one keysym to more than one modifier
+      mapping (Ivan Pascal).  This really fixes the Alt-Tab issue.
+      (Closes: #256706)
+    + Fix xmodmap's output of the modifiers map to not display BadKey when
+      the first column keysym is empty (Ivan Pascal).
+    + The "modifier_add none" construct was introduced by Ivan Pascal some
+      times ago, but it was not documented in the changelog and a piece was
+      missing to enable this feature.  No keymaps currently use this feature,
+      but users may find it useful with broken keymaps.
 
+ -- Branden Robinson <branden@debian.org>  Mon, 30 Aug 2004 14:22:49 -0500
+
 xfree86 (4.3.0.dfsg.1-6) unstable; urgency=low
 
   Changes by Branden Robinson:
@@ -769,6 +790,7 @@
     mysterious failures; see #251690 and #252348.  Thanks also to the libcurl2
     maintainer for the reminder that "The issue isn't whether you're
     paranoid...The issue is whether you're paranoid *enough*." [Max Peltier])
+    (Closes: #149586)
 
   * Expand patch #104 to prevent 64-bit addressing modes from being used when
     they should not be on UltraSPARC Linux.

Modified: branches/ubuntu/debian/patches/000_stolen_from_HEAD.diff
===================================================================
--- branches/ubuntu/debian/patches/000_stolen_from_HEAD.diff	2004-10-30 05:31:48 UTC (rev 1986)
+++ branches/ubuntu/debian/patches/000_stolen_from_HEAD.diff	2004-10-30 05:33:16 UTC (rev 1987)
@@ -608,6 +608,18 @@
   Fix manual pages typos (Bugzilla #316, Jens Schweikhardt).
 [Matthieu Herrb]
 
+xc/programs/xmodmap/exec.c @ 1.6
+  634. Fix xmodmap's output of the modifiers map when the first column keysym
+       is empty (Ivan Pascal).
+
+xc/programs/xkbcomp/expr.c @ 3.7
+  351. Fixed string octal number parsing and string to int conversion for "\00"
+       in xkbcomp (BugzillaR #553, Egbert Eich).
+xc/programs/xkbcomp/expr.c @ 3.8
+  Add "modifier_map none" syntax to remove modifier bindings from the
+  specified key.  [Ivan Pascal]
+
+
 diff -urN xc.orig/config/imake/imake.c xc/config/imake/imake.c
 --- xc.orig/config/imake/imake.c	2002-12-17 09:48:27.000000000 +1100
 +++ xc/config/imake/imake.c	2003-04-09 01:58:14.000000000 +1000
@@ -23491,3 +23503,97 @@
  and serving as an example for XTrap functionality, it can reasonably used as
  a primitive "playback" client for X sessions.
  .LP
+Index: xc/programs/xmodmap/exec.c
+===================================================================
+RCS file: /cvs/xc/programs/xmodmap/exec.c,v
+retrieving revision 1.5
+retrieving revision 1.6
+diff -u -r1.5 -r1.6
+--- xc/programs/xmodmap/exec.c	14 Dec 2001 20:02:13 -0000	1.5
++++ xc/programs/xmodmap/exec.c	2 Dec 2003 13:13:57 -0000	1.6
+@@ -56,7 +56,7 @@
+  * Author:  Jim Fulton, MIT X Consortium; derived from parts of the
+  * original xmodmap, written by David Rosenthal, of Sun Microsystems.
+  */
+-/* $XFree86: xc/programs/xmodmap/exec.c,v 1.5 2001/12/14 20:02:13 dawes Exp $ */
++/* $XFree86: xc/programs/xmodmap/exec.c,v 1.6 2003/12/02 13:13:57 pascal Exp $ */
+ 
+ #include <X11/Xos.h>
+ #include <X11/Xlib.h>
+@@ -212,8 +212,13 @@
+ PrintModifierMapping(XModifierKeymap *map, FILE *fp)
+ {
+     int i, k = 0;
++    int min_keycode, max_keycode, keysyms_per_keycode = 0;
+ 
+-    fprintf (fp, 
++    XDisplayKeycodes (dpy, &min_keycode, &max_keycode);
++    XGetKeyboardMapping (dpy, min_keycode, (max_keycode - min_keycode + 1),
++			 &keysyms_per_keycode);
++
++    fprintf (fp,
+     	     "%s:  up to %d keys per modifier, (keycodes in parentheses):\n\n", 
+     	     ProgramName, map->max_keypermod);
+     for (i = 0; i < 8; i++) {
+@@ -222,8 +227,14 @@
+ 	fprintf(fp, "%-10s", modifier_table[i].name);
+ 	for (j = 0; j < map->max_keypermod; j++) {
+ 	    if (map->modifiermap[k]) {
+-		KeySym ks = XKeycodeToKeysym(dpy, map->modifiermap[k], 0);
+-		char *nm = XKeysymToString(ks);
++		KeySym ks;
++		int index = 0;
++		char *nm;
++		do {
++		    ks = XKeycodeToKeysym(dpy, map->modifiermap[k], index);
++		    index++;
++		} while ( !ks && index < keysyms_per_keycode);
++		nm = XKeysymToString(ks);
+ 
+ 		fprintf (fp, "%s  %s (0x%0x)", (j > 0 ? "," : ""), 
+ 			 (nm ? nm : "BadKey"), map->modifiermap[k]);
+Index: xc/programs/xkbcomp/expr.c
+===================================================================
+RCS file: /cvs/xc/programs/xkbcomp/expr.c,v
+retrieving revision 3.6
+retrieving revision 3.8
+diff -u -r3.6 -r3.8
+--- xc/programs/xkbcomp/expr.c	5 Jun 2002 00:00:37 -0000	3.6
++++ xc/programs/xkbcomp/expr.c	18 Dec 2003 14:14:37 -0000	3.8
+@@ -24,7 +24,7 @@
+  THE USE OR PERFORMANCE OF THIS SOFTWARE.
+ 
+  ********************************************************/
+-/* $XFree86: xc/programs/xkbcomp/expr.c,v 3.6 2002/06/05 00:00:37 dawes Exp $ */
++/* $XFree86: xc/programs/xkbcomp/expr.c,v 3.8 2003/12/18 14:14:37 pascal Exp $ */
+ 
+ #include "xkbcomp.h"
+ #include "tokens.h"
+@@ -245,6 +245,7 @@
+ 	{	"mod3",		Mod3MapIndex	},
+ 	{	"mod4",		Mod4MapIndex	},
+ 	{	"mod5",		Mod5MapIndex	},
++	{	"none",		XkbNoModifier	},
+ 	{	NULL,		0		}
+ };
+ 
+@@ -569,9 +570,16 @@
+ 	    if (expr->type==TypeString) {
+ 		register char *str;
+ 		str= XkbAtomGetString(NULL,expr->value.str);
+-		if ((str!=None)&&(strlen(str)==1)) {
+-		    val_rtrn->uval= str[0];
+-		    return True;
++		if (str!=None)
++		    switch (strlen(str)) {
++			case 0:
++			    val_rtrn->uval= 0;
++			    return True;
++			case 1:
++			    val_rtrn->uval= str[0];
++			    return True;
++			default:
++			    break;
+ 		}
+ 	    }
+ 	    if ((expr->type!=TypeInt)&&(expr->type!=TypeFloat)) {

Modified: branches/ubuntu/debian/patches/000_stolen_from_HEAD_xkb_data.diff
===================================================================
--- branches/ubuntu/debian/patches/000_stolen_from_HEAD_xkb_data.diff	2004-10-30 05:31:48 UTC (rev 1986)
+++ branches/ubuntu/debian/patches/000_stolen_from_HEAD_xkb_data.diff	2004-10-30 05:33:16 UTC (rev 1987)
@@ -178,6 +178,10 @@
         [...]
   [Ivan Pascal]
 
+xc/programs/xkbcomp/symbols/altwin @ 1.5
+   646. Fix for XKB map 'altwin' to avoid one keysym to more than one modifier
+        mapping (Ivan Pascal).
+
 xc/programs/xkbcomp/symbols/pc/us @ 1.7
    667. Fixes and updates for XKB keyboard maps:
         - Add to 'us' keymap backslash key that can be rewritten in other layouts
@@ -2530,6 +2534,38 @@
 -
 +    include "level3(ralt_switch_multikey)"
  };
+Index: xc/programs/xkbcomp/symbols/altwin
+===================================================================
+RCS file: /cvs/xc/programs/xkbcomp/symbols/altwin,v
+retrieving revision 1.4
+retrieving revision 1.5
+diff -u -r1.4 -r1.5
+--- xc/programs/xkbcomp/symbols/altwin	10 Oct 2001 19:18:32 -0000	1.4
++++ xc/programs/xkbcomp/symbols/altwin	3 Dec 2003 14:09:08 -0000	1.5
+@@ -1,4 +1,4 @@
+-// $XFree86: xc/programs/xkbcomp/symbols/altwin,v 1.4 2001/10/10 19:18:32 herrb Exp $
++// $XFree86: xc/programs/xkbcomp/symbols/altwin,v 1.5 2003/12/03 14:09:08 pascal Exp $
+ 
+ partial modifier_keys 
+ xkb_symbols "meta_alt" {
+@@ -15,7 +15,7 @@
+     key <LWIN> {	[ 	Meta_L			]	};
+     key <RWIN> {	[ 	Meta_R			]	};
+     modifier_map Mod1	{ Alt_L, Alt_R };
+-    modifier_map Mod4	{ Meta_L, Meta_R };
++    modifier_map Mod4	{ <META>, Meta_L, Meta_R };
+ };
+ 
+ partial modifier_keys 
+@@ -23,7 +23,7 @@
+     key <LALT> {	[ 	Alt_L,	Alt_L		]	};
+     key <LWIN> {	[ 	Meta_L			]	};
+     modifier_map Mod1	{ Alt_L };
+-    modifier_map Mod4	{ Meta_L };
++    modifier_map Mod4	{ <META>, Meta_L };
+ };
+ 
+ partial modifier_keys 
 Index: xc/programs/xkbcomp/symbols/pc/am
 ===================================================================
 RCS file: /cvs/xc/programs/xkbcomp/symbols/pc/am,v

Modified: branches/ubuntu/debian/patches/087_xkb_fix_altwin.diff
===================================================================
--- branches/ubuntu/debian/patches/087_xkb_fix_altwin.diff	2004-10-30 05:31:48 UTC (rev 1986)
+++ branches/ubuntu/debian/patches/087_xkb_fix_altwin.diff	2004-10-30 05:33:16 UTC (rev 1987)
@@ -20,7 +20,7 @@
      key <LWIN> {	[ 	Meta_L			]	};
      key <RWIN> {	[ 	Meta_R			]	};
 -    modifier_map Mod1	{ Alt_L, Alt_R };
-     modifier_map Mod4	{ Meta_L, Meta_R };
+     modifier_map Mod4	{ <META>, Meta_L, Meta_R };
  };
  
  partial modifier_keys 
@@ -28,6 +28,6 @@
 -    key <LALT> {	[ 	Alt_L,	Alt_L		]	};
      key <LWIN> {	[ 	Meta_L			]	};
 -    modifier_map Mod1	{ Alt_L };
-     modifier_map Mod4	{ Meta_L };
+     modifier_map Mod4	{ <META>, Meta_L };
  };
  

Modified: branches/ubuntu/debian/xserver-xfree86.config.in
===================================================================
--- branches/ubuntu/debian/xserver-xfree86.config.in	2004-10-30 05:31:48 UTC (rev 1986)
+++ branches/ubuntu/debian/xserver-xfree86.config.in	2004-10-30 05:33:16 UTC (rev 1987)
@@ -274,10 +274,12 @@
     fi
     db_go
     db_get "$TEMPLATE"
-    # string, needs input validation; a simple regex match will have to do;
-    # force first character to be a number to avoid hideous problems in the
-    # debconf dialog frontend in 0.3.83 (it needs to be one anyway)
-    if [ "$(echo "$RET" | grep -E '^[0-9]{2,3}-[0-9]{2,3}')" ]; then
+    # This is a string, and needs input validation; a regex match will have to
+    # do.  We force the first character to be a number to avoid hideous problems
+    # in the debconf dialog frontend in 0.3.83 (it needs to be one anyway).  We
+    # don't allow values less than 10.
+    if echo "$RET" | grep -qE \
+      '^[0-9]{2,3}(\.[0-9]+)?(-[0-9]{2,3}(\.[0-9]+)?)?$'; then
       break # valid input
     fi
     # we only get to this point if the input was invalid; restore the known

Modified: branches/ubuntu/debian/xserver-xfree86.postinst.in
===================================================================
--- branches/ubuntu/debian/xserver-xfree86.postinst.in	2004-10-30 05:31:48 UTC (rev 1986)
+++ branches/ubuntu/debian/xserver-xfree86.postinst.in	2004-10-30 05:33:16 UTC (rev 1987)
@@ -88,10 +88,12 @@
     fi
     db_go
     db_get "$TEMPLATE"
-    # string, needs input validation; a simple regex match will have to do;
-    # force first character to be a number to avoid hideous problems in the
-    # debconf dialog frontend in 0.3.83 (it needs to be one anyway)
-    if [ "$(echo "$RET" | grep -E '^[0-9]{2,3}-[0-9]{2,3}')" ]; then
+    # This is a string, and needs input validation; a regex match will have to
+    # do.  We force the first character to be a number to avoid hideous problems
+    # in the debconf dialog frontend in 0.3.83 (it needs to be one anyway).  We
+    # don't allow values less than 10.
+    if echo "$RET" | grep -qE \
+      '^[0-9]{2,3}(\.[0-9]+)?(-[0-9]{2,3}(\.[0-9]+)?)?$'; then
       break # valid input
     fi
     # we only get to this point if the input was invalid; restore the known



Reply to: