Hallo Agustin, Just tested the exp2... Looks good from here!
Just one question: disabling auto-compat does not have any effect anymore? (I think thats good, just want to ensure that this was intended) When I'm back from my business trip, I will then proceed to start the discussion for the overhaul of the simple aspell directoires on debian-devel et.al, if that is ok with you. Best regards, -- Tobias Am Donnerstag, den 13.02.2014, 17:22 +0100 schrieb Agustin Martin: > 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, > -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
