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

Zbigniew Jędrzejewski-Szmek <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |POST
                 CC|                            |[email protected]
           Assignee|[email protected]    |[email protected]
              Flags|                            |fedora-review+



--- Comment #1 from Zbigniew Jędrzejewski-Szmek <[email protected]> ---
URL is messed up. Also, no https?

Suggestion: make %{?_smp_mflags} V=1 → %make_build
Suggestion: make install DESTDIR=%{buildroot} → %make_install

+ package name is OK
+ license is acceptable for Fedora (BSD)
+ license is specified correctly
+ builds and install OK
+ fedora-review finds no issues
+ scriptlets are correct
+ R/BR/P look correct
+ %check is present and passes

Package is APPROVED. Please note the comments at the top.

-- 
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 -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to