compilation for php 5.5 on windows

[bazo]

Hi,

i'm trying to compile php with phpredis. i've been writing down the steps , here https://gist.github.com/bazo/5994044

nmake throws the following error:

NMAKE : fatal error U1073: don't know how to make 'ext\phpredis\igbinary\hash_fu
nction.c'

i tried both x64 and x86 comilation. then i've added igbinary extension from https://github.com/phadej/igbinary
it wouldn't compile either
then i've tried the compile flags from here http://char101.github.io/phpredis/
--enable-redis-igbinary is unknown option

php itself compiles just fine

this is my first time compiling php on windows, so i'm at a loss here.

any help is apreciated

[michael-grunder]

Hey there,

Neither @nicolasff or I run windows, so I'm not sure how much help we can be getting it to compile. I've toyed with the idea of setting up a windows box so I could help when this sort of thing comes up, but honestly, I'm not very good with the MS compiler stuff.

@char101 builds them from time to time and may be able to point you in the right direction.

Cheers,
Mike

[char101]

The config.w32 in phpredis upstream is obsolete. It now dynamically use existing igbinary extension. So what you should do is compile igbinary first. Then if you want to use the code from upstream, replace the config.w32 there with config.w32 from my repository. After that it's all about fixing compilation errors because VC compiler does not support C99 standard.

[michael-grunder]

@char101 Thanks for the tips. You're awesome! 😃

[alexpts]

Where can I download the compiled extension for php 5.5 x64?

In order to compile redis PHP extension PHP_REDIS_API should be used in many functions from library.c/h files.
f.e.
PHPAPI int redis_sock_connect(RedisSock *redis_sock TSRMLS_DC);
should be:
PHP_REDIS_API int redis_sock_connect(RedisSock *redis_sock TSRMLS_DC);

PHP_REDIS_API is already defined in php_redis.h.

config.w32 supports compile options: redis and redis-session while config.m4 supports also enable-redis-igbinary.

[f0ff886f]

There's a few other things in order to compile this using VC*, since files have a .c extension they need to be C89 compliant (right now there's lots of areas where vars are defined not at the beginning of a block statement), and a few other things.

I'm trying to fix this up and build a version for php5.5, but its very very slow going. It sounds like a lot of people would like to use phpredis in Windows (it helps a lot for quick development if we're not on a Linux platform and not deploying remotely).

[f0ff886f]

OK, I've got the threadsafe DLL building with the same config flags that @char101 uses on his page.

The only thing is that the non-threadsafe compilation fails due to: ZEND_DECLARE_MODULE_GLOBALS(redis), but I don't know enough about zend modules to fix that.

I have made my DLL available for test here: http://d-h.st/GDR

php_redis built from 00233a (latest commit as of right now), against php5.5, vc11, threadsafe, x64.

I will open a pull request with the new updates, credit goes to @char101 for his config.w32 and initial work

[f0ff886f]

Ah, and scratch that, I see that @char101 just wrapped the ZEND_DECLARE_MODULE_GLOBAL in an #ifdef ZTS, did that and the non-Threadsafe version compiled just fine.

I uploaded everything as separate DLLs and they are available here:

php_redis-5.5-vc11-ts-x64-00233a.zip http://d-h.st/1tO
php_igbinary-5.5-vc11-ts-x64-c35d48.zip http://d-h.st/rYb

php_redis-5.5-vc11-nts-x64-00233a.zip http://d-h.st/N0d
php_igbinary-5.5-vc11-nts-x64-c35d48.zip http://d-h.st/c1a

Cheers, pull request incoming.

[alexpts]

Thanks! )

[char101]

Thanks for your work, I hope your pull request gets merged so that phpredis can be compiled easily in windows in the future.

[michael-grunder]

Thanks for the pull request!

I'm going to get this running locally and make sure that everything still compiles on the linux end of things. I should have some time to do this over the weekend.

Cheers,
Mike

[Andrewpk]

@vostok4 if it's not too much trouble could you post 5.5 x86 editions as well? x64 is still experimental on windows. If not I'll try to compile tomorrow and post.

[f0ff886f]

@Andrewpk done. I also added one more change for the CREDITS file (somehow that missed the original commit) which is the last fix necessary, to the pull request.

php_redis-5.5-vc11-ts-x86-00233a.zip http://d-h.st/4A5
php_igbinary-5.5-vc11-ts-x86-c35d48.zip http://d-h.st/QGH

php_redis-5.5-vc11-nts-x86-00233a.zip http://d-h.st/uGS
php_igbinary-5.5-vc11-nts-x86-c35d48.zip http://d-h.st/bei

Cheers,
Matt

[Andrewpk]

Thanks a lot for compiling those!

On Oct 16, 2013, at 5:55 AM, Matt Rusiniak notifications@github.com wrote:

@Andrewpk done. I also added one more change for the CREDITS file (somehow that missed the original commit) which is the last fix necessary, to the pull request.

php_redis-5.5-vc11-ts-x86-00233a.zip http://d-h.st/4A5
php_igbinary-5.5-vc11-ts-x86-c35d48.zip http://d-h.st/QGH

php_redis-5.5-vc11-nts-x86-00233a.zip http://d-h.st/uGS
php_igbinary-5.5-vc11-nts-x86-c35d48.zip http://d-h.st/bei

Cheers,
Matt

—
Reply to this email directly or view it on GitHub.

[remicollet]

I think new release will be automatically build under windows and made available from official pecl web site, like for other ext, ex, see: http://pecl.php.net/package/xmldiff (the DLL link)