[Bug 484591] Review Request: muse - Midi/Audio Music Sequencer

[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.


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





--- Comment #3 from Hans de Goede <hdegoede@xxxxxxxxxx>  2009-02-10 03:14:23 EDT ---
With that said, and hopefully made very clear! I would like to review this
package as I think it is great that someone is working on moving packages from
planetccrma into Fedora proper.

The Epoch is fine, but with the other 2 issues I have to side with Ralf, using
"AutoProv: no" is not acceptable, your package will miss essential requires and
adding them manually is error prone and will break when the soname's of
libraries you use change (and you do a rebuild). The correct solution here is
to use filtering of the generated provides as explained in the previous review
request.

Also the warnings are serious and need fixing, Ralf has already explained how
(replace "%d" with "%zd"), let me know if there are other warnings which you
need help fixing.

Can you please do a new revision with these 2 issues fixed? then I'll do a full
review.

-- 
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

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