Ticket #247 (closed enhancement: fixed)

Opened 1 year ago

Last modified 5 months ago

Win32 support for 0.9.6

Reported by: Daan Assigned to: somebody
Priority: blocker Milestone: 0.9.6
Component: eAccelerator Version:
Keywords: Cc:

Description

While tinkering on my windows box with XAMPP I ran into a bug which had been fixed in SVN, so I went out to grab the source to compile a new DLL, only to find out that Windows support was broken, ach wee. But what is open source if you don't try fix it yourself, eh?

So here is a patch will bring the trunk back into the Windows support zone:
http://home.parse.nl/~daan/eAccelerator/win32_for_0.9.6-dev.patch

It fixes 3 things which made VC++ 6.0 hickup, nl:

  • No fnmatch for windows: fixed by adding fnmatch.c & fnmatch.h from a FreeBSD project
  • Using php_strtok_r() instead of strtok_r()
  • Casting of var 'len' to (int) in decode_version() to get rid of an VC++ 6 warning

Change History

03/18/07 10:06:21 changed by bart

  • priority changed from minor to blocker.
  • milestone set to 0.9.6.

Thanks, that was the plan. At the moment I'm very busy with other stuff, but this patch will certainly be included!

05/10/07 00:32:04 changed by bart

These patches have been commit in revisions [306], [307] and [308]

05/10/07 00:42:26 changed by bart

  • status changed from new to closed.
  • resolution set to fixed.