Menu

#4450 [New] [Game] Cannonball

Added_to_Fink
closed-accepted
5
2015-05-17
2015-01-06
__||__||__
No
  • Build as -m --build-as-nobody on Yosemite
  • I'm not shure sdl-mixer / sdl-ttf are required but I added them just in case
  • Maybe the dependency to gcc49 can be skipped too ?
  • Tested, it's running !
  • I also added a desktop entry for gnome

Discussion

  • Hanspeter Niederstrasser

    • 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?
     
  • Hanspeter Niederstrasser

    • labels: game --> New Package, games
    • Group: Undergoing_Validation --> Awaiting_Update_from_Submitter
     
  • __||__||__

    __||__||__ - 2015-01-09

    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
  • __||__||__

    __||__||__ - 2015-01-09
    • Group: Awaiting_Update_from_Submitter --> Undergoing_Validation
     
  • Hanspeter Niederstrasser

    • 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.

     
  • Hanspeter Niederstrasser

    • Group: Undergoing_Validation --> Awaiting_Update_from_Submitter
     
  • __||__||__

    __||__||__ - 2015-02-02
    • 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

     
  • Hanspeter Niederstrasser

     
  • Hanspeter Niederstrasser

    1. You still have sdl-ttf-shlibs and sdl-mixer-shlibs listed in the Depends: field
    2. 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.
    3. 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?
    4. Don't erase older versions of .info/.patch files from the tracker. That makes it impossible to see what changed between uploads.
     
  • __||__||__

    __||__||__ - 2015-05-10
    1. Thank you, fixed
    2. Updated
    3. There are no way to test without the roms. I added more informations in the Description about how to launch the game
    4. Understood, I'll do it from now
     
  • __||__||__

    __||__||__ - 2015-05-10
    • assigned_to: Hanspeter Niederstrasser
    • Group: Awaiting_Update_from_Submitter --> Undergoing_Validation
     
  • Hanspeter Niederstrasser

    • status: open --> closed-accepted
    • Group: Undergoing_Validation --> Added_to_Fink
     

Log in to post a comment.