search

gkralik / php7-sapnwrfc

SAP NW RFC SDK extension for PHP 7.3+ & PHP 8
https://gkralik.github.io/php7-sapnwrfc
MIT License
90 stars 35 forks source link

Windows shutdown error #6

Closed ThaDafinser closed 8 years ago

ThaDafinser commented 8 years ago

Now i get this error on shutdown.

Calling RFC functions before works fine.

This error also comes with the simple command php -m

image

gkralik commented 8 years ago

Which version did you use?

Can you tell what was the last version where you did not experience this error?

On 02/16/2016 10:28 AM, Martin Keckeis wrote:

Now i get this error on shutdown.

Calling RFC functions before works fine.

This error also comes with the simple command |php -m|

image https://cloud.githubusercontent.com/assets/533017/13072147/e6609ac0-d497-11e5-8f3c-717cd2cc010e.png

— Reply to this email directly or view it on GitHub https://github.com/gkralik/php7-sapnwrfc/issues/6.

ThaDafinser commented 8 years ago

Latest commit (master). Need to go through the commits, will inform you when i found the last working commit.

ThaDafinser commented 8 years ago

Needed to build a lot of times to get the commit :+1:

With this commit fd56ff26c12ef099e4c6a21aa39494432cba58af it stopped to work

https://github.com/gkralik/php7-sapnwrfc/commit/fd56ff26c12ef099e4c6a21aa39494432cba58af

;extension=php_sapnwrfc-c4ecab0276d2e8159e8138680c8571a3b50da935.dll    ; nope
;extension=php_sapnwrfc-20f4f437a2d0f5fd7074afbcf5df883882bc18ff.dll    ; WORK
;extension=php_sapnwrfc-afd90b6e667f89ad586caa193bda842c914aca98.dll    ; nope
;extension=php_sapnwrfc-58251b123861eddf76406120a8a909dc7c20e19c.dll    ; nope      
;extension=php_sapnwrfc-8d1f9276b84842ec7d53ece3fd1d0e75c12d2d8c.dll    ; WORK
;extension=php_sapnwrfc-88b3ba2319a94417f2cbf6184de427c1aa6a6a5d.dll    ; WORK
 extension=php_sapnwrfc-fd56ff26c12ef099e4c6a21aa39494432cba58af.dll  ; nope
gkralik commented 8 years ago

Ok, thanks. I'll have a look in the evening.. seems strange. Can you show me a stripped down version of the PHP script you use that I can use to try to reproduce the issue?

gkralik commented 8 years ago

@ThaDafinser can you try the fix-shutdown-error-on-windows branch and check if it makes a difference? Just a wild guess, but maybe it helps me narrow down the problem...

On 02/16/2016 12:21 PM, Martin Keckeis wrote:

Needed to build a lot of times to get the commit :+1:

With this commit |fd56ff26c12ef099e4c6a21aa39494432cba58af| it stopped to work

fd56ff2 https://github.com/gkralik/php7-sapnwrfc/commit/fd56ff26c12ef099e4c6a21aa39494432cba58af

|;extension=php_sapnwrfc-c4ecab0276d2e8159e8138680c8571a3b50da935.dll ; nope ;extension=php_sapnwrfc-20f4f437a2d0f5fd7074afbcf5df883882bc18ff.dll ; WORK ;extension=php_sapnwrfc-afd90b6e667f89ad586caa193bda842c914aca98.dll ; nope ;extension=php_sapnwrfc-58251b123861eddf76406120a8a909dc7c20e19c.dll ; nope ;extension=php_sapnwrfc-8d1f9276b84842ec7d53ece3fd1d0e75c12d2d8c.dll ; WORK ;extension=php_sapnwrfc-88b3ba2319a94417f2cbf6184de427c1aa6a6a5d.dll ; WORK extension=php_sapnwrfc-fd56ff26c12ef099e4c6a21aa39494432cba58af.dll ; nope |

— Reply to this email directly or view it on GitHub https://github.com/gkralik/php7-sapnwrfc/issues/6#issuecomment-184637482.

ThaDafinser commented 8 years ago

@gkralik that branch works. :smile:

Compiler shows me two warnings now image

gkralik commented 8 years ago

Very well. The please update the fixed branch and try again... I changed the handling of the errorInfo member. Curious if that works now.

On 02/16/2016 01:12 PM, Martin Keckeis wrote:

@gkralik https://github.com/gkralik that branch works.

Compiler shows me two warnings now image https://cloud.githubusercontent.com/assets/533017/13075912/e90404ee-d4ae-11e5-8e17-a2281a935c9d.png

— Reply to this email directly or view it on GitHub https://github.com/gkralik/php7-sapnwrfc/issues/6#issuecomment-184659617.

ThaDafinser commented 8 years ago

@gkralik still work with the 2nd commit in that branch. Two warnings still exist

Do i need to get a Exception? Did not tested that...

Just a RFC call + php -m

gkralik commented 8 years ago

OK, I'll fix the unused error_info error, but the signed/unsigned mismatch should already have been resolved in master...

On 02/16/2016 01:20 PM, Martin Keckeis wrote:

@gkralik https://github.com/gkralik still work with the 2nd commit in that branch. Two warnings still exist

— Reply to this email directly or view it on GitHub https://github.com/gkralik/php7-sapnwrfc/issues/6#issuecomment-184661476.

gkralik commented 8 years ago

Merged and fixed on current master. Thanks for your help.