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