Spec URL: http://zanoni.jcomserv.net/fedora/solarwolf/solarwolf.spec SRPM URL: http://zanoni.jcomserv.net/fedora/solarwolf/solarwolf-1.5-1.fc7.src.rpm Description: The point of the game is to scramble through 60 levels collecting space boxes. Each level gets is harder than the previous. Obstacles like bullets, mines, and asteroids cover your every move. Beat the Skip timer and grab the powerups for your only chance.
Shall we swap reviews again? It would be nice if you could review sdljava, then I'm one step closer to getting bolzplatz approved (funny 3d footbal game, only works on Fedora 8 test).
Fully reviewed. Must Fix: --------- * Source0 gives a 404 error * Use install -p -m 755 for .py file to preserve timestamp Should fix: ----------- * Use install -p -m 644 for the data files, moving them away will break --shortcircuit builds * Once this is done the chmod's are no longer needed, instead put a "#nothing to build" line in %build
I fixed the Source0. If I change to install, it complains that it omits the data/audio and data/music directories, and kills the build. How do I get around that? That's why I used mv to begin with. Also, the chmods are to correct incorrect permissions in the source, not to specify them during the install of the package. Without the chmod, the %clean and %install sections do not have rights to rm -rf %{buildroot}.
(In reply to comment #3) > If I change to install, it complains that it omits the data/audio and data/music > directories, and kills the build. How do I get around that? That's why I used > mv to begin with. > Ah, my bad, I missed that there were subdirs, use "cp -a" then (cp to not erase the source files, -a to preserve timestamps) > Also, the chmods are to correct incorrect permissions in the source, not to > specify them during the install of the package. Without the chmod, the %clean > and %install sections do not have rights to rm -rf %{buildroot}. Ah, I see, put them in %prep then please
Addressed. Spec URL: http://zanoni.jcomserv.net/fedora/solarwolf/solarwolf.spec SRPM URL: http://zanoni.jcomserv.net/fedora/solarwolf/solarwolf-1.5-2.fc7.src.rpm
Approved!
Thank you! New Package CVS Request ======================= Package Name: solarwolf Short Description: A Python port of SolarFox Owners: limb Branches: FC-6 F-7 InitialCC: Cvsextras Commits: yes
You accidentally set the relase notes flag instead of the cvs one, fixing.
Thanks. modprobe caffiene
cvs done.
solarwolf-1.5-2.fc7 has been pushed to the Fedora 7 stable repository. If problems still persist, please make note of it in this bug report.