Bug 444625 - Review Request: GMT-doc - Documentation for Generic Mapping Tools
Summary: Review Request: GMT-doc - Documentation for Generic Mapping Tools
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Patrice Dumas
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 438043
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-04-29 16:02 UTC by Orion Poplawski
Modified: 2008-05-20 16:23 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-05-20 16:23:24 UTC
Type: ---
Embargoed:
pertusus: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Orion Poplawski 2008-04-29 16:02:20 UTC
Spec URL: http://www.cora.nwra.com/~orion/fedora/GMT-docs
SRPM URL: http://www.cora.nwra.com/~orion/fedora/GMT-docs-4.2.1-1.src.rpm
Description:
This package provides the documentation for the GMT (Generic Mapping Tools)
package.

Comment 1 Jochen Schmitt 2008-04-29 18:26:39 UTC
Some review checks:

+ rpmlint of source package ok.
+ Local buid works fine.
+ Copyright ok

- I think the whole documentatioon should put into %{_docdir]/GMT
- What is the task of the all subpackage which doesn't contain any files.



Comment 2 Orion Poplawski 2008-04-29 19:23:55 UTC
(In reply to comment #1)
> - I think the whole documentation should put into %{_docdir]/GMT

Possibly.  The GMT install script would make the following subdirs:

pdf
man
web
tut

I could see tutorial going in doc/tutorial.  As for html, can web servers serve
content out of /usr/share?

> - What is the task of the all subpackage which doesn't contain any files.

%package        all
Summary:        All documentation for GMT
Group:          Applications/Engineering
Requires:       %{name} = %{version}-%{release}
Requires:       %{name}-pdf = %{version}-%{release}
Requires:       %{name}-html = %{version}-%{release}
Requires:       %{name}-tutorial = %{version}-%{release}

to bring in all doc packages in one shot.



Comment 3 Patrice Dumas 2008-04-29 22:25:29 UTC
I think that it would be better to call the package GMT-doc for
consistency with most of the other packages. Also the Group should
be
Group:  Documentation

I think that all the docs should be in only one package.

The html docs should not be installed to be served by a web server but 
for local browsing.

Also I think that it would be better if the docs were installed in
%{_docdir}/GMT-%{version}/

One think which is not completely clear to me is whether this 
package should depend on GMT or not. This is not so clear to me.
Are the examples used in the tutorial or the doc? If so it could depend
on GMT-examples.

Comment 4 Orion Poplawski 2008-05-07 15:18:32 UTC
How about this one:

Spec URL: http://www.cora.nwra.com/~orion/fedora/GMT-docs
SRPM URL: http://www.cora.nwra.com/~orion/fedora/GMT-docs-4.3.0-1.src.rpm

* Tue Apr 6 2008 Orion Poplawski <orion.com> 4.3.0-1
- Update to 4.3.0
- Make single package

Technically, GMT owns %{_docdir}/GMT-%{version}/, so it should require GMT,
though perhaps not a big deal.


Comment 5 Patrice Dumas 2008-05-12 10:45:59 UTC
I suggest calling the package GMT-doc and not GMT-docs to be
more consistent with the guidelines and most of the packages.

Also the requires for GMT should be
Requires:       GMT = %{version}
since the release are not correlated.

Otherwise looks good.

Comment 6 Orion Poplawski 2008-05-12 16:00:33 UTC
* Mon Apr 12 2008 Orion Poplawski <orion.com> 4.3.0-2
- Rename to GMT-doc
- Require only GMT-%%{version}, drop release

Spec URL: http://www.cora.nwra.com/~orion/fedora/GMT-doc.spec
SRPM URL: http://www.cora.nwra.com/~orion/fedora/GMT-doc-4.3.0-2.src.rpm

Thanks!

Comment 7 Patrice Dumas 2008-05-12 22:14:36 UTC
* rpmlint not problematic
GMT-doc.noarch: E: zero-length /usr/share/doc/GMT-4.3.0/html/GMT_Docs/img246.png
* free software, license included
* follow guidelines
* match upstream
e068d5be0859c1ea5a8a0b15d4ad5810  GMT4.3.0_pdf.tar.bz2
633f4e648c80d920c4c2080b58fc1fe8  GMT4.3.0_tut.tar.bz2
3fcef23a62b61d4d6abcb8b5b1a4a701  GMT4.3.0_web.tar.bz2
* %files section right

APPROVED

Comment 8 Orion Poplawski 2008-05-12 22:24:25 UTC
New Package CVS Request
=======================
Package Name: GMT-doc
Short Description: Documentation for Generic Mapping Tools
Owners: orion
Branches: F-9 F-8 F-7 EL-5 EL-4
InitialCC: 
Cvsextras Commits: yes


Comment 9 Kevin Fenzi 2008-05-13 17:09:37 UTC
cvs done.

Comment 10 Orion Poplawski 2008-05-20 16:23:24 UTC
Checked in and built.


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