https://bugzilla.redhat.com/show_bug.cgi?id=978494 Debarshi Ray <debarshir@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution|--- |NEXTRELEASE Flags|fedora-cvs+ |fedora-cvs? Last Closed| |2013-07-11 06:55:26 --- Comment #4 from Debarshi Ray <debarshir@xxxxxxxxxx> --- (In reply to Mathieu Bridon from comment #1) > The package has only two minor issues, but I won't block the review on that. > > You can just fix the first issue when you import the package (it really is a > nitpick), and the second one is just something you could report upstream I > think. > > In any case, the package is approved. > > Summary > ======= > > [!]: Package consistently uses macro is (instead of hard-coded directory > names). > > => One included file is: > %{_prefix}/libexec/%{name}-service > Please use %{_libexecdir} instead. (you can fix it when you import the > package) Fixed and built for F20. -- 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=Hc5FLgtTwB&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review