Win32 compile fixes #321

Closed
wants to merge 7 commits into
from

Conversation

Projects
None yet
5 participants
@char101
Contributor

char101 commented Mar 28, 2013

No description provided.

@char101

This comment has been minimized.

Show comment Hide comment
@char101

char101 Mar 28, 2013

Contributor

I have put latest binaries for PHP 5.4 here

http://char101.github.com/phpredis/

Contributor

char101 commented Mar 28, 2013

I have put latest binaries for PHP 5.4 here

http://char101.github.com/phpredis/

@michael-grunder

This comment has been minimized.

Show comment Hide comment
@michael-grunder

michael-grunder Mar 28, 2013

Member

@char101 Very cool dude. We might want to add that page to the documentation . :)

Member

michael-grunder commented Mar 28, 2013

@char101 Very cool dude. We might want to add that page to the documentation . :)

@char101

This comment has been minimized.

Show comment Hide comment
@char101

char101 Mar 29, 2013

Owner

Using < and > causes the compile command to fail in windows console since they are probably treated as redirection operators.

Owner

char101 commented on CREDITS in 22ecea0 Mar 29, 2013

Using < and > causes the compile command to fail in windows console since they are probably treated as redirection operators.

@char101

This comment has been minimized.

Show comment Hide comment
@char101

char101 Mar 29, 2013

Owner

PHPAPI should be expanded to dllexport in redis sources but it needs to be expanded as dllimport in basic_functions.h, so there is a conflicting use of the same macro.

Owner

char101 commented on library.c in 22ecea0 Mar 29, 2013

PHPAPI should be expanded to dllexport in redis sources but it needs to be expanded as dllimport in basic_functions.h, so there is a conflicting use of the same macro.

@char101

This comment has been minimized.

Show comment Hide comment
@char101

char101 Mar 29, 2013

Owner

This macro is not used.

Owner

char101 commented on php_redis.h in 22ecea0 Mar 29, 2013

This macro is not used.

@radhikagiri

This comment has been minimized.

Show comment Hide comment
@radhikagiri

radhikagiri Aug 5, 2013

Hi,

I'm using WAMP stack and trying to load

PHP 5.4
php_redis-5.4-vc9-ts-7c52f3.dll
php_igbinary-5.4-vs9-ts-c35d48.dll.

My php.ini looks like this

extension=php_igbinary-5.4-vs9-ts-c35d48.dll
extension=php_redis-5.4-vc9-ts-7c52f3.dll

I've copied these two dll to php/ext directory. When I restart my WAMP server, I'm getting PHP warning

[04-Aug-2013 14:15:52 UTC] PHP Warning: PHP Startup: Unable to load dynamic library 'F:/wamp/bin/php/php5.4.12/ext/php_igbinary-5.4-vs9-ts-c35d48.dll' - %1 is not a valid Win32 application.

in Unknown on line 0

[04-Aug-2013 14:15:53 UTC] PHP Warning: PHP Startup: Unable to load dynamic library 'F:/wamp/bin/php/php5.4.12/ext/php_redis-5.4-vc9-ts-7c52f3.dll' - %1 is not a valid Win32 application.

in Unknown on line 0

Please help me resolve this.

Hi,

I'm using WAMP stack and trying to load

PHP 5.4
php_redis-5.4-vc9-ts-7c52f3.dll
php_igbinary-5.4-vs9-ts-c35d48.dll.

My php.ini looks like this

extension=php_igbinary-5.4-vs9-ts-c35d48.dll
extension=php_redis-5.4-vc9-ts-7c52f3.dll

I've copied these two dll to php/ext directory. When I restart my WAMP server, I'm getting PHP warning

[04-Aug-2013 14:15:52 UTC] PHP Warning: PHP Startup: Unable to load dynamic library 'F:/wamp/bin/php/php5.4.12/ext/php_igbinary-5.4-vs9-ts-c35d48.dll' - %1 is not a valid Win32 application.

in Unknown on line 0

[04-Aug-2013 14:15:53 UTC] PHP Warning: PHP Startup: Unable to load dynamic library 'F:/wamp/bin/php/php5.4.12/ext/php_redis-5.4-vc9-ts-7c52f3.dll' - %1 is not a valid Win32 application.

in Unknown on line 0

Please help me resolve this.

@yanxhyxz

This comment has been minimized.

Show comment Hide comment
@yanxhyxz

yanxhyxz Dec 1, 2013

Is there a 'php-redis.dll' for php 5.5.6 vc11 nts & redis 2.6 on windows?

yanxhyxz commented Dec 1, 2013

Is there a 'php-redis.dll' for php 5.5.6 vc11 nts & redis 2.6 on windows?

@yatsukhnenko

This comment has been minimized.

Show comment Hide comment
@yatsukhnenko

yatsukhnenko Mar 5, 2017

Member

This PR is to old. It is impossible to review ~9K lines patch

Member

yatsukhnenko commented Mar 5, 2017

This PR is to old. It is impossible to review ~9K lines patch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment