[Bug 204125] Review Request: tremulous-data - Data files for tremulous the FPS game
bugzilla at redhat.com
bugzilla at redhat.com
Tue Sep 5 19:09:28 UTC 2006
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: tremulous-data - Data files for tremulous the FPS game
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204125
------- Additional Comments From j.w.r.degoede at hhs.nl 2006-09-05 15:09 EST -------
(In reply to comment #6)
> - Spec file seems fine, three minor nitpicks :
> - The Source1 "tremulous-copyright.txt" gets installed as
> "fedora-copyright.txt", why two different names?
The name starting with tremulous is to make clear to which package it belongs
when the src rpm gets installed and the files dropped under /usr/src/redhat/SOURCES
Since under /usr/share/doc it already is in a tremulous dir the trmulous in the
name isn't needed and since the file is added to meet the fedora copyright
guidelines I call it fedora-copyright.txt .
> - Maybe you should uncomment the %build line, even though the section is
> empty, since IIRC not having %build has already caused weird things to
> happen.
I actually commented it because in another review of a similar data only package
the reviewer wanted it commented, so I'm leaving it as is as this seems to work
fine for the other package.
> - It's "basically", not "basicly".
Fixed
I'm importing it now (slow slow lookaside cache upload, at times like the the A
in ADSL sucks).
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.
More information about the Fedora-package-review
mailing list