[Bug 226388] Merge Review: sane-backends

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Nils Philippsen <nphilipp at redhat.com> 2010-02-26 12:09:18 EST ---
- PASS: rpmlint

sane-backends.src: W: strange-permission hal-usb-scanner-add 0755

-> bogus: it's an executable script

sane-backends.x86_64: W: file-not-utf8 /usr/share/doc/sane-backends-1.0.20/NEWS
sane-backends.x86_64: W: file-not-utf8
/usr/share/doc/sane-backends-1.0.20/plustek/Plustek-PARPORT.txt
sane-backends.x86_64: W: file-not-utf8
/usr/share/doc/sane-backends-1.0.20/AUTHORS
sane-backends.x86_64: W: file-not-utf8
/usr/share/doc/sane-backends-1.0.20/ChangeLog
sane-backends.x86_64: W: file-not-utf8
/usr/share/doc/sane-backends-1.0.20/gt68xx/gt68xx.CHANGES
sane-backends.x86_64: W: file-not-utf8
/usr/share/doc/sane-backends-1.0.20/u12/U12.todo
sane-backends.x86_64: W: file-not-utf8
/usr/share/doc/sane-backends-1.0.20/plustek/Plustek-USB.txt
sane-backends.x86_64: W: file-not-utf8
/usr/share/doc/sane-backends-1.0.20/mustek/mustek.CHANGES
sane-backends.x86_64: W: file-not-utf8
/usr/share/doc/sane-backends-1.0.20/plustek/Plustek-PARPORT-TODO.txt
sane-backends.x86_64: W: file-not-utf8
/usr/share/doc/sane-backends-1.0.20/u12/U12.changes

-> convert to UTF-8

sane-backends-devel.x86_64: W: spurious-executable-perm
/usr/lib64/pkgconfig/sane-backends.pc

-> fix permissions

sane-backends-libs.x86_64: W: obsolete-not-provided sane-backends

-> required for splitting off libs subpackage

sane-backends-libs.x86_64: W: no-documentation

-> in main package

sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-ma1509.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-canon_pp.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-epson2.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-leo.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-dc210.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-pie.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-apple.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-as6e.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-dc25.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-epson.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-umax.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-canon630u.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-teco1.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-coolscan3.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-umax_pp.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-avision.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-hp3500.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-v4l.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-net.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-nec.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-sm3600.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-hp5400.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-mustek_usb2.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-matsushita.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-test.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-umax1220u.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-hpljm1005.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-hp5590.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-bh.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-canon.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-ibm.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-dc240.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-snapscan.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-sm3840.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-agfafocus.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-coolscan2.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-hpsj5s.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-epjitsu.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-genesys.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-niash.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-hp.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-teco2.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-sp15c.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-st400.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-hp4200.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-ricoh.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-artec.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-abaton.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-artec_eplus48u.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-canon_dr.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-mustek_usb.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-sceptre.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-cardscan.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-pixma.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-u12.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-plustek.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-plustek_pp.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-xerox_mfp.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-coolscan.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-gt68xx.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-teco3.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-sharp.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-lexmark.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-microtek.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-hs2p.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-qcam.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-stv680.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-microtek2.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-s9036.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-dell1600n_net.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-fujitsu.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-rts8891.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-mustek_pp.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-dll.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-tamarack.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-hp3900.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-mustek.so
sane-backends-libs.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/sane/libsane-dmc.so

-> bogus: these are the SANE driver plugins

sane-backends-libs-gphoto2.x86_64: W: spelling-error Summary(en_US) gphoto ->
photo, g photo, Photostat

-> bogus: gphoto is a proper name

sane-backends-libs-gphoto2.x86_64: W: no-documentation

-> in main package

5 packages and 0 specfiles checked; 0 errors, 94 warnings.

- GOOD: package is named according to Package Naming Guidelines
- GOOD: spec file named correctly
- GOOD: package meets Packaging Guidelines
- GOOD: licensed with approved license and meets Licensing Guidelines
- GOOD: license field matches actual license
- GOOD: license text contained as documentation
- GOOD: spec file written in American English
- GOOD: spec file is legible
- GOOD: source matches upstream
- GOOD: package already build multiple times in koji
- GOOD: all build dependencies listed
- GOOD: handles locales properly
- GOOD: calls ldconfig
- GOOD: does not bundle copies of system libraries
- N/A: not relocatable
- GOOD: owns all directories it creates
- GOOD: no duplicate file listings
- GOOD: file permissions set properly
- GOOD: has %clean
- GOOD: uses macros consistently
- GOOD: contains code
- N/A: no large documentation
- GOOD: %doc doesn't affect runtime
- GOOD: header files in devel package
- N/A: no static libs
- GOOD: devel package requires pkgconfig
- GOOD: development *.so files in devel package
- GOOD: devel package requires fully versioned main package
- GOOD: removes libtool archives
- N/A: no GUI program
- GOOD: doesn't own files of other packages
- GOOD: removes buildroot in %install
- GOOD: all file names valid UTF-8

--> recommendations:
1. convert documentation files to UTF-8
2. fix permissions on pkgconfig file (needs rework of pkgconfig patch, redo
autoreconf patch)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.