[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 02:16 EST -------
(In reply to comment #4)
> Preliminary comments :
> 
> This notice seems like a leftover from a copy/paste of the main spec file :
> # this is basicly
> # http://dl.sourceforge.net/tremulous/tremulous-%{version}.zip
> # repackaged with tremulous-%{version}-src.tar.gz  removed as that contains
> # non Free software (the lcc compiler).
> 

Nope, in tremolous.spec it says:
# this is %{name}-%{version}-src.tar.gz as containted in:
# http://dl.sourceforge.net/%{name}/%{name}-%{version}.zip
# with the src/tools/lcc dir removed as that contains non Free software

Notice the subtile difference, which makes each explain prettty exact what the
tarbal is and how it was created.

> - Why do you hardcode the name and more importantly the version in the Source0
> line? This can lead to the typical case where you increase the "Verrsion:" line
> but forget the Source0 one and the resulting package seems to be the new version
> when it's still the old ;-) Some people prefer like that, but in the main
> tremulous package you use %{version}.

I dunno why I did that, fixed.

> - The cp %{SOURCE1} fedora-copyright.txt should be replaced by "install -p -m
> 0644 %{SOURCE1} fedora-copyright.txt" since a wrong umask could lead to a world
> writeable file, like it has already happened in the Extras build system...
Fixed

> - I'd suggest removing the %{?dist} from the release since this data can and
> should be shared across all releases (i.e. packages copied as-is for FC-5 and
FC-6).
> - The %description should end with a dot :-)

Good point, done.

Here is a new spec file, I didn't upload a new SRPM as that is huge:
http://people.atrpms.net/~hdegoede/tremulous-data.spec


-- 
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]