[Bug 168624] Review Request: hspell: a Hebrew spell checker

[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: Review Request: hspell: a Hebrew spell checker


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





------- Additional Comments From tcallawa at redhat.com  2005-09-22 14:57 EST -------
(In reply to comment #5)
> Hi Tom, thanks for your review of my spec file and its corrections.
> 
> - I believe that for the casual user, the "slim" flavor of the package is
> better that the "fat" one. It takes only 200Kb (instead of 700Kb) and would
> generate less false positives. Don't misunderstand that I dislike the
> morphological analyzer; it is my code and hopefully some great things would be
> based on it. But currently I suspect that it is not too useful for John Doe. If
> you think differently, please let me know.

I think that this is something that you should conditionalize within the
application itself, and enable by default. That way, one binary rpm package
provides the fat functionality for the power users, but it is not enabled by
default to avoid the false positives. I don't think the size difference merits
disabling it by default, since it is only 500K more (if it was 50MB more, then
perhaps).

Basically, what I'm saying is that the binary should come with the fat support
enabled, but in order to use that, they should have to pass hspell -fat (or
something).

> - I respect your judgment regarding the perl-require hack. Too bad that we have
> so many requirements only because we happen to distribute an addon script that
> is written in perl.

Well, you really do want those perl requirements to be there for users, so that
when they go to run that addon script, it actually works. :)

> - I don't think that there are many developers who needs the Hebrew description
> of what a -devel package is. Thanks for suggesting it, though.

You should add it to be complete, or remove the hebrew translation for the base
package. All or nothing, really. :)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.