Re: [Chicken-users] New egg: physics

[Mario Domenech Goulart]

Hi,

On Thu, 15 May 2014 23:14:35 +0200 Richard <address@hidden> wrote:

> I would like to announce my first egg: physics. It is a more
> high-level wrapper to the brilliant Chipmunk2D physics library.
>
> Some highlights include:
>
> - The possibility of passing regular Chicken functions as callbacks.
> - Using regular Chicken objects as user data.
> - Garbage collection.
> - The possibility of optionally using 'map' instead of Chipmunk's
> iterator functions.
>
> Things left to do are:
> - More testing and adding some unit tests.
> - Finishing the documentation and adding some examples.
>
> If you want to take a look, the source can be found at:
> https://github.com/pluizer/chicken-physics
>
> A more low-level wrapper to Chipmunk is required and can be found at:
> https://github.com/pluizer/chicken-chipmunk
>
> The documentation can be found at:
> https://wiki.call-cc.org/eggref/4/physics
>
> As this is my first egg I might have done things wrongly or
> suboptimal, any pointers are welcome.

Very nice.  Thanks for contributing to the CHICKEN coop.  It's actually
two eggs!

Here are a couple of suggestions to improve your eggs:

- you probably don't need the .import.scm files in the code
  repository. csc generates them

- instead of relying on an external make, you can use the make egg
  (http://wiki.call-cc.org/egg/make -- just mentioning in case you don't
  know it).  If you prefer using the external make, that is fine,
  although it may be inconvenient and will probably require some
  tweaking.  For example, if I want to use a chicken installation whose
  "bin" dir is not in PATH, I guess make will invoke the wrong csc.

- the .meta files should contain at least category and license

- physics' .meta should declare chipmunk as dependency

- a very hardcore nitpicking: source files usually don't need the
  execute flag. :-)

Please, consider fixing the .meta files at least before we add them to
the official coop.

Best wishes.
Mario
-- 
http://parenteses.org/mario