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: cohoba - Cohoba is a GNOME interface for Telepathy. It aims to be innovative and simple https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=205343 ------- Additional Comments From sander@xxxxxxxxxxx 2006-09-11 05:02 EST ------- (In reply to comment #7) > Bad: > * There is an ownership problem with the '%{_libdir}/cohoba' directory. will fix > * I believe you still need a requires on pygtk2, since I don't see anything that > will pull in this requirement. will fix > * rpmlint produces the following error: > E: cohoba only-non-binary-in-usr-lib Hmm yes this is about /usr/lib64/cohoba/cohoba-applet I guess. Should that become /usr/libexec/cohoba-applet? > * The biggest issue was that I could never create an account to get it working > on FC5. The problem seemed to be due to account dialog's network spinner button > not being populated. I was only allowed to add my account & password, and not > the network. see below > > Minor: > * Group Tag should probably be Applications/Communications will fix (In reply to comment #8) > Ok, I found out what the problem was with the account not being able to work. > You missing a requires on telepathy-gabble. cohoba needs a connection manager, but not necessarily telepathy-gabble. IMHO it should be fixed upstream (show a message that a connection manager is needed if none are found). I know you package gossip, and it will have the same requirement (*a* connection manager) but for gossip it might make sense to require telepathy-gabble since that would make upgrading less painful. -- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review