Req #63362 [Opn->Wfx]: Not needed but installed headers.
| From: | petk@php.net | Date: | Wed, 13 Mar 2019 12:35:05 +0000 |
| Subject: | Req #63362 [Opn->Wfx]: Not needed but installed headers. | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-219942@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=63362&edit=1
ID: 63362
Updated by: petk@php.net
Reported by: remi@php.net
Summary: Not needed but installed headers.
-Status: Open
+Status: Wont fix
Type: Feature/Change Request
Package: *General Issues
Operating System: GNU/Linux (Fedora 18)
PHP Version: 5.4.8
Block user comment: N
Private report: N
New Comment:
According to the discussion on GitHub [1], the list of Windows specific headers is in the current
version (PHP 7.2+) reduced to 5 files. Also total size of these files are about 24KB which is very
small, even for very light Linux distributions such as Alpine and others. These additional files
really shouldn't cause any issues with the *nix environments out there, yet for possible future
other build options that might need these files it is a good step to leave them installable and not
omit them or remove them.
If there's any other solution here, let m, @krakjoe, or @kalle know about it. Closing this.
[1] https://github.com/php/php-src/pull/3937
Previous Comments:
------------------------------------------------------------------------
[2019-03-12 00:41:25] petk@php.net
The following pull request has been associated:
Patch Name: Fix #63362: Not needed but installed headers
On GitHub: https://github.com/php/php-src/pull/3937
Patch: https://github.com/php/php-src/pull/3937.patch
------------------------------------------------------------------------
[2016-08-06 01:22:32] kalle@php.net
Hi Remi
Now I'm no expert on the Unix build system, but it seems like a rather simple task and it does
make a lot of sense to do that. On Windows we don't install non Windows platform specific
headers with phpize either.
------------------------------------------------------------------------
[2012-10-25 13:43:46] remi@php.net
Description:
------------
Various Windows specific headers are provided in the php sources
(of course, this is absolutely not a problem).
It will be great to not install them on other OS.
Installed headers list:
TSRM/tsrm_win32.h
TSRM/tsrm_config.w32.h
Zend/zend_config.w32.h
ext/mysqlnd/config-win.h
ext/standard/winver.h
main/win32_internal_function_disabled.h
main/win95nt.h
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=63362&edit=1