Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497441 --- Comment #4 from Christoph Wickert <fedora@xxxxxxxxxxxxxxxxxxxx> 2009-04-24 07:24:04 EDT --- I'm not really into that but I suggest you look at the old bugs first to see if there is anything you can take from theses packages: https://bugzilla.redhat.com/show_bug.cgi?id=294001 https://bugzilla.redhat.com/show_bug.cgi?id=294001 Some quick comments on the spec file: Source0: Change it as described in http://fedoraproject.org/wiki/Packaging/SourceURL BuildRequires: Use several lines for better legibility %description: line breaks at 80 characters Do not create main.pro in the spec but include it as Source1 Package murmur and mumble into different packages Include an initscript for the murmur server. Get it from one of the other reviews. Mumbles needs to be patched to use the speex from Fedora and not the included one. See http://fedoraproject.org/wiki/Packaging:Guidelines#Duplication_of_system_libraries That's all I can say, because I have never used mumble and have no idea how it works. Simon, please step in here to get this review going. -- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review