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

Debarshi Ray <[email protected]> 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 <[email protected]> ---
(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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to