Skip to content

php@8.0: switch to openssl@3 [not working with openssl@3]#134833

Closed
chenrui333 wants to merge 1 commit into
Homebrew:openssl-migration-stagingfrom
chenrui333:php@8.0-openssl3
Closed

php@8.0: switch to openssl@3 [not working with openssl@3]#134833
chenrui333 wants to merge 1 commit into
Homebrew:openssl-migration-stagingfrom
chenrui333:php@8.0-openssl3

Conversation

@chenrui333

Copy link
Copy Markdown
Member

See #134251.

Signed-off-by: Rui Chen <rui@chenrui.dev>
@chenrui333 chenrui333 added openssl-3-migration Related to switching to an OpenSSL 3 dependency staging-branch-pr This pull request targets a *-staging branch. labels Jun 24, 2023
@github-actions github-actions Bot added legacy Relates to a versioned @ formula formula deprecated Formula deprecated icu4c ICU use is a significant feature of the PR or issue labels Jun 24, 2023
@chenrui333

Copy link
Copy Markdown
Member Author
  In file included from /tmp/phpA8.0-20230624-11296-s9iqpx/php-8.0.29/main/php.h:439,
                   from /tmp/phpA8.0-20230624-11296-s9iqpx/php-8.0.29/ext/openssl/openssl.c:27:
  /tmp/phpA8.0-20230624-11296-s9iqpx/php-8.0.29/ext/openssl/openssl.c: In function ‘zm_startup_openssl’:
  /tmp/phpA8.0-20230624-11296-s9iqpx/php-8.0.29/ext/openssl/openssl.c:1328:58: error: ‘RSA_SSLV23_PADDING’ undeclared (first use in this function); did you mean ‘RSA_PKCS1_PADDING’?
   1328 |         REGISTER_LONG_CONSTANT("OPENSSL_SSLV23_PADDING", RSA_SSLV23_PADDING, CONST_CS|CONST_PERSISTENT);
        |                                                          ^~~~~~~~~~~~~~~~~~
  /tmp/phpA8.0-20230624-11296-s9iqpx/php-8.0.29/Zend/zend_constants.h:50:105: note: in definition of macro ‘REGISTER_LONG_CONSTANT’
     50 | #define REGISTER_LONG_CONSTANT(name, lval, flags)  zend_register_long_constant((name), sizeof(name)-1, (lval), (flags), module_number)
        |                                                                                                         ^~~~
  /tmp/phpA8.0-20230624-11296-s9iqpx/php-8.0.29/ext/openssl/openssl.c:1328:58: note: each undeclared identifier is reported only once for each function it appears in
   1328 |         REGISTER_LONG_CONSTANT("OPENSSL_SSLV23_PADDING", RSA_SSLV23_PADDING, CONST_CS|CONST_PERSISTENT);
        |                                                          ^~~~~~~~~~~~~~~~~~
  /tmp/phpA8.0-20230624-11296-s9iqpx/php-8.0.29/Zend/zend_constants.h:50:105: note: in definition of macro ‘REGISTER_LONG_CONSTANT’
     50 | #define REGISTER_LONG_CONSTANT(name, lval, flags)  zend_register_long_constant((name), sizeof(name)-1, (lval), (flags), module_number)
        |                                                                                                         ^~~~
  make: *** [Makefile:819: ext/openssl/openssl.lo] Error 1
  make: *** Waiting for unfinished jobs....

@chenrui333 chenrui333 added build failure CI fails while building the software linux Linux is specifically affected CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. and removed linux Linux is specifically affected labels Jun 24, 2023
@mchoo7

mchoo7 commented Jun 24, 2023

Copy link
Copy Markdown
Contributor

PHP: Requirements - Manuel

PHP 7.1-8.0 requires OpenSSL >= 1.0.1, < 3.0.

php@8.0 doesn't work on openssl@3.

@chenrui333

Copy link
Copy Markdown
Member Author

PHP: Requirements - Manuel

PHP 7.1-8.0 requires OpenSSL >= 1.0.1, < 3.0.

php@8.0 doesn't work on openssl@3.

make sense, closing this.

@chenrui333 chenrui333 closed this Jun 24, 2023
@chenrui333 chenrui333 changed the title php@8.0: switch to openssl@3 php@8.0: switch to openssl@3 [not working with openssl] Jun 24, 2023
@chenrui333 chenrui333 changed the title php@8.0: switch to openssl@3 [not working with openssl] php@8.0: switch to openssl@3 [not working with openssl@3] Jun 24, 2023
@github-actions github-actions Bot added the outdated PR was locked due to age label Jul 25, 2023
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jul 25, 2023
@chenrui333 chenrui333 deleted the php@8.0-openssl3 branch January 22, 2024 14:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

build failure CI fails while building the software CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. formula deprecated Formula deprecated icu4c ICU use is a significant feature of the PR or issue legacy Relates to a versioned @ formula openssl-3-migration Related to switching to an OpenSSL 3 dependency outdated PR was locked due to age staging-branch-pr This pull request targets a *-staging branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants