[go: up one dir, main page]
Skip to content
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.

Sign up for GitHub

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

Jump to bottom

PHP fails to build if openssl was built with --no-ec #9064

Closed
Lex-2008 opened this issue Jul 20, 2022 · 1 comment
Closed

PHP fails to build if openssl was built with --no-ec #9064

Lex-2008 opened this issue Jul 20, 2022 · 1 comment
Labels
Bug Extension: openssl Status: Verified

Comments

@Lex-2008
Copy link

Description

Looking at openssl.c:

Worth noting that further down it's properly #ifdef'd out:
https://github.com/php/php-src/blob/php-8.1.8/ext/openssl/openssl.c#L5003

#ifdef HAVE_EVP_PKEY_EC
		case EVP_PKEY_EC:
			ktype = OPENSSL_KEYTYPE_EC;

Probably it should be #ifdef'd on line 4899, too?

PHP Version

8.1.8

Operating System

No response
Lex-2008 pushed a commit to Lex-2008/buildscripts that referenced this issue Jul 20, 2022
php compilation breaks if openssl built with no-ec
ba11f1a 
php/php-src#9064
@cmb69
Copy link
Member
cmb69 commented Jul 20, 2022

Indeed, looks like this occurrence has been overlooked.

Lex-2008 pushed a commit to Lex-2008/buildscripts that referenced this issue Jul 21, 2022
Add many no's to openssl configure
ef0ce61 
also remove no-ssl from solaris

Note that:

PHP doesn't work without these:

no-md2
no-md4
no-md5
no-rmd160

according to php/php-src#8430

And also no-ec:

php/php-src#9064
vpodzime pushed a commit to Lex-2008/buildscripts that referenced this issue Aug 2, 2022
@vpodzime
Add many no's to openssl configure
5a3cd18 
also remove no-asm from solaris

Note that:

PHP doesn't work without these:

no-md2
no-md4
no-md5
no-rmd160

according to php/php-src#8430

And also no-ec:

php/php-src#9064
vpodzime pushed a commit to Lex-2008/buildscripts that referenced this issue Aug 2, 2022
@vpodzime
Add many no's to openssl configure
31f7b12 
also remove no-asm from solaris

Note that:

PHP doesn't work without these:

no-md2
no-md4
no-md5
no-rmd160

according to php/php-src#8430

And also no-ec:

php/php-src#9064
Lex-2008 pushed a commit to Lex-2008/buildscripts that referenced this issue Aug 19, 2022
Add many no's to openssl configure
8b62307 
also remove no-asm from solaris

Note that:

PHP doesn't work without these:

no-md2
no-md4
no-md5
no-rmd160

according to php/php-src#8430

And also no-ec:

php/php-src#9064
Lex-2008 pushed a commit to Lex-2008/buildscripts that referenced this issue Sep 12, 2022
Add many no's to openssl configure
1caaacd 
also remove no-asm from solaris

Note that:

PHP doesn't work without these:

no-md2
no-md4
no-md5
no-rmd160

according to php/php-src#8430

And also no-ec:

php/php-src#9064
Lex-2008 pushed a commit to Lex-2008/buildscripts that referenced this issue Sep 12, 2022
Disable unused openssl features
09a3e88 
Also, store all openssl config flags in dedicated files, to be stored
centrally and used on all platforms.

Also run configdata.pm --dump to show enabled features

Also note that this removes no-asm from solaris

Ticket: ENT-8933

Note that PHP doesn't work without: md2 md4 md5 rmd160, according to
php/php-src#8430

And also without ec, according to
php/php-src#9064
Lex-2008 pushed a commit to Lex-2008/buildscripts that referenced this issue Sep 21, 2022
Disable unused openssl features
b31aa93 
Also, store all openssl config flags in dedicated files, to be stored
centrally and used on all platforms.

Also run configdata.pm --dump to show enabled features

Also note that this removes no-asm from solaris

Ticket: ENT-8933

Note that PHP doesn't work without: md2 md4 md5 rmd160, according to
php/php-src#8430

And also without ec, according to
php/php-src#9064
Lex-2008 pushed a commit to Lex-2008/buildscripts that referenced this issue Sep 22, 2022
Disable unused openssl features
b75b527 
Also, store all openssl config flags in dedicated files, to be stored
centrally and used on all platforms.

Also run configdata.pm --dump to show enabled features

Also note that this removes no-asm from solaris

Ticket: ENT-8933

Note that PHP doesn't work without: md2 md4 md5 rmd160, according to
php/php-src#8430

And also without ec, according to
php/php-src#9064
@bukka bukka closed this as completed in ce57221 Nov 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Extension: openssl Status: Verified
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@cmb69 @Lex-2008 and others