[Bug 225286] Merge Review: aspell

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: aspell


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225286





------- Additional Comments From enrico.scholz at informatik.tu-chemnitz.de  2007-02-13 19:24 EST -------
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

=======

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

* are all the 'Conflicts:' really required for a recent system?

* the
  | Provides: pspell < 0.13
  | Obsoletes: pspell < 0.13

  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)'

* /sbin/ldconfig MUST be required

* afair, there was a rule, that Summary: must not begin with 'A'.

* the 'exit 0' in the scriptlets does not make sense


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.