[Bug 204125] Review Request: tremulous-data - Data files for tremulous the FPS game

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxx  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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]