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=611175 Iain Arnell <iarnell@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Flag|fedora-review? |fedora-review+ --- Comment #1 from Iain Arnell <iarnell@xxxxxxxxx> 2010-07-04 03:11:09 EDT --- + source files match upstream. c05a0bf6a5a6ae0a6c15d1401bcfeeed yash-2.22b0.tar.gz a1be011e63f94f4a2c3df802815b17e5 yash-doc-2.22b0-ja.tar.gz + package meets naming and versioning guidelines. + specfile is properly named, is cleanly written and uses macros consistently. + summary is OK. + description is OK. + dist tag is present. + license field matches the actual license. GPLv2+ + license is open source-compatible. + upstream license is included as %doc + latest version is being packaged. + BuildRequires are proper. + compiler flags are appropriate. + %clean is present. + package builds in mock http://koji.fedoraproject.org/koji/taskinfo?taskID=2293436 + package installs properly. + rpmlint has no significant complaints: yash.src: I: checking yash.src: W: spelling-error %description -l en_US zsh -> sh, ssh, ash yash.src: W: spelling-error %description -l en_US redirections -> redirection, re directions, re-directions yash.src: I: checking-url http://yash.sourceforge.jp/ (timeout 10 seconds) yash.src:54: W: configure-without-libdir-spec yash.src: W: no-cleaning-of-buildroot %install yash.src: W: no-buildroot-tag yash.src: I: checking-url http://dl.sourceforge.jp/yash/48168/yash-doc-2.22b0-ja.tar.gz (timeout 10 seconds) yash.src: I: checking-url http://dl.sourceforge.jp/yash/48168/yash-2.22b0.tar.gz (timeout 10 seconds) yash.x86_64: I: checking yash.x86_64: W: spelling-error %description -l en_US zsh -> sh, ssh, ash yash.x86_64: W: spelling-error %description -l en_US redirections -> redirection, re directions, re-directions yash.x86_64: I: checking-url http://yash.sourceforge.jp/ (timeout 10 seconds) yash.x86_64: W: no-manual-page-for-binary yash yash-debuginfo.x86_64: I: checking yash-debuginfo.x86_64: I: checking-url http://yash.sourceforge.jp/ (timeout 10 seconds) yash-doc.noarch: I: checking yash-doc.noarch: I: checking-url http://yash.sourceforge.jp/ (timeout 10 seconds) 4 packages and 0 specfiles checked; 0 errors, 8 warnings. + final provides and requires are sane: yash = 2.22-0.1.b0.fc14 yash(x86-64) = 2.22-0.1.b0.fc14 = libc.so.6()(64bit) libc.so.6(GLIBC_2.2.5)(64bit) libc.so.6(GLIBC_2.3.4)(64bit) libc.so.6(GLIBC_2.3)(64bit) libc.so.6(GLIBC_2.4)(64bit) libm.so.6()(64bit) libm.so.6(GLIBC_2.2.5)(64bit) libtinfo.so.5()(64bit) rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(FileDigests) <= 4.6.0-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 rpmlib(PayloadIsXz) <= 5.2-1 rtld(GNU_HASH) + %check is present and all tests pass. Executing(%check): /bin/sh -e /var/tmp/rpm-tmp.5FWdCk + umask 022 + cd /builddir/build/BUILD + cd yash-2.22b0 + unset DISPLAY + make test make[1]: Entering directory `/builddir/build/BUILD/yash-2.22b0/tests' gcc -std=c99 -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -D HAVE_CONFIG_H -o invoke invoke.c -lm -ltinfo gcc -std=c99 -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -D HAVE_CONFIG_H -o resetsig resetsig.c -lm -ltinfo make[2]: Entering directory `/builddir/build/BUILD/yash-2.22b0' make[3]: Entering directory `/builddir/build/BUILD/yash-2.22b0/builtins' make[3]: Nothing to be done for `all'. make[3]: Leaving directory `/builddir/build/BUILD/yash-2.22b0/builtins' make[3]: Entering directory `/builddir/build/BUILD/yash-2.22b0/lineedit' make[3]: Nothing to be done for `all'. make[3]: Leaving directory `/builddir/build/BUILD/yash-2.22b0/lineedit' make[2]: Leaving directory `/builddir/build/BUILD/yash-2.22b0' Testing ../yash for *.tst alias.p ok alias.y ok array.y ok builtin.p ok builtin.y ok dirstack.y ok error.p ok error.y ok expand.p ok expand.y ok fnmatch.p ok help.y ok history.y ok input.p ok input.y ok job.p ok job.y skipped lineedit.y ok option.p ok option.y ok parser.p ok parser.y ok path.p ok path.y ok printf.y ok redir.p ok redir.y ok sig.p ok sig.y ok test.y ok variable.p ok variable.y ok All test(s) completed successfully. make[1]: Leaving directory `/builddir/build/BUILD/yash-2.22b0/tests' + exit 0 + no shared libraries are added to the regular linker search paths. + owns the directories it creates. + doesn't own any directories it shouldn't. + no duplicates in %files. + file permissions are appropriate. + no generically named files + code, not content. + large documentation is in separate -doc sub-package. + %docs are not necessary for the proper functioning of the package. The only minor complaint is there's no man page, but it's not a blocker. APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review