Bug 243501 - Review Request: qtiplot-doc - Documentation of qtiplot
Summary: Review Request: qtiplot-doc - Documentation of qtiplot
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Package Reviews List
Depends On: 240373
TreeView+ depends on / blocked
Reported: 2007-06-09 07:53 UTC by Frank Büttner
Modified: 2008-11-07 11:44 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2008-11-07 11:44:00 UTC
Type: ---

Attachments (Terms of Use)

Comment 1 Till Maas 2007-09-08 14:43:11 UTC
License should be GFDL  according to
and index.html

You use tabs and spaces in the spec, e.g. in Group you align with spaces but in
URL you align with tabs. Please use either tabs or spaces.

I did not try it but I guess you can simplify the spec when you use in %prep
%setup -q -n manual-en
dos2unix qtiplot.adp

and in %files
%doc *

Then you can remove
mkdir -p $RPM_BUILD_ROOT/%{_docdir}/%{name}
tar -xjf %{SOURCE0} -C $RPM_BUILD_ROOT/%{_docdir}/%{name}
dos2unix $RPM_BUILD_ROOT/%{_docdir}/%{name}/manual-en/qtiplot.adp
from %install.

In %files a %defattr(-,root,root,-) before the %doc is missing.

According to http://soft.proindependent.com/manuals.html the documentation is
version 0.8.9

Imho you should skip the %{?dist} tag in Release and ask rel-eng everytime you
built a package to sync it to all Fedora releases, this will also avoid updating
the manual everytime one updrades the system.

Comment 2 Till Maas 2007-09-08 14:47:27 UTC
I forgot: run dos2unix with --keepdate

Comment 3 Frank Büttner 2007-09-08 16:24:49 UTC
The simple setup will not work.:( So I must use complex one.
So here the next try.(I have correct the version number of the doc)
SPEC: http://downloads.sourceforge.net/qsmartcard/qtiplot-doc.spec?use_mirror=osdn

Comment 4 Frank Büttner 2007-09-30 09:53:06 UTC
The new spec file:

Comment 5 Till Maas 2008-06-09 21:34:25 UTC
I just realized, that I totally neglected this review request. I noticed
however, that there is now a new release of the manual. I just tried the new
version, the Sources needs to be adjusted and the dos2unix invokation does not
work. The upstream tarball contains .svn directories, which creates a huge
rpmlint warning. Imho they should be removed. 

Also the specfile contains both space and tabs, only after the Buildroot and the
Group tag are spaces, after the other tags are tabs. I would only use spaces,
but you need to choose one method to align the tags.

If you still want to get this package in, please reply with a new spec and I
will try very hard to complete this review soon after your reply.

Comment 6 Rakesh Pandit 2008-09-03 15:01:09 UTC
Any updates in here?
This request will be closed with in a week if no updates are provided.

Comment 7 Till Maas 2008-11-07 11:44:00 UTC
There was no response for a while and iirc I read somewhere that the submitter is currently very busy.

Therefore I close this for now, feel free to reopen this once you want to finish it.

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