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

Kalev Lember <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?(kalevlember@gmail |
                   |.com)                       |



--- Comment #4 from Kalev Lember <[email protected]> ---
(In reply to Debarshi Ray from comment #3)
> gegl03-devel.x86_64: E: zero-length /usr/share/vala/vapi/gegl-0.3.deps
> 1 packages and 0 specfiles checked; 1 errors, 2 warnings.
> 
> All those are harmless, except the one about the zero length VAPI file.
> Maybe we should build with --without-vala ?

It's just the .deps file that's zero length, the vapi file seems fine. I think
it's harmless, but would be nice to fix it upstream by not including the empty
.deps file.

>     Some minor grammatical errors in %description. "GIMPs itches" should be
> "GIMP's itches", "minimal dependencies. and a" should be "minimal
> dependencies and a".

Fixed both.


>     There should be a Requires on ImageMagick.
> operations/common/magick-load.c has a fallback image loader which uses
> convert(1).

Done.


* Fri Mar 13 2015 Kalev Lember <[email protected]> - 0.3.0-0.3.gitc9bbc81
- Package review fixes (#1201469)
- Fix grammar errors in package description
- Add a runtime dep on /usr/bin/convert

Spec URL: https://kalev.fedorapeople.org/gegl03.spec
SRPM URL:
https://kalev.fedorapeople.org/gegl03-0.3.0-0.3.gitc9bbc81.fc22.src.rpm

-- 
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]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to