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

Reminder: Builds for windows #36

Closed ThaDafinser closed 7 years ago

ThaDafinser commented 7 years ago

Just a reminder for me :+1:

ThaDafinser commented 7 years ago

Build are available here for 7.0 https://github.com/ThaDafinser/php7-sapnwrfc/pull/1

@gkralik could you already build your extension for 7.1 on unix? I can't build it on windows currently.

gkralik commented 7 years ago

👍 Yes, I built for 7.1, did not notice any problems. Do you get build errors?

gkralik commented 7 years ago

Just confirmed, builds fine for 7.0 and 7.1 under Linux.

ThaDafinser commented 7 years ago

I used the new guide + the new sdk-binary-tools (which worked nice for this php 7.0 build) https://wiki.php.net/internals/windows/stepbystepbuild_sdk_2

I would say no ...using php 7.1.3 (and the 7.1. deps ...)

Do you changed anything SAP related to build for 7.1?

configure --disable-all --enable-cli --disable-zts --with-sapnwrfc=shared

log1.txt

nmake

log2.txt

nmake snap

log3.txt

image

ThaDafinser commented 7 years ago

BTW this warning also pops up in PHP 7.0 WARNING: distro depends on sapnwrfc.dll, but could not find it on your system

gkralik commented 7 years ago

Hmm, strange. I did not change anything. Seems we need to have a more detailed look at this issue...

ThaDafinser commented 7 years ago

building is fine :see_no_evil: : failed to properly configure PHP 7.1 in my local dev enviroment...(add sap .dll to php root)

added the build for 7.1 now to the PR / branch https://github.com/ThaDafinser/php7-sapnwrfc/tree/build-1.1.2 https://github.com/ThaDafinser/php7-sapnwrfc/pull/1/files

ThaDafinser commented 7 years ago

Note: VC15 is a typo (it's VC14)

gkralik commented 7 years ago

Thanks, I have added them to the release page 👍