| Summary: | Review Request: bolzplatz2006 - Slam Soccer 2006 is a funny football game in 3D-comic-style | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Hans de Goede <hdegoede> |
| Component: | Package Review | Assignee: | Marcela Mašláňová <mmaslano> |
| Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | fedora-package-review, mmaslano, notting |
| Target Milestone: | --- | Flags: | mmaslano:
fedora-review+
kevin: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2007-12-01 09:54:52 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
|
Description
Hans de Goede
2007-09-08 20:10:19 UTC
Unfortunately upstream couldn't get all data-files contributers to agree to a license change, so here is a new version using autodownloader instead :( Spec URL: http://people.atrpms.net/~hdegoede/bolzplatz2006.spec SRPM URL: http://people.atrpms.net/~hdegoede/bolzplatz2006-1.0.3-2.fc8.src.rpm I can't build it. There some problem with gcc option. Have you any advice? g++ -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m32 -march=i386 -mtune=generic -fasynchronous-unwind-tables -fPIC -fno-strict-aliasing -g -shared lib/irrlicht_wrap.o -o libirrlicht_wrap.so -L../irrlicht-0.14-patched/lib/Linux -lIrrlicht -ljpeg -lpng -lz -L/usr/X11R6/lib -lX11 -lGL -lXxf86vm -lXext -lc -lGLU /usr/bin/ld: cannot find -lXxf86vm collect2: ld returned 1 exit status make: *** [libirrlicht_wrap.so] Error 1 error: Bad exit status from /var/tmp/rpm-tmp.4599 (%build) rpmlint bolzplatz2006.src: W: strange-permission bolzplatz2006.sh 0755 Spec file looks ok. (In reply to comment #2) > I can't build it. There some problem with gcc option. Have you any advice? > g++ -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector > --param=ssp-buffer-size=4 -m32 -march=i386 -mtune=generic > -fasynchronous-unwind-tables -fPIC -fno-strict-aliasing -g -shared > lib/irrlicht_wrap.o -o libirrlicht_wrap.so -L../irrlicht-0.14-patched/lib/Linux > -lIrrlicht -ljpeg -lpng -lz -L/usr/X11R6/lib -lX11 -lGL -lXxf86vm -lXext -lc -lGLU > /usr/bin/ld: cannot find -lXxf86vm > collect2: ld returned 1 exit status > make: *** [libirrlicht_wrap.so] Error 1 > error: Bad exit status from /var/tmp/rpm-tmp.4599 (%build) > Hmm, missing BuildRequires on: libXxf86vm-devel can you install libXxf86vm-devel and give it another try please? Hm and what's now? BUILD FAILED /usr/src/redhat/BUILD/bolzplatz2006-1.0.3/libsrc/lwjgl/build.xml:564: The following error occurred while executing this line: /usr/src/redhat/BUILD/bolzplatz2006-1.0.3/libsrc/lwjgl/build.xml:617: Warning: Could not find file /usr/src/redhat/BUILD/bolzplatz2006-1.0.3/libsrc/lwjgl/src/native/linux/liblwjgl.so to copy. Did you build it in mock? (In reply to comment #4) > Did you build it in mock? Nope, I normally just get the BuildRequires from the configure script as I have a slow internet link at home, but that was a bad idea in this case. I've put it through mock now and added lots of missing BuildRequires, new version here: Spec URL: http://people.atrpms.net/~hdegoede/bolzplatz2006.spec SRPM URL: http://people.atrpms.net/~hdegoede/bolzplatz2006-1.0.3-3.fc9.src.rpm > bolzplatz2006.i386: W: file-not-utf8 > /usr/share/doc/bolzplatz2006-1.0.3/license.txt This should be fixed, because of our utf-8 policy. > bolzplatz2006.src: W: strange-permission bolzplatz2006.sh 0755 You should comment this unusual permission. SPEC: > export JAVA_HOME=/usr/lib/jvm/java-icedtea use %_libdir instead of path (In reply to comment #6) > > bolzplatz2006.i386: W: file-not-utf8 > > /usr/share/doc/bolzplatz2006-1.0.3/license.txt > This should be fixed, because of our utf-8 policy. > Oops, will fix, I missed this as when I first packaged this rpmlint didn't do UTF-8 checks for some reason. > > bolzplatz2006.src: W: strange-permission bolzplatz2006.sh 0755 > You should comment this unusual permission. > It doesn't need to be 755 in the src.rpm install will fix the permissions when installing, I'll fix this. > SPEC: > > export JAVA_HOME=/usr/lib/jvm/java-icedtea > use %_libdir instead of path > Can't do java is always under /usr/lib even on x86_64. I'll fix the 2 other minor issues once a full review is done, thanks for the input sofar. Now I have only one problem, the game isn't running for me. Maybe that's because of my two lcd's. Hmm, Thats strange, it worked fine for me last time I tried it, you are launching it through the shell script, and it does first download the needed datafiles? What error do you get? At the first time some data start downloading and then nothing. I try to run it only on laptop without another lcd and change settings, but nothing happened. There's no response from programme only try to change screen and then nothing. If it's working for someone else, then it is ok. I'll give you +, because everything from the review point has been done. Thanks for the review and the trust! I'll make sure to double check it still works for me on both i386/i810 video and x86_64/radeon video. It worked fine there when I initially packaged it, but maybe something else has changed causing problems. New Package CVS Request ======================= Package Name: bolzplatz2006 Short Description: Slam Soccer 2006 is a funny football game in 3D-comic-style Owners: jwrdegoede Branches: F-8 devel InitialCC: <empty> Cvsextras Commits: Yes cvs done. Tested on a pristine x86_64 install with ati video, works as advertised for me. Imported and build, closing. |