On Wed, Dec 9, 2009 at 17:18:49 +0700, Theppitak Karoonboonyanan wrote: > On Thu, Dec 03, 2009 at 02:04:01AM +0100, Julien Cristau wrote: > > I think I've fixed this, I put a package up for testing at > > http://people.debian.org/~jcristau/xfonts-utils_7.5+1~thep1.dsc > > I haven't tested that this works properly, and I don't know when I can > > do that, so any help would be welcome (install that package, purge it, > > install a font package using the patched debhelper, etc). > > Problems found: > - xfonts-utils.{postinst,postrm}.in include shell lib without defining > $THIS_PACKAGE and $THIS_SCRIPT variables. > - xfonts-utils.postinst.in tries to create the $EXCLUDE file without preparing > the directory. > - xfonts-utils.postrm.in leaves /var/lib/xfonts directory on purge, > as detected by piuparts.
Oops, thanks for catching this. postinst configure could probably avoid the version check, and postrm purge can use rmdir --ignore-fail-on-non-empty (or even rm -rf) to remove /var/lib/xfonts instead of using ls to check. With those changes I think this is ready. Cheers, Julien -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected]
