[Bug 172521] Review Request: TeX4ht - Translates TeX and LaTeX into HTML or XML+MathML

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: TeX4ht - Translates TeX and LaTeX into HTML or XML+MathML


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=172521





------- Additional Comments From mpeters at mac.com  2005-11-13 18:55 EST -------
Good:
* uses %{?dist} tag
* md5sum matches upstream for Source
* md5sum matches upstream for Source1
* rpmlint *mostly* likes it
* mock builds it (i386 tested)
* installs and uninstalls cleanly (Tested on FC4 i386) - works as expected (FC4
i386)

-=-
Suggestion:
* rename Source to Source0
* add a datestamp to tex4ht-all.zip - it will probably change in future updates.
  (ie tex4ht-all-YYYYMMDD.zip - based on last timestamp in directory)

The way that is typically done is something like:
Source1: tex4ht-all-YYYYMMDD.zip
# unversioned upstream source
# http://www.cse.ohio-state.edu/~gurari/TeX4ht/tex4ht-all.zip

-=-
Question - upstream version has an underscore in the version name, corresponding
to the packaging date. Fedora Extras tetex packages that use a date in the
version string typically don't.

Should we change it - or leave it as upstream? I would be inclined to just leave
it be the way upstream is currently doing it. If there are potential problems
with how rpm evaluates the version with underscores present (I don't know) then
it should be changed.

-=-
Bad:
* Timestamp of source in src.rpm does not match rpm - grab sources with wget -N
to preserve timestamp on upstream tarball
* rpmlint - complains about license
Change it to
License:        LaTeX Project Public License

(drop the '(LPPL)' and change 'Latex to LaTeX')
and rpmlint will shut up.

-=-
Don't Know (probably harmless):
rpmlint gives the following errors on debuginfo:
E: tetex-tex4ht-debuginfo script-without-shellbang
/usr/src/debug/tex4ht-1.0.2005_11_06_1516/src/t4ht.c
E: tetex-tex4ht-debuginfo script-without-shellbang
/usr/src/debug/tex4ht-1.0.2005_11_06_1516/src/tex4ht.c


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.