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

Re: wmaker: restarts on window resize



Hi Marcelo,

I've prepared a 0-day NMU for this bug, based on Timo's patch.  The full
diff is attached.

I made an abortive attempt to use "variable" as the fallback instead of
"fixed", since that seemed more in line with the use of helvetica; but it
turns out that the "variable" font alias in xfonts-base *points* to
helvetica, which is not part of xfonts-base and therefore cannot be relied
on.

The NMU will be uploaded shortly.

-release: please approve wmaker_0.91.0-7.1 for sarge, fixing this RC bug.

Cheers,
-- 
Steve Langasek
postmodern programmer
reverted:
--- wmaker-0.91.0/debian/patches/40_switchpanel.diff
+++ wmaker-0.91.0.orig/debian/patches/40_switchpanel.diff
@@ -1,11 +0,0 @@
---- wmaker-0.91.0/src/switchpanel.c	2004-10-26 03:13:50.000000000 +0200
-+++ switchpanel.c	2005-01-07 19:13:47.000000000 +0100
-@@ -687,7 +687,7 @@
-         }
- 
-         if (focus < 0)
--          focus= 0;
-+          return NULL;
-         
-         if (focus >= 0 && panel->current != focus) {
-             WWindow *wwin;
diff -u wmaker-0.91.0/debian/patches/60_optional_switch_panel.diff wmaker-0.91.0/debian/patches/60_optional_switch_panel.diff
--- wmaker-0.91.0/debian/patches/60_optional_switch_panel.diff
+++ wmaker-0.91.0/debian/patches/60_optional_switch_panel.diff
@@ -215,7 +215,7 @@
          }
 -
 -        if (focus < 0)
--          return NULL;
+-          focus= 0;
 -        
 -        if (focus >= 0 && panel->current != focus) {
 -            WWindow *wwin;
diff -u wmaker-0.91.0/debian/changelog wmaker-0.91.0/debian/changelog
--- wmaker-0.91.0/debian/changelog
+++ wmaker-0.91.0/debian/changelog
@@ -1,3 +1,17 @@
+wmaker (0.91.0-7.1) unstable; urgency=high
+
+  * Non-maintainer upload.
+  * High-urgency upload for sarge-targetted RC bugfix.
+  * Make sure wmaker falls back to "fixed" in the absence of gsfonts-x11,
+    since X clients can't depend directly on font packages for their
+    operation.  Thanks to Timo Lindfors for finding this bug.
+    Closes: #279489.
+  * Remove debian/patches/40_switchpanel.diff, which conflicts with
+    debian/patches/60_switchpanel.diff; merge the patches together,
+    otherwise dpatch doesn't unpatch cleanly in the "clean" target.
+
+ -- Steve Langasek <vorlon@debian.org>  Wed, 11 May 2005 22:56:50 -0700
+
 wmaker (0.91.0-7) unstable; urgency=low
 
   * debian/rules: move the ugly hack earlier in the rules, I'm stepping
only in patch2:
unchanged:
--- wmaker-0.91.0.orig/src/screen.c
+++ wmaker-0.91.0/src/screen.c
@@ -808,6 +808,8 @@
     scr->info_text_font = WMBoldSystemFontOfSize(scr->wmscreen, 12);
 
     scr->tech_draw_font= XLoadQueryFont(dpy, "-adobe-helvetica-bold-r-*-*-12-*-*-*-*-*-*-*");
+    if (scr->tech_draw_font == NULL) /* fallback to our guaranteed font */
+        scr->tech_draw_font == XLoadQueryFont(dpy, "fixed");
 
     scr->gview = WCreateGeometryView(scr->wmscreen);
     WMRealizeWidget(scr->gview);

Attachment: signature.asc
Description: Digital signature


Reply to: