Fedora Merge Review: aspell-de http://cvs.fedora.redhat.com/viewcvs/devel/aspell-de/
Can you update SPEC as suggested in other aspell package reviews?
I saw new tarball available on upstream site aspell6-de-20030222-1.tar.gz2. Can you update source in SPEC?
Also found, W: aspell-de summary-ended-with-dot German dictionaries for Aspell. Summary ends with a dot. E: aspell-de no-cleaning-of-buildroot %install You should clean $RPM_BUILD_ROOT in the %clean section and just after the beginning of %install section. Use "rm -Rf $RPM_BUILD_ROOT".
Review: + package builds in mock (development i386). - rpmlint is NOT silent for SRPM and RPM. But following messages are ignorable E: aspell-de no-binary E: aspell-de only-non-binary-in-usr-lib E: aspell-de configure-without-libdir-spec + SPEC file contains explanation about above warnings. + source files match upstream. 5950c5c8a36fc93d4d7616591bace6a6 aspell6-de-20030222-1.tar.bz2 + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is small; no -doc subpackage required. + %doc does not affect runtime. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage exists. + no .la files. + no translations are available. + Dose owns the directories it creates. + no duplicates in %files. + file permissions are appropriate. + Requires: aspell >= 12:0.60 + Provides: aspell-de = 50:20030222-3 + Not a GUI APP. APPROVED.
As build is available now, therefore CLOSING this review.