Req #63362 [Opn->Wfx]: Not needed but installed headers.

From: 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

« previous php.bugs (#219942) next »