Saturday, March 1, 2008

xpaint snapshot

Wow - xpaint has a snapshot feature now. Awesome.

Here's my original contribution to this cute little app:

= = = =
jspath at mail.bcpl.lib.md.NOT (Webmaster Jim) suggested this:

I don't have the "Fixed 12" font in my XFree86 system, so I've patched
the earlier version to include a font that I like, Sony 16 (and Sony 24).
I don't recommend this for everyone, but including the following in the
distribution can help others get different startup fonts:

--- operation.c.df Fri Apr 19 04:53:30 1996
+++ operation.c Sun Jun 16 12:10:16 1996
@@ -830,12 +830,14 @@
fontSet, "-*-helvetica-medium-r-normal-*-*-120-*-*-p-*-*-*"),
MI_FLAGCB("Helvetica Bold 12", MF_CHECK | MF_GROUP1,
fontSet, "-*-helvetica-bold-r-normal-*-*-120-*-*-p-*-*-*"),
- MI_FLAGCB("Fixed 12", MF_CHECK | MF_GROUP1,
- fontSet, "-*-fixed-medium-r-normal-*-*-120-*-*-m-*-*-*"),
+ MI_FLAGCB("Sony 16", MF_CHECK|MF_GROUP1,
+ fontSet, "-*-fixed-medium-r-normal--16-120-100-100-c-80-iso8859-1"),
+ MI_FLAGCB("Sony 24", MF_CHECK|MF_GROUP1,
+ fontSet, "-*-fixed-medium-r-normal--24-170-100-100-c-120-iso8859-1"),
MI_FLAGCB("Courier 12", MF_CHECK | MF_GROUP1,
fontSet, "-*-courier-medium-r-normal-*-*-120-*-*-m-*-*-*"),
MI_SEPERATOR(),
-#define FM_SELECT 11
+#define FM_SELECT 12
MI_FLAGCB("select", MF_CHECK | MF_GROUP1, fontSet, NULL),
};

= = = =

Awesome, my patch lives on 12 years later:

= = = =

bash-3.2$ diff operation.c.orig operation.c
1269,1270c1269,1272
<> MI_FLAGCB("Sony 16", MF_CHECK | MF_GROUP1,
> fontSet, "-*-fixed-medium-r-normal--16-120-100-100-c-80-iso8859-1"),
> MI_FLAGCB("Sony 24", MF_CHECK | MF_GROUP1,
> fontSet, "-*-fixed-medium-r-normal--24-170-100-100-c-120-iso8859-1");

= = = =

Drat. The 16 and 24 fonts don't show up; the original bug with Fixed 12 missing is still there. Time to dust the cobwebs and open a ticket.

However, the screen shot function works great!






p.s. here's the NetBSD ticket info:

Thank you very much for your problem report.
It has the internal identification `pkg/38129'.
The individual assigned to look at your
report is: pkg-manager.

>Category: pkg
>Responsible: pkg-manager
>Synopsis: xpaint package has patch for font selection that do not appear
>Arrival-Date: Sat Mar 01 16:15:00 +0000 2008

No comments: