Spec URL: http://sundaram.fedorapeople.org/packages/simple-scan.spec
SRPM URL: http://sundaram.fedorapeople.org/packages/simple-scan-0.9.5-1.fc12.src.rpm
Simple Scan is an easy-to-use application, designed to let users connect their
scanner and quickly have the image/document in an appropriate format.
Builds in mock,
simple-scan.src: W: strange-permission simple-scan-0.9.5.tar.gz 0600L
simple-scan.src: W: no-cleaning-of-buildroot %install
simple-scan.src: W: no-buildroot-tag
simple-scan.x86_64: W: non-conffile-in-etc /etc/gconf/schemas/simple-scan.schemas
3 packages and 0 specfiles checked; 0 errors, 4 warnings.
The permission issue, you should probably fix when committing this to cvs, the rest are ignorable.
As an aside, I notice the following in the docs:
PNG files are not compressed like JPEG so they contain the raw data from the scanner. This makes them much larger and not suitable for distributing. If you have scanned multiple pages then a PNG file will be written for each page.
This seems confusing at best. The important difference between PNG and JPEG is not compressed vs uncompressed, but lossless vs lossy compression. Of course, it _could_ be that the application decides to save PNG uncompressed, but that would just be a very bad choice...
I don't have a scanner to test this with here at home, but at least the application detects this and handles it nicely.
package name: ok
spec file name: ok
packaging guidelines: you need to add the usual gconf schema scriptlets, and the patch should have an upstream bug ref or something like that
license field: ok
license file: ok
spec file language: ok
spec file legible: ok
upstream sources: ok
ExcludeArch: ok, none
BuildRequires: the GConf2 BR is redundant, GConf2-devel will pull it in. Might be nice to group the autofoo + gnome-common requires with the patch that makes them necessary, and also add a comment to the autogen call that ties it to the patch
locale handling: ok
shared libs: ok, none
system libs: ok
directory ownership: might want to require yelp, for /usr/share/gnome/help (and to make help work)
duplicate files: ok
file permissions: ok
macro use: ok
permissible content: yes
large docs: ok
%doc content: ok
headers: ok, none
static libs: ok, none
pc files: ok, none
shared libs: ok
devel package: ok
libtool archives: ok
gui apps: ok
duplicate ownership: ok
%install: outdated checklist item
utf8 filenames: ok
trivia: some trailing whitespace in the spec...
(In reply to comment #2)
> As an aside, I notice the following in the docs:
> PNG files are not compressed like JPEG
Filed a bug report at
Updated spec as per review
Would be great to see this in F13 as gnome-scan does not really work for me and is basicly unmaintained¹ now...
Looks good now. Approved.
New Package CVS Request
Package Name: simple-scan
Short Description: Simple scanning utility
CVS done (by process-cvs-requests.py).