Bug 739263 - Review Request: sugar-bounce - Fast paced 3D action game
Summary: Review Request: sugar-bounce - Fast paced 3D action game
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2011-09-17 02:13 UTC by Kalpa Welivitigoda
Modified: 2013-11-08 19:34 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2013-11-08 19:34:21 UTC
Type: ---

Attachments (Terms of Use)

Description Kalpa Welivitigoda 2011-09-17 02:13:41 UTC
Spec URL: http://callkalpa.fedorapeople.org/sugar-bounce/sugar-bounce.spec
SRPM URL: http://callkalpa.fedorapeople.org/sugar-bounce/sugar-bounce-7-1.fc15.src.rpm

Hi I just finished packaging sugar-bounce. I highly appreciate a review.

sugar-bounce is an activity for sugar learning environment. sugar-bounce is a game which is analogous to the arcade game Pong, however it takes 
place within a three dimensional box with physical effects such 
as gravity. Additionally, bounce features an editor which allows 
children to create their own version of the game.

Comment 1 Brendan Jones 2011-11-11 06:26:08 UTC
I will take this on

Comment 2 Brendan Jones 2011-11-11 06:49:51 UTC
Hi Kalpa

this is not a noarch package. You are missing a build step to compile the c++ library _pong.so. Have a look in the INSTALL document of the source package.


Comment 3 Kalpa Welivitigoda 2011-11-11 14:34:51 UTC
Hi Brendan

Would you please explain it in detail. btw, build needs in the INSTALL are all listed in the spec.

Comment 4 Brendan Jones 2011-11-11 16:38:14 UTC
Your package contains a C library which must be compiled in the %build section and installed in the appropriate location. Being compiled that means the package is architecture specific - so you have to remove the BuildArch tag.

You will be doing something along the lines of this in your build and prep section and make sure the files are installed in the correct place.

%setup -q -n Bounce-%{version}
sed -i -e 's|-fPIC -O2|%{optflags}|' pongc/Makefile

# build C binaries
pushd pongc

For reference: http://fedoraproject.org/wiki/Packaging:SugarActivityGuidelines#Architecture-specific_Activities

Comment 5 Brendan Jones 2011-12-15 17:44:26 UTC
I would think that you can simply leave the BuildArch out and make sure that the arch specific stuff goes in the correct %{_libdir}/sugar/activities rather than %{sugaractivitydir}.
 %{_libdir} will change to lib/lib64 depending on the arch the rpm is building. You can test it out in mock: 
mock -r fedora-16-i386 <src.rpm> 
mock -r fedora-16-x86_64 <src.rpm> 

I'm new to Sugar so I would suggest contacting one of the other Sugar maintainers on how to package arch specific Sugar Activities. (http://fedoraproject.org/wiki/Sugar_Activities)

Comment 6 Peter Robinson 2011-12-16 08:57:53 UTC
Yes, the BuildArch line can just be dropped.

Comment 8 Kalpa Welivitigoda 2012-03-18 12:31:43 UTC
Sorry incorrect srpm link, here is the correct one

Spec URL: http://callkalpa.fedorapeople.org/sugar-bounce/sugar-bounce.spec

Comment 9 Brendan Jones 2012-03-25 20:03:19 UTC
Hi Kalpa,

still a few issues here. 

- When I run the application I'm getting the following:

fedora16:~ $ python /usr/share/sugar/activities/Bounce.activity/bounce.pyc
Traceback (most recent call last):
  File "/usr/share/sugar/activities/Bounce.activity/bounce.py", line 28, in <module>
    from pongc.pongc import *
ImportError: No module named pongc.pongc

pongc.py is missing from your package

- You also are using %buildroot and $RPM_BUILDROOT
- defattr(....) present in %files section. This is OK if packaging for EPEL5. Otherwise not needed
- remove rm -rf $RPM_BUILD_ROOT in your %install section

Comment 10 Mario Blättermann 2013-06-03 17:49:13 UTC
No progress for more than a year. Is this ticket still reviewable?

Comment 11 Mario Blättermann 2013-11-01 08:57:41 UTC
Nothing has happened for almost two years. I close this ticket, adding FE-DEADREVIEW.

Note You need to log in before you can comment on or make changes to this bug.