On Sat, Feb 08, 2014 at 07:09:22PM +0100, Agustin Martin wrote: > 2014-02-08 11:19 GMT+01:00 Tobias Frost <t...@frost.de>: > > Hallo Augustin, > > > > thanks for the patch, look and feel of it is very good. > > I also tried it on the modified aspell-it (just randomly picked pet > > package to test piuparts-happy-dictionaries packaging) and the result > > was as expected. > > > > However, when I disabled "auto-compat" by removing the line in > > *.info-aspell something seems goes wrong: > > The package then some creates a bogus hashfile name like > > HASH(0x6e9848).rws and the installed autoscripts looks like it would > > ignore the disabled auto-compat. > > Hi, > > Thanks a lot for the testing, this was a bug at my side. I prepared a > preliminary version for simple packages like aspell-it. Then improved > it and extended to more complex dicts like aspell-en, and in this step > I broke the aspell-it like case :-(.
Hi, Just uploaded 1.22.0~exp2 to experimental. Besides dh_aspell-simple stuff it contains some code to stop using debconf as only registry. > > I just have an additional suggestion, however this is just something > > something "pedantic". > > Currently installdeb-aspell also installs a (dangling) symlink when > > auto-compat is turned on. Lintian does emit a experimental warning on > > that. (e.g X: aspell-it: package-contains-broken-symlink) > > My proposal would be to add a snippet to the preinst-compat and > > postrm-compat scripts to create and delete the symlinks. > > Please see the attached patch, but please be warned: Perl's not a > > language I'm fluent in. > > Thanks, I will look at it. But I am beginning to think that this is > something that should have been dealt with from the very beginning by > aspell itself, without the need of any symlink, that is making aspell > look for hashes with this precedence > > /var/lib/aspell > /usr/lib/aspell > > This would make symlinks unneeded and be a cleaner implementation. Not > sure if aspell can have more than one search path, but if possible may > be the best choice. Lived this before :-( (http://bugs.debian.org/310590#48), seems this is not possible. I have asked upstream to confirm that to be sure. In the meantime I have been thinking that those symlinks could be created by the $class-autobuildhash scripts and record the target .rws and symlink full paths in a file like $compat.remove. This file could be parsed from postrm to make sure hashes and symlinks are removed. If there is no simple way of adding the paths, this could also be useful if multiarch is ever implemented for aspell. Added some code for this removal in the experimental package. After upload noticed that I should also check /usr/lib/$class, this is ~exp2. Regards, -- Agustin -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
