Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=882559 --- Comment #2 from Nelson Marques <nmo.marques@xxxxxxxxx> --- Dan, I'm going to give a +1 to this package, but first I need you clarify 2 things for me: 1) BuildRequires: mate-file-manager-devel mate-file-manager-extensions, since the -extensions and -devel originate from the same sources why isn't -devel pulling the extensions (ex: through a Requires)? This would be in my opinion a good practice. 2) The style on the spec in different from previous packages; This isn't for sure an issue, so there's no big deal with this. If you believe 1) can be improved, it would be nice; otherwise, if no one else objects, I let it slip in. -- 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=6W40LddYDk&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review