Bug 442640 - Review Request: epeg - Immensely fast JPEG thumbnailer
Summary: Review Request: epeg - Immensely fast JPEG thumbnailer
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Patrice Dumas
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 442353
Blocks: 442808
TreeView+ depends on / blocked
 
Reported: 2008-04-15 23:23 UTC by Pavel Shevchuk
Modified: 2008-05-05 17:55 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2008-05-05 17:55:40 UTC
Type: ---
Embargoed:
pertusus: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Pavel Shevchuk 2008-04-15 23:23:10 UTC
Spec URL: http://rpm.scwlab.com/epeg-goes-rawhide/epeg.spec
SRPM URL: http://rpm.scwlab.com/epeg-goes-rawhide/epeg-0.9.1.042-1.fc9.src.rpm
RPMs for i386 and x86_64 built in mock: http://rpm.scwlab.com/epeg-goes-rawhide/

Description:
Epeg is insanely fast at loading large JPEG images and scaling them down to
tiny thumbnails. It's speedup will be proportional to the size difference
between the source image and the output thumbnail size as a count of their
pixels.

It makes use of libjpeg features of being able to load an image by only
decoding the DCT coefficients needed to reconstruct an image of the size
desired. This gives a massive speedup. If you do not try and access the pixels
in a format other than YUV (or GRAY8 if the source is grascale) then it also
avoids colorspace conversions as well.


RPMLint spits out only W: no-documentation on -devel package because there's no documentation in upstream package. Epeg mostly helper library for higher-lever thumbnailing API - epsilon.
Package works as expected - makes thumbnails of jpegs faster than imagemagick ;)

Comment 1 Pavel Shevchuk 2008-05-04 00:01:40 UTC
* Sun May 04 2008 Pavel "Stalwart" Shevchuk <stlwrt> - 0.9.1.042-2
- Fixed timestamp of source tarball
- Preserve timestamps of installed files

New spec: http://rpm.scwlab.com/epeg-goes-rawhide/epeg.spec
New SRPM: http://rpm.scwlab.com/fedora/e/9/epeg-0.9.1.042-2.x86_64/
epeg-0.9.1.042-2.fc9.src.rpm
Built RPMs: http://rpm.scwlab.com/fedora/e/9/

Comment 2 Patrice Dumas 2008-05-05 09:39:33 UTC
In the pc file there is -ljpeg, so a dependency for the devel package
to jpeg-devel is missing. Otherwise looks good.

Comment 3 Pavel Shevchuk 2008-05-05 11:47:23 UTC
* Tue May 06 2008 Pavel "Stalwart" Shevchuk <stlwrt> - 0.9.1.042-3
- Added missing dependencies for epeg-devel


New spec: http://rpm.scwlab.com/epeg-goes-rawhide/epeg.spec
New SRPM: http://rpm.scwlab.com/fedora/e/9/epeg-0.9.1.042-3.fc9.x86_64/
epeg-0.9.1.042-3.fc9.src.rpm
Built RPMs: http://rpm.scwlab.com/fedora/e/9/

Comment 4 Patrice Dumas 2008-05-05 12:35:55 UTC
* rpmlint is almost silent
epeg-devel.i386: W: no-documentation
* follow guidelines
* free software, MIT-like license included
* match upstream
b6eada50c40f46810530fe85a67f558d  epeg-0.9.1.042.tar.bz2
* %files section right
* library well packaged

APPROVED

Comment 5 Patrice Dumas 2008-05-05 12:39:17 UTC
A pkgconfig dependency for devel is missing.

Comment 6 Pavel Shevchuk 2008-05-05 13:00:04 UTC
New Package CVS Request
=======================
Package Name: epeg
Short Description: Immensely fast JPEG thumbnailer
Owners: stalwart
Branches: F-8 F-9
InitialCC:
Cvsextras Commits: yes

Comment 7 Kevin Fenzi 2008-05-05 15:48:24 UTC
cvs done.

Comment 8 Pavel Shevchuk 2008-05-05 17:54:48 UTC
Pushed to rawhide, thanks to everyone!


Note You need to log in before you can comment on or make changes to this bug.