[Bug 448215] Review Request: mozvoikko - Finnish Voikko spell-checker extension for Mozilla programs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


ville.skytta@xxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|nobody@xxxxxxxxxxxxxxxxx    |ville.skytta@xxxxxx
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review+




------- Additional Comments From ville.skytta@xxxxxx  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@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]