On 2014-02-08 11:19, Tobias Frost wrote: > 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
Thanks for testing Agustin#s patch. > 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. I don't think we should put this complexity into the maintainer scripts. I'd rather consider this as a "false positive" in lintian since the package ships a dangling link, but after finished installation the link won't be dangling any more. So we should rather try to get this "fixed" in lintian, maybe even conditionally depending on some command in the postinst: if (has_dangling_link(/usr/lib/*spell/*.$hash -> /var/lib/*spell/*.$hash) and postinst_calls(update-foo) ignore_dangling_link() I'd prefer this way instead of (automatically) adding lintian-overrides to every aspell-xx package. Andreas -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
