[Bug 1201469] Review Request: gegl03 - Graph based image processing framework

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Kalev Lember <kalevlember@xxxxxxxxx> changed:

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



--- Comment #4 from Kalev Lember <kalevlember@xxxxxxxxx> ---
(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 <kalevlember@xxxxxxxxx> - 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
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]