Bug 468155 - Review Request: robodoc - Extract documentation from source code
Summary: Review Request: robodoc - Extract documentation from source code
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Lucian Langa
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-10-23 09:36 UTC by Patrick Monnerat
Modified: 2008-12-07 04:28 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-12-07 04:24:06 UTC
Type: ---
Embargoed:
lucilanga: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Patrick Monnerat 2008-10-23 09:36:13 UTC
Spec URL: http://monnerat.fedorapeople.org/robodoc.spec
SRPM URL: http://monnerat.fedorapeople.org/robodoc-4.99.36-1.fc8.src.rpm
Description:
  ROBODoc is a documentation tool (based on the AutoDocs program written
a long time ago by Commodore). It extracts specially formated comment
headers from the source file and puts them in a separate file. ROBODoc
thus allows you to include the program documentation in the source
code and avoid having to maintain two separate documents.
  ROBODoc can format the documentation in HTML, ASCII, AmigaGuide,
LaTeX, or RTF format. It is even possible to include parts of the
source code with function names that point their the documentation. It
also can create index tables for all your variables, classes,
functions, etc.
  The best feature of ROBODoc is that it works with many languages:
Assembler, C, Perl, LISP, Occam, Tcl/Tk, Pascal, Fortran, shell
scripts, and COBOL, basically any language that supports
comments/remarks.

rpmlint is silent.
koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=897383

Comment 1 Lucian Langa 2008-12-02 06:36:02 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.

Comment 2 Patrick Monnerat 2008-12-02 16:02:25 UTC
Many thanks for the review, Lucian. Will apply your suggestion.

Comment 3 Patrick Monnerat 2008-12-02 16:06:24 UTC
New Package CVS Request
=======================
Package Name: robodoc
Short Description: Extract documentation from source code
Owners: monnerat
Branches: F-9 F-10
InitialCC:

Comment 4 Kevin Fenzi 2008-12-04 00:43:29 UTC
Consider also using the standard sourceforge url in Source:
http://fedoraproject.org/wiki/Packaging/SourceURL#Sourceforge.net

cvs done.

Comment 5 Fedora Update System 2008-12-04 11:06:31 UTC
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

Comment 6 Fedora Update System 2008-12-04 11:07:58 UTC
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

Comment 7 Patrick Monnerat 2008-12-04 11:18:05 UTC
Thanks for CVS Kevin.
Source URL was a typo :-( Applied your suggestion. Thanks.

Comment 8 Fedora Update System 2008-12-07 04:24:03 UTC
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.

Comment 9 Fedora Update System 2008-12-07 04:28:30 UTC
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.


Note You need to log in before you can comment on or make changes to this bug.