Kalev Lember <> changed:

           What    |Removed                     |Added
             Status|NEW                         |ASSIGNED
                 CC|                            |
           Assignee|    |
              Flags|                            |fedora-review+

--- Comment #18 from Kalev Lember <> ---
Packaging looks nice and clean to me. Approving based on Jan Kalina's
unofficial review above.

I'd maybe change two minor things, one is that

%if 0%{?usesnapshot}
    %autosetup -n %name-%{commit0}
    %setup -q

... seems a bit unsymmetrical; would be easier to read if both were %autosetup
(or both were %setup -q) but not a mix like this.

The other minor thing is that -devel package doesn't need to contain '%license
LICENSE' since it's already included in the base package.

Feel free to change these if you want after importing. APPROVED

You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
package-review mailing list --
To unsubscribe send an email to

Reply via email to