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

[bugzilla at redhat.com]

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

Debarshi Ray <debarshir at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|fedora-review?              |fedora-review+



--- Comment #5 from Debarshi Ray <debarshir at redhat.com> ---
(In reply to Kalev Lember from comment #4)
> (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.

Ok.

> >     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.

Thanks, Kalev.

ACCEPTED

-- 
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