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

Lubomir Rintel <lkund...@v3.sk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |lkund...@v3.sk
         Depends On|                            |1375380
           Assignee|nob...@fedoraproject.org    |lkund...@v3.sk
              Flags|                            |fedora-review+



--- Comment #2 from Lubomir Rintel <lkund...@v3.sk> ---
This looks good, thanks.

* Package name correct
* License tag correct
* License good for Fedora
* SPEC file clean and legible
* Filelist sane
* Requires/provides look good
* Uses proper compiler flags

I didn't really bother to check the mock build since icestorm is not yet in,
but I guess it doesn't matter too much.

This package is APPROVED


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1375380
[Bug 1375380] Review Request: icestorm - Lattice iCE40 FPGA bitstream
creation/analysis/programming tools
-- 
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 -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

Reply via email to