Bug 739263
Summary: | Review Request: sugar-bounce - Fast paced 3D action game | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Kalpa Welivitigoda <callkalpa> |
Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | brendan.jones.it, callkalpa, mario.blaettermann, notting, package-review, pbrobinson |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2013-11-08 19:34:21 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 201449 |
Description
Kalpa Welivitigoda
2011-09-17 02:13:41 UTC
I will take this on 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. Brendan Hi Brendan Would you please explain it in detail. btw, build needs in the INSTALL are all listed in the spec. 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. %prep %setup -q -n Bounce-%{version} sed -i -e 's|-fPIC -O2|%{optflags}|' pongc/Makefile %build # build C binaries pushd pongc make popd For reference: http://fedoraproject.org/wiki/Packaging:SugarActivityGuidelines#Architecture-specific_Activities 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) Yes, the BuildArch line can just be dropped. fixed the issues Spec URL: http://callkalpa.fedorapeople.org/sugar-bounce/sugar-bounce.spec SRPM URL: http://callkalpa.fedorapeople.org/sugar-bounce/sugar-bounce-7-2.fc15.src.rpm Sorry incorrect srpm link, here is the correct one Spec URL: http://callkalpa.fedorapeople.org/sugar-bounce/sugar-bounce.spec SRPM URL: http://callkalpa.fedorapeople.org/sugar-bounce/sugar-bounce-7-2.fc16.src.rpm 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 No progress for more than a year. Is this ticket still reviewable? Nothing has happened for almost two years. I close this ticket, adding FE-DEADREVIEW. |