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: mozvoikko - Finnish Voikko spell-checker extension for 
Mozilla programs


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


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[EMAIL PROTECTED]    |[EMAIL PROTECTED]
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review+




------- Additional Comments From [EMAIL PROTECTED]  2008-07-07 14:53 EST -------
I finally got around to updating to F-9 and thus reviewing this.

If I understand correctly, using xulrunner-unstable makes this prone to breakage
on updates - is there some versioned dependency towards some package that could
be used so that it would be easier to notice such cases?

The dependency on libvoikko works, but as the sources hardcode libvoikko.so.1
and libmalaga.so.7, it could be good to have the dependency to those sonames. 
I've done that in the pcsc-perl package, check it out for an example if you
agree implementing that would be feasible.

Would Applications/Internet be a better Group: value?  Aren't all Mozilla apps
this package works with Internet related ones?

When other Mozilla apps that can use this are shipped in Fedora, the dependency
on firefox should probably be dropped, right?

None of the above are blockers but just comments and food for thought, approved
as is.

-- 
Configure bugmail: https://bugzilla.redhat.com/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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to