Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Nov 25, 2024

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

ndossche and others added 22 commits November 25, 2024 19:38
This testing code was never meant to be used this way, but fixing this
will at least stop fuzzers from complaining about this, so might still
be worthwhile.

Closes GH-16919.
* PHP-8.2:
  Fix GH-16908: _ZendTestMagicCallForward does not handle references well
* PHP-8.3:
  Fix GH-16908: _ZendTestMagicCallForward does not handle references well
* PHP-8.4:
  Fix GH-16908: _ZendTestMagicCallForward does not handle references well
* PHP-8.2:
  Fix GH-16902: Set of opcache tests fail zts+aarch64 (8.2-8.3)
* PHP-8.3:
  Fix GH-16902: Set of opcache tests fail zts+aarch64 (8.2-8.3)
* PHP-8.4:
  Fix GH-16902: Set of opcache tests fail zts+aarch64 (8.4+)
  Fix GH-16902: Set of opcache tests fail zts+aarch64 (8.2-8.3)
* PHP-8.3:
  Fix test expectation for PHP 8.3+
* PHP-8.4:
  Fix test expectation for PHP 8.3+
This port is already used by san_ipv6_peer_matching.phpt, wo we choose
another port which is not explicitly used in our test suite.  The
proper solution would be to use ephemeral ports[1], but our OpenSSL
`ServerClientTestCase` does not support this yet.

[1] <6ab4e33>

Closes GH-16871.
* PHP-8.3:
  Use another for bug51056.phpt
* PHP-8.4:
  Use another for bug51056.phpt
* PHP-8.4:
  [skip ci] chore: update minimum required PHP version.
If `_DEBUG` is set, assertion failures and errors are directed to a
debug message window by default[1].  That causes a process to hang,
since these dialogs are modal.  While we already cater to assertion
failures, errors have apparently been overlooked.

We choose a minimal fix for BC reasons; although passing `0` as
`reportMode` is undocumented, it obviously works fine for a long time.
We may consider to improve on this for the `master` branch.

[1] <https://learn.microsoft.com/en-us/cpp/c-runtime-library/reference/crtsetreportmode>

Closes GH-16850.
* PHP-8.2:
  Fix GH-16849: Error dialog causes process to hang
* PHP-8.3:
  Fix GH-16849: Error dialog causes process to hang
* PHP-8.4:
  Fix GH-16849: Error dialog causes process to hang
@pull pull bot added the ⤵️ pull label Nov 25, 2024
@pull pull bot merged commit ba6c005 into wudi:master Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants