Bug #69781 [Csd]: phpinfo() reports Professional Editions of Windows 7/8/8.1/10 as "Business"

Edit report at https://bugs.php.net/bug.php?id=69781&edit=1 ID: 69781 Updated by: ab@php.net Reported by: wenz@php.net Summary: phpinfo() reports Professional Editions of Windows 7/8/8.1/10 as "Business" Status: Closed Type: Bug Package: PHP options/info functions Operating System: Windows PHP Version: Irrelevant Assigned To: ab Block user comment: N Private report: N New Comment: @wenz, thanks for working on this. If you see something to do more - yep, just do it. I'd have been probably already drowned in all that product types :) With 0x0600 - we still can't drop the Vista support. I think you and me were not alone in willing that. But it's supported now, so has to stay. I'm going to close the #55319 then. Thanks. Previous Comments: ------------------------------------------------------------------------ [2015-06-10 10:10:10] wenz@php.net Related To: Bug #54020 ------------------------------------------------------------------------ [2015-06-10 10:09:18] wenz@php.net yes, this can be closed. Thanks for accepting the PR! If we would set _WIN32_WINNT to 0x0601, GetProductInfo would return correct results, but then PHP would not run under Vista any longer, that's why we need to workaround. Sorry for the confusion! :) I will still work on adding the remaining SKUs, probably next week. I have seen code that uses the phpinfo() output in weird ways, so the more accurate the details, the better ;-) ------------------------------------------------------------------------ [2015-06-10 09:40:18] ab@php.net @wenz can the related ticket be closed as well? With _WIN32_WINNT - what do you mean? It is already 0x0600 as it corresponds to the targeting Vista, so how we would loose it? See main/config.w32.h and makefile for WINVER in CFLAGS. Thanks. ------------------------------------------------------------------------ [2015-06-10 09:28:12] ab@php.net Automatic comment on behalf of christian@wenz.org Revision: http://git.php.net/?p=php-src.git;a=commit;h=741b949a6f7c3bc3e171622fe289ad2c0871f9cc Log: Fix #69781: phpinfo() reports Professional Editions of Windows 7/8/8.1/10 as "Business" ------------------------------------------------------------------------ [2015-06-10 09:28:10] ab@php.net Automatic comment on behalf of christian@wenz.org Revision: http://git.php.net/?p=php-src.git;a=commit;h=82e5260104021076402430b1fabc8af99860104c Log: Revert "Fix #69781: phpinfo() reports Professional Editions of Windows 7/8/8.1/10 as "Business"" ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=69781 -- Edit this bug report at https://bugs.php.net/bug.php?id=69781&edit=1