No *.def file for Visual Studio

[GoogleCodeExporter]

Without it functions won't export. Also for Release.DLL/Debug.DLL 
configurations doesn't set Configuration Type to "Dynamic Library"(at least for 
Visual Studio 2008).

I built def file, so I hope it will be included as patch, or you will build 
your own :)

Original issue reported on code.google.com by dr.andru@gmail.com on 6 Dec 2010 at 6:41

[GoogleCodeExporter]

Seems I forgot some main functions.

Original comment by dr.andru@gmail.com on 6 Dec 2010 at 8:41

[GoogleCodeExporter]

Original comment by dr.andru@gmail.com on 6 Dec 2010 at 2:17

Attachments:

• chipmunk.def

[GoogleCodeExporter]

Hey thanks. Could you possibly make a patch for that? I don't actually know how 
to use MSVC myself. I only have the express version on my netbook so I can 
check that it still builds correctly every once in a while.

Original comment by slemb...@gmail.com on 7 Dec 2010 at 2:05

[GoogleCodeExporter]

Here a patch for the trunk version of Chipmunk, made by TurtoiseSVN(I'm not 
home right now, and can't make it by diff in my ArchLinux... so, maybe it must 
be modified by removing few lines with "Index: file"). Maybe little bit later I 
will make it for Visual Studio 2010. This patch also contains fix for M_PI(for 
use this constant with Visual Studio you must define _USE_MATH_DEFINES in 
chipmunk.c)

Original comment by dr.andru@gmail.com on 7 Dec 2010 at 6:49

Attachments:

• patch.diff

[GoogleCodeExporter]

Sorry for so many messages :) Here the patch with Visual Studio 2010 "fixes", 
and with all modifications from patch above.

Original comment by dr.andru@gmail.com on 7 Dec 2010 at 7:36

Attachments:

• patch2.diff

[GoogleCodeExporter]

/me Faceplam

Crap, I totally forgot about this. I just uploaded 5.3.3 a couple of hours ago. 
I guess I'll just have to sneak in a replacement archive.

Original comment by slemb...@gmail.com on 7 Dec 2010 at 8:45

[GoogleCodeExporter]

I couldn't get the patch to apply beyond generating the .def files. I wasn't 
able to figure out how to fix the merge by hand. I tried taking a second look 
at it today, but didn't have any luck. If you can send me the modified files I 
can just copy them in the old fashioned way. Sorry to bother you about this 
again. I do appreciate the help.

Original comment by slemb...@gmail.com on 9 Dec 2010 at 6:44

[GoogleCodeExporter]

Strange, but I got the same problem at home. Seems TurtoiseSVN generated wrong 
diff... Ok, tomorrow I will take my laptop to work(where I can use Visual 
Studio :)) and will try to fix this.

Original comment by dr.andru@gmail.com on 9 Dec 2010 at 8:59

[GoogleCodeExporter]

So, seems TurtoiseSVN really create a crap instead of normal patch... I 
attached normal patch that can be applied in Linux :) Also I attached project 
files. And I found another small compilation problem - cpSpace.c, line 487:
cpBody *root = components->arr[i];
correct is:
cpBody *root = (cpBody*)components->arr[i];

Original comment by dr.andru@gmail.com on 10 Dec 2010 at 7:25

Attachments:

• patch.diff
• msvc.7z

[GoogleCodeExporter]

Thanks. I committed the changes to trunk.

Original comment by slemb...@gmail.com on 11 Dec 2010 at 5:51

• Changed state: Fixed