Skip to content

PHPC-1503: Fix config.w32 for PHP 7.4.0 #1067

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

cmb69
Copy link
Contributor

@cmb69 cmb69 commented Dec 4, 2019

The VS version evaluation has been reworked[1] for PHP 7.4.0, which
includes the removal of the VC_VERSIONS array. To avoid the JScript
error, we check whether VC_VERSIONS is defined, and otherwise fall back
to COMPILER_NAME_LONG.

[1] http://git.php.net/?p=php-src.git;a=commit;h=6a624c1dfda0dbfaaff3e453e6cb58de12748fb3

The VS version evaluation has been reworked[1] for PHP 7.4.0, which
includes the removal of the VC_VERSIONS array.  To avoid the JScript
error, we check whether VC_VERSIONS is defined, and otherwise fall back
to COMPILER_NAME_LONG.

[1] <http://git.php.net/?p=php-src.git;a=commit;h=6a624c1dfda0dbfaaff3e453e6cb58de12748fb3>
Copy link
Member

@jmikola jmikola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution! I've created PHPC-1503 as a tracking ticket and will propose we get this merged and released as 1.6.1 soon.

@alcaeus: I think we'll want to add PHP 7.4 to the AppVeyor config alongside merging this.

@jmikola jmikola changed the title Fix config.w32 for PHP 7.4.0 PHPC-1503: Fix config.w32 for PHP 7.4.0 Dec 4, 2019
@jmikola
Copy link
Member

jmikola commented Dec 4, 2019

Cherry-picked into #1068 and merged in 06e45f6.

@jmikola jmikola closed this Dec 4, 2019
@cmb69 cmb69 deleted the cmb/vc_versions branch February 5, 2020 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants