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=676930 HaÃkel GuÃmar <karlthered@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ --- Comment #2 from HaÃkel GuÃmar <karlthered@xxxxxxxxx> 2011-02-14 11:27:29 EST --- gmock (C++ package requires gtest >= 1.5 which is only available for f15+) MUST: rpmlint must be run on src.rpm and rpm. KO $ rpmlint -iv mock-1.5.0-1.fc15.src.rpm gmock.src: I: checking gmock.src: W: spelling-error %description -l en_US jMock -> j Mock, jock, mock The value of this tag appears to be misspelled. Please double-check. gmock.src: W: spelling-error %description -l en_US matchers -> marchers, matcher, matches The value of this tag appears to be misspelled. Please double-check. gmock.src: W: spelling-error %description -l en_US minGW -> mingy, mingle, Mingus The value of this tag appears to be misspelled. Please double-check. gmock.src: I: checking-url http://code.google.com/p/googlemock/ (timeout 10 seconds) gmock.src:43: W: make-check-outside-check-section # save the rest for use in make check Make check or other automated regression test should be run in %check, as they can be disabled with a rpm macro for short circuiting purposes. gmock.src: I: checking-url http://googlemock.googlecode.com/files/gmock-1.5.0.tar.bz2 (timeout 10 seconds) gmock.src: W: invalid-url Source0: http://googlemock.googlecode.com/files/gmock-1.5.0.tar.bz2 HTTP Error 404: Not Found The value should be a valid, public HTTP, HTTPS, or FTP URL. $ rpmlint -iv gmock-1.5.0-1.fc15.x86_64.rpm gmock.x86_64: I: checking gmock.x86_64: W: spelling-error %description -l en_US jMock -> j Mock, jock, mock The value of this tag appears to be misspelled. Please double-check. gmock.x86_64: W: spelling-error %description -l en_US matchers -> marchers, matcher, matches The value of this tag appears to be misspelled. Please double-check. gmock.x86_64: W: spelling-error %description -l en_US minGW -> mingy, mingle, Mingus The value of this tag appears to be misspelled. Please double-check. gmock.x86_64: I: checking-url http://code.google.com/p/googlemock/ (timeout 10 seconds) gmock.x86_64: W: shared-lib-calls-exit /usr/lib64/libgmock.so.0.0.0 _exit@xxxxxxxxxxx This library package calls exit() or _exit(), probably in a non-fork() context. Doing so from a library is strongly discouraged - when a library function calls exit(), it prevents the calling program from handling the error, reporting it to the user, closing files properly, and cleaning up any state that the program has. It is preferred for the library to return an actual error code and let the calling program decide how to handle the situation. 1 packages and 0 specfiles checked; 0 errors, 4 warnings. $ rpmlint -iv gmock-devel-1.5.0-1.fc15.x86_64.rpm gmock-devel.x86_64: I: checking gmock-devel.x86_64: I: checking-url http://code.google.com/p/googlemock/ (timeout 10 seconds) gmock-devel.x86_64: W: no-documentation The package contains no documentation (README, doc, etc). You have to include documentation files. gmock-devel.x86_64: W: no-manual-page-for-binary gmock-config Each executable in standard binary directories should have a man page. gmock-devel.x86_64: W: no-manual-page-for-binary gmock_doctor.py Each executable in standard binary directories should have a man page. 1 packages and 0 specfiles checked; 0 errors, 3 warnings. the make check warning is a false positive. the shared-lib-exit-call warning is non-relevant due to package nature (mocking framework) the other warnings are non-relevant. MUST: package named accordingly to package naming guidelines. OK MUST: spec file name match %{name} OK MUST: package meets packaging guidelines OK MUST: package must be licensed under a fedora-compliant license OK (BSD) MUST: License field in package spec match actual license OK MUST: spec in legible american english OK MUST: sources provided match upstream's OK provided sources sha1sum: 76d8f5a221c93105304d71e33391dc70af573d6a upstream sources sha1sum: 76d8f5a221c93105304d71e33391dc70af573d6a MUST: package successfully compiles on at least one primary architecture (all of them) OK MUST: all build dependencies are listed in BR OK MUST: package must not bundle copies of system libraries. OK Note: it does bundle google test sources (gtest) but by default it builds agains system copy. MUST: package must own all directories it creates. OK MUST: package does not list a file more than once in %files section. OK MUST: permissions are properly set. OK MUST: package consistenly use macros. OK MUST: package contains permissable content. OK MUST: package does not own directories owned by other packages. OK MUST: all filenames in package are valid UTF-8. OK SHOULD: mock builds were done for fedora devel on all primary architectures (x86/x86_64). OK http://koji.fedoraproject.org/koji/taskinfo?taskID=2839341 SHOULD: the module provided works (tested with gtest). OK May i suggest you to open a f14 branch with gmock 1.4.0 ? it shouldn't be too hard though. Hereby, i give you my blessing for importing this package into fedora collection. -- 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