Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=919469 --- Comment #18 from Michael Scherer <misc@xxxxxxxx> --- Wolfgang, you should fill the empty checkbox in fedora-review, not paste the output without checking. Anyway, I have a few questions : - why does it requires both yumex and packagekit ? To me, that's redundant, since the configure will use the first one that will be seen, so forcing one or the other kinda defeat the point. And it will fallback on yum in all case it seems. - the patches should be commented in the spec https://fedoraproject.org/wiki/Packaging:Guidelines#All_patches_should_have_an_upstream_bug_link_or_comment - I am not sure also that the directory ownership is clean, but that's pretty hard to see as mate is already installing lots of stuff without clear package ownership, and I am still looking on how to fill useful bug report for now based on my automated check :/ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=YkTPoUlibY&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review