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

Bug#543512: update-fonts-alias should skip removed fonts



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



Reply to: