Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225286 --- Comment #17 from Ivana Varekova <varekova@xxxxxxxxxx> 2009-11-27 06:16:13 EDT --- Hello, thanks for the comments, they are incorporated in aspell-0.60.6-8.fc13 in the last version I move aspell-import to documentation part (in the previous version aspell-import have not executable bit and nobody wants the rights back so I think the better solution is ot put it to the documentation). (In reply to comment #15) > I would even prefer a yet more radical splitting: > > * a -lib package with content of /usr/lib/aspell-0.60, the libs and > the .mo files > > * move the precat, preunzip, prezip, prezip-bin and word-list-compress > binaries either into -devel (afais, they are only used to create > dictionaries), or into a new -utils subpackage > > * move 'aspell-import' into -utils > > * keep aspell, ispell, run-with-aspell, spell and related man pages in > main package. 'aspell' adds a ncurses dep which is not needed by > -libs > > * at first glance, 'aspell.info' contains end user information only > and should stay in main and does not need to be moved into -lib For me the current system seems to be good enough (aspell-import is in documentation section - but nobody wants it) > Other issues: > > * remove the explicit 'Requires: aspell-en'; I am pretty sure that my > mom will never have use for the english dictionary but uses the > german one only fixed > * are all the 'Conflicts:' really required for a recent system? thanks, removed > * the > | Provides: pspell < 0.13 > | Obsoletes: pspell < 0.13 thanks, removed > does not seem to make sense; either write 'Provides: pspell = 0.13', > or remove it completely. Ditto for -devel > > * the > > | Requires(pre): /sbin/install-info > > is wrong and should be 'Requires(post)' thanks, fixed > * /sbin/ldconfig MUST be required the dependency is automatically added - so no need to be there > * afair, there was a rule, that Summary: must not begin with 'A'. thanks, fixed > * the 'exit 0' in the scriptlets does not make sense thanks, fixed Add #16 | exec "$@" is bogus - try e.g. exec "echo hello". Is the package ok now? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review