forked from php/php-src
-
Notifications
You must be signed in to change notification settings - Fork 0
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
[pull] master from php:master #440
Open
pull
wants to merge
10,000
commits into
jamlee-t:master
Choose a base branch
from
php:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+940,589
−757,167
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dstogov
force-pushed
the
master
branch
2 times, most recently
from
October 11, 2023 06:58
63c8ce8
to
7dc352e
Compare
iluuu1994
force-pushed
the
master
branch
4 times, most recently
from
October 13, 2023 09:46
49b8168
to
6d3d4df
Compare
* PHP-8.4: [skip ci] Fix array_any() and array_all() descriptions
* PHP-8.4: [skip ci] Fix typo in test name
On NixOS we run `make` & `make check` inside `ext/soap` which broke the test like this: 001+ Fatal error: Uncaught SoapFault exception: [WSDL] SOAP-ERROR: Parsing WSDL: Couldn't load from 'ext/soap/tests/gh15711.wsdl' : failed to load "ext/soap/tests/gh15711.wsdl": No such file or directory 002+ in /build/php-8.3.13/ext/soap/tests/gh15711.php:29 003+ Stack trace: 004+ #0 /build/php-8.3.13/ext/soap/tests/gh15711.php(29): SoapClient->__construct('ext/soap/tests/...', Array) 005+ #1 {main} 006+ thrown in /build/php-8.3.13/ext/soap/tests/gh15711.php on line 29 Fix is to make the path dependant on `__DIR__` as it's the case in other testcases including WSDLs. Closes GH-16733.
* PHP-8.3: ext/soap: fix `make check` being invoked in `ext/soap`
* PHP-8.4: ext/soap: fix `make check` being invoked in `ext/soap`
When passing an array, the key entry can get converted to a string if it is an object, but this actually modifies the original array entry. The test originally outputted: ``` array(2) { [0]=> string(...) => ... [1]=> string(0) "" } ``` This is unexpected. Use zval_try_get_string() to prevent this behaviour. Closes GH-16693.
* PHP-8.2: Fix memory leak in php_openssl_pkey_from_zval() Fix various memory leaks related to openssl exports Prevent unexpected array entry conversion when reading key
* PHP-8.3: Fix memory leak in php_openssl_pkey_from_zval() Fix various memory leaks related to openssl exports Prevent unexpected array entry conversion when reading key
* PHP-8.4: Fix memory leak in php_openssl_pkey_from_zval() Fix various memory leaks related to openssl exports Prevent unexpected array entry conversion when reading key
* PHP-8.4: Make MySQLnd protocol stmt test work on 32bit
There are compile errors with 8.4 that we are no longer fixing.
* PHP-8.1: [skip ci] Don't test mysqli with libmysqlclient 8.4
* PHP-8.2: [skip ci] Don't test mysqli with libmysqlclient 8.4
* PHP-8.3: [skip ci] Don't test mysqli with libmysqlclient 8.4
* PHP-8.4: [skip ci] Don't test mysqli with libmysqlclient 8.4
* PHP-8.1: Move FreeBSD build to GitHub actions with QEMU
* PHP-8.2: [skip ci] Add --no-progress to FreeBSD build Move FreeBSD build to GitHub actions with QEMU
* PHP-8.3: [skip ci] Add --no-progress to FreeBSD build Move FreeBSD build to GitHub actions with QEMU
* PHP-8.4: Enable -Werror for FreeBSD build [skip ci] Add --no-progress to FreeBSD build Move FreeBSD build to GitHub actions with QEMU
While the test obviously succeeds on Windows, it may occasionally conflict with file_put_contents_variation7-win32.phpt[1], so we skip it like we do for many other of these tests which have win32 pendants. [1] <https://github.com/php/php-src/actions/runs/11527743659/job/32093951818> Closes GH-16722.
* PHP-8.2: Skip file_put_contents_variation7.phpt on Windows
* PHP-8.3: Skip file_put_contents_variation7.phpt on Windows
* PHP-8.4: Skip file_put_contents_variation7.phpt on Windows
There doesn't seem to be a thread post-startup hook that runs after zend_startup_cb() that could be used for this this fix is similar to accel_startup_ok() as seen here: https://github.com/php/php-src/blob/fc1db70f106525e81f9a24539340b7cf2e82e844/ext/opcache/ZendAccelerator.c#L2631-L2634 Closes GH-16853.
* PHP-8.2: Fix GH-16851: JIT_G(enabled) not set correctly on other threads
* PHP-8.3: Fix GH-16851: JIT_G(enabled) not set correctly on other threads
* PHP-8.4: Fix GH-16851: JIT_G(enabled) not set correctly on other threads
* PHP-8.4: Revert "Merge branch 'PHP-8.3' into PHP-8.4"
Co-authored-by: Marcos Marcolin <[email protected]>
This reverts commit 83ca374, and fixes the previous bad merge.
* PHP-8.4: Reapply "Merge branch 'PHP-8.3' into PHP-8.4"
Allows to select an alternate TCP stack. For example with RACK, a fast loss detection relying on timestamp per packet. While it works system-wide, it can also apply in an individual socket level too. close GH-16842
It seems at one point these were (partially) rewritten, changing their meaning. This patch fixes this.
* PHP-8.4: [ci skip] UPGRADING fixes (#16893)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )