Bug 468155
Summary: | Review Request: robodoc - Extract documentation from source code | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Patrick Monnerat <patrick> |
Component: | Package Review | Assignee: | Lucian Langa <lucilanga> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, lucilanga, notting |
Target Milestone: | --- | Flags: | lucilanga:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2008-12-07 04:24:06 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Patrick Monnerat
2008-10-23 09:36:13 UTC
Review: OK source files match upstream: b1a0e0dda99bb26623682e62f0e59051 robodoc-4.99.36.tar.bz2 OK package meets naming and versioning guidelines. OK specfile is properly named, is cleanly written and uses macros consistently. OK summary is OK. OK description is OK. OK dist tag is present. OK build root is OK. OK license field matches the actual license. OK license is open source-compatible. OK license text included in package. OK BuildRequires are proper. OK compiler flags are appropriate. OK %clean is present. OK package builds in mock (rawhide, x86_64). OK package installs properly. OK debuginfo package looks complete. OK rpmlint is silent. OK final provides and requires are sane: robodoc = 4.99.36-1.fc10 robodoc(x86-64) = 4.99.36-1.fc10 = libc.so.6()(64bit) OK %check is not present; no test suite upstream. I was able to run program from the command line and generate a doc file. OK no shared libraries are added to the regular linker search paths. OK owns the directories it creates. OK doesn't own any directories it shouldn't. OK no duplicates in %files. OK file permissions are appropriate. OK code, not content. OK documentation is small, so no -doc subpackage is necessary. OK %docs are not necessary for the proper functioning of the package. OK no headers. OK no pkgconfig files. OK no static libraries. OK no libtool .la files. Suggestion: Please consider preserving the timestamp of the modified file (ChangeLog). A simple call to 'touch -r' will do. APPROVED. Many thanks for the review, Lucian. Will apply your suggestion. New Package CVS Request ======================= Package Name: robodoc Short Description: Extract documentation from source code Owners: monnerat Branches: F-9 F-10 InitialCC: Consider also using the standard sourceforge url in Source: http://fedoraproject.org/wiki/Packaging/SourceURL#Sourceforge.net cvs done. robodoc-4.99.36-2.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/robodoc-4.99.36-2.fc10 robodoc-4.99.36-2.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/robodoc-4.99.36-2.fc9 Thanks for CVS Kevin. Source URL was a typo :-( Applied your suggestion. Thanks. robodoc-4.99.36-2.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. robodoc-4.99.36-2.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report. |