Use SourceRename to give the downloaded zip file a meaningful name (%n-%v.zip).
gcc4.9 compiler is not being used, and it builds on 10.9 with system-clang. Maybe gcc49 is not needed.
-- The C compiler identification is AppleClang 6.0.0.6000054
-- The CXX compiler identification is AppleClang 6.0.0.6000054
Add -MD to CPPFLAGS and CXXFLAGS, then use fink-package-precedence to find what headers are actually used (the dep file extension is .d). And use otool -L on the installed files to verify what packages are actually linked to. We don't want to be adding extra packages to the *Depends fields that are not actually being used.
Use %p and not /sw in DescDetail.
There's no Homepage field.
There's a newer release from upstream. Is there a reason to not use it?
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
You still have sdl-ttf-shlibs and sdl-mixer-shlibs listed in the Depends: field
fink-package-precedence is finding (and not recognizing) the internal buildtime headers. To make f-p-p ignore these, run it from the root srcdir, and not the builddir.
The executable crashes with Segmentation fault: 11 and multiple cannot open rom: epr-10381a.132 alerts, which I'm assuming occur because I don't have the propietary ROM. Is there a way to test the pkg without this ROM?
Don't erase older versions of .info/.patch files from the tracker. That makes it impossible to see what changed between uploads.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Use
SourceRename
to give the downloaded zip file a meaningful name (%n-%v.zip).gcc4.9 compiler is not being used, and it builds on 10.9 with system-clang. Maybe gcc49 is not needed.
Add -MD to CPPFLAGS and CXXFLAGS, then use fink-package-precedence to find what headers are actually used (the dep file extension is .d). And use
otool -L
on the installed files to verify what packages are actually linked to. We don't want to be adding extra packages to the *Depends fields that are not actually being used.Use %p and not /sw in DescDetail.
Thank you again Hanspeter
1) I removed the gcc4.9 and set GCC: 4.0. Looks good
2) Adding the fink-package-precendence and check with the otool -L.
I updated the BuildDependencies and Dependencies fields
3) Added %p in the DescDetail
4) Added the Homepagefield
5) I'm using the latest "Tags" from git, not from the branch.
Did I miss something or is there a better practice ?
Last edit: __||__||__ 2015-01-09
This says that 0.3 is the latest release: https://github.com/djyt/cannonball/tags
If it's not using sdl-ttf or sdl-mixer, don't include them 'just in case'. Determine which is it.
It's linking to %p/lib/libfreeglut.dylib. You probably want to make sure it's linking to the GLUT.framework since our freeglut is X11 based.
Yes, you're right. Updated to 0.3
sdl-ttf or sdl-mixer aren't needed at all
It was linking to the /sw or /opt freeglut. I changed the patch with -framework hardcoded and it's working properly
Segmentation fault: 11
and multiplecannot open rom: epr-10381a.132
alerts, which I'm assuming occur because I don't have the propietary ROM. Is there a way to test the pkg without this ROM?