Fedora Merge Review: sane-frontends http://cvs.fedora.redhat.com/viewcvs/devel/sane-frontends/ Initial Owner: nphilipp
Got following rpmlint output E: sane-frontends file-in-usr-marked-as-conffile /usr/share/sane/sane-style.rc A file in /usr is marked as being a configuration file. Store your conf files in /etc/ instead. W: sane-frontends unversioned-explicit-obsoletes sane The specfile contains an unversioned Obsoletes: token, which will match all older, equal and newer versions of the obsoleted thing. This may cause update problems, restrict future package/provides naming, and may match something it was originally not inteded to match -- make the Obsoletes versioned if possible. W: sane-frontends unversioned-explicit-provides sane The specfile contains an unversioned Provides: token, which will match all older, equal, and newer versions of the provided thing. This may cause update problems and will make versioned dependencies, obsoletions and conflicts on the provided thing useless -- make the Provides versioned if possible. E: sane-frontends configure-without-libdir-spec A configure script is run without specifying the libdir. configure options must be augmented with something like --libdir=%{_libdir}. Description is included along with warnings.
any updates here also?
No Response from maintainer
I'm currently building sane-frontends-1.0.14-3.fc7 which should fix the problems found by rpmlint, please check.
http://cvs.fedoraproject.org/viewcvs/devel/sane-frontends/ This has been lying around for too long already, I'll do the review myself.
- PASS: rpmlint sane-frontends.src: W: spelling-error Summary(en_US) frontend -> fronted, front end, front-end sane-frontends.src: W: spelling-error %description -l en_US scanadf -> scandal, scanned, scandium sane-frontends.src: W: spelling-error %description -l en_US xcam -> cam, scam, x cam sane-frontends.x86_64: W: spelling-error Summary(en_US) frontend -> fronted, front end, front-end 2 packages and 0 specfiles checked; 0 errors, 4 warnings. "frontend" is a usual form of spelling this term, "scanadf" and "xcam" are names of binaries - 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 - BAD: license text not 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 - N/A: no locale files included - N/A: no libraries in system paths included - GOOD: does not bundle copies of system libraries - N/A: not relocatable - BAD: does not own all directories it creates: /usr/share/sane/sane-style.rc - GOOD: no duplicate file listings - GOOD: file permissions set properly - GOOD: has %clean - BAD: inconsistently uses $RPM_BUILD_ROOT, %buildroot - GOOD: contains code - N/A: no large documentation - GOOD: %doc doesn't affect runtime - N/A: no header files - N/A: no static libs - N/A: no pkgconfig files - N/A: no library files - N/A: no devel package - N/A: no libtool archives - BAD: no desktop files for GUIish xcam program, no explanation either - GOOD: doesn't own files of other packages - GOOD: removes buildroot in %install - GOOD: all file names valid UTF-8 --> recommendations: 1. add license file COPYING to %doc, consider adding AUTHORS, README as well 2. don't distribute /usr/share/sane/sane-style.rc since it's only used by xscanimage (which we don't ship) 3. use either %buildroot or $RPM_BUILD_ROOT consistently 4. explain why xcam has no desktop file
fixed in sane-frontends-1.0.14-10.fc14, approved