Skip to content
This repository was archived by the owner on Jan 31, 2023. It is now read-only.

Conversation

@vjardin
Copy link

@vjardin vjardin commented Jan 3, 2019

Another benefit of this nice framework is to be able to generate
and then to compile any standalone PHP extension (module).

It is mostly thanks to the phpize.bat tool that has been made
available.

TBC: should we add ext_skel.php and its template into the
deliverable of this SDK (c:\php\SDK\TBD) ?

Another benefit of this nice framework is to be able to generate
and then to compile any standalone PHP extension (module).

It is mostly thanks to the phpize.bat tool that has been made
available.

TBC: should we add ext_skel.php and its template into the
deliverable of this SDK (c:\php\SDK\TBD) ?
@msftclas
Copy link

msftclas commented Jan 3, 2019

CLA assistant check
All CLA requirements met.

@vjardin
Copy link
Author

vjardin commented Jan 8, 2019

Please, what is the process to get the integration of this pull request ?

@pierrejoye
Copy link

pierrejoye commented Jan 8, 2019 via email

@weltling
Copy link
Member

@vjardin thanks for the PR, I just came to review it. While it might work the way you do it, it was never intended. The way it should be done involves the usage of the development packages provided with the https://windows.php.net/downloads/releases/. It should not be necessary to compile the PHP core for that, that's the basic principle. To generate an extension one currently indeed needs the core to be checked out. Otherwise, phpize say on Linux never require to have the whole php-src. Of course, on a more advanced level it could be done, say by using a custom php installed into a prefix. However that's not the basic case that is concerned.

Also I think it's great that you go for making a sample extension and documenting some ways to compile it. However IMO this effort should go into the php.net documentation and not be spread over random resources on GitHub or elsewhere. Also with the regard to the binary SDK - it only provides a toolset, the actual documentation for building PHP on Windows is located under https://wiki.php.net/internals/windows. That's where the efforts should go.

Thanks.

@weltling weltling closed this Jan 12, 2019
@vjardin
Copy link
Author

vjardin commented Jan 12, 2019

Thanks for all the advises since I am quite new with the community. I'll review them carefully and I'll check how to apply them.

@vjardin
Copy link
Author

vjardin commented Jan 12, 2019

Regarding an out of tree compilation of the extensions, I feel that https://wiki.php.net/internals/windows/stepbystepbuild_sdk_2 does not provide the complete solutions. Can I add phpize tips on this page ?

@weltling
Copy link
Member

@vjardin contributions in this area are actually quite badly wanted. Specifically the documentation wrt. Windows build and ecosystem. These efforts are always welcome. FYI there are also specific mailing lists where things can be discussed, for Windows there's also #winphp-dev channel on freenode. Tthanks!

@weltling
Copy link
Member

@vjardin oh, and regarding your second comment - perhaps it would make sense to create a new page for that. The stepbystep one is about the core build, so that's fine as it is. Please put some draft to discuss on the Windows mailing lists first, so more people are involved.

Thanks.

@vjardin
Copy link
Author

vjardin commented Jan 12, 2019

Got it.
But which mailing list https://pecl.php.net/support.php#lists ?

@weltling
Copy link
Member

There are some here http://php.net/mailing-lists.php, but i think internals-win@ would be appropriate in first place.

Thanks.

@weltling
Copy link
Member

@vjardin perhaps more to illustrate here https://github.com/krakjoe/sandbox/blob/develop/.appveyor.yml.

Thanks.

vjardin added a commit to vjardin/php-bonjour that referenced this pull request Jan 13, 2019
Microsoft maintainer did some reviews and some comments. He provided
a good tip so that we can avoid recompiling PHP on Windows. The
PHP devel framework is already packaged too. Let's use it.

See: microsoft/php-sdk-binary-tools#48 (comment)

Currently, it is WIP, it is not yet properly tested
vjardin added a commit to vjardin/php-bonjour that referenced this pull request Jan 22, 2019
Following the previous commit and the advices from Microsoft,
I have provided a complete section for Microsoft Windows.
Now, there are 2 build options: along with php source code or
standalone with with the PHP develpment environment.

You should read:
  microsoft/php-sdk-binary-tools#48 (comment)

if you want to understand the rationals.

Down the road, I hope it can become a reference for:
   https://wiki.php.net/internals/windows

TODO: extend this tutorial and illustrate it using appveyor CI.
XXX: why are bison, sed, re2c required knowing they are not used ?!
@vjardin
Copy link
Author

vjardin commented Jan 22, 2019

@weltling Anatol: I'll update the PHP wiki, but first I need to make sure that I get it right. Per your advise, I was able to describe how to build any PHP module without recompiling PHP source code too. I made an update to my notes here, but I need to hack the configure.js in order to avoid the checks for re2c, bison, etc. which are not mandatory in fact. Is there a better way ?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants