On Tue, Mar 24, 2015 at 09:20:29AM +0100, Robert Luberda wrote: > reassign 781068 dictionaries-common-dev 1.23.17 > thanks > > 積丹尼 Dan Jacobson wrote: > > Package: iamerican > > Version: 3.4.00-1 > > > > Unpacking iamerican (3.4.00-1) over (3.3.02-6) ... > > dpkg: warning: unable to delete old directory '/var/lib/ispell': Directory > > not empty > > > > $ ls -Al /var/lib/ispell > > -rw-r--r-- 1 root root 7 03-24 10:07 american.compat > > -rw-r--r-- 1 root root 1245028 03-24 10:07 american.hash
Hi, Robert and Dan > Oh, I didn't notice this, as I have some other ispell dictionary > installed that keeps /var/lib/ispell known for dpkg. > > However you are right: previous versions of iamerican and ibritish > packages used to have the directory in *.deb, and updated version > creates the directory in the postinst script, so dpkg shows the warnings. The warning appears because of a race condition between old and new dicts. Dicts using old dictionaries-common-dev contained /var/lib/ dir in the package, dicts using new d-c-d don't. Since new dict no longer ships /var/lib subdir but creates it from autobuildhash dpkg complains when unpacking unless dicts using old schema are present. Note that this will happen only the first time a package is upgraded from old to new system. Since this will happen as much only once per package I do not think we should worry too much about it. > I can work-around the issue by adding the dir into ispell package, but > actually ispell does not needed by itself. The directory is maintained > by ispell dictionaries via dictionary-common, that's why I am reassiging > the bug. > > In my opinion it should be fixed by dictionary-common-dev by: > 1. Ensuring that install-deb-ispell creates an empty > debian/$package/var/lib/ispell directory during the build process. This was old behavior. The intention behind the new stuff was to leave room for a more complex handling of stuff there regarding aspell multiarch. For aspell things were finally done in a better way, however I do not think it worths re-adding this. > 2. Moving to prerm the stuff currently added into postrm - otherwise > when only the first point is implemented, dpkg will still show the > warnings during removal of the last installed ispell dictionary. This would require all aspell and ispell dicts using autobuildhash to ship modified prerm. Do not think it worths. > Most probably the prerm script could remove the files from > /var/lib/ispell not only when "$1" = "remove", but also when "$1" = > "upgrade" - this would ensure some flexibility, for example if at some > point in the future somebody for some reason would like to switch from > /var/lib/ispell to another directory. In case this is ever needed it can be implemented in autobuildhash script. No need to add new stuff to already existing dict packages. Regards, -- Agustin -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
