00:02:33.277 There were 648 errors:
00:02:33.278
00:02:33.278 1) MediaWiki\Extension\Math\Tests\WikiTexVC\AllTest::testRunCases with data set "Box functions" (array('\text {-0-9a-zA-Z+*,=():/;?.!... -ÿ} ', '{\text{-0-9a-zA-Z+*,=():/;?.!... -ÿ}}'))
00:02:33.278 Creation of dynamic property MediaWiki\Extension\Math\WikiTexVC\Parser::$tu is deprecatedDescription
Description
Details
Details
Related Changes in Gerrit:
Customize query in gerrit
Event Timeline
Comment Actions
Change #1013399 had a related patch set uploaded (by Jforrester; author: Jforrester):
[integration/config@master] Zuul: [mediawiki/extensions/Math] Disable PHP 8.2 testing for now
Comment Actions
Change #1013399 merged by jenkins-bot:
[integration/config@master] Zuul: [mediawiki/extensions/Math] Disable PHP 8.2 testing for now
Comment Actions
Mentioned in SAL (#wikimedia-releng) [2024-03-21T20:20:03Z] <James_F> Zuul: [mediawiki/extensions/Math] Disable PHP 8.2 testing for now, for T360709
Comment Actions
@Jdforrester-WMF did we eventually drop support for PHP7. If so this might be easy to fix, by upgrading the library that generates the code that produces the errors.
Comment Actions
Dropping official PHP 7.4 support (and PHP 8.0 support) is happening from the REL1_42 branch, but CI will still enforce PHP 7.4 compatibility until Wikimedia production migrates to PHP 8.1.
Comment Actions
Ok. Do you think it is good enough to wait until then? If so we can maybe make this task a subtask of the migration of production migration task.
Comment Actions
Change #1015079 had a related patch set uploaded (by Physikerwelt; author: Physikerwelt):
[integration/config@master] Zuul: [mediawiki/extensions/MathSearch] Disable PHP 8.2 testing for now
Comment Actions
Change #1015079 merged by jenkins-bot:
[integration/config@master] Zuul: [mediawiki/extensions/MathSearch] Disable PHP 8.2 testing for now
Comment Actions
The only problem I see locally is the problem with dynamic property creation T314099.
Comment Actions
Change #1017425 had a related patch set uploaded (by Physikerwelt; author: Physikerwelt):
[mediawiki/extensions/Math@master] Declare properties statically
Comment Actions
Math depends on Wikibase T360560: Make PHP 8.2 voting on development branch of Wikibase
As Wikibase fails in step 2 of 3 (first is unit test, second "PHPUnit extensions suite (without database or standalone)" and third "PHPUnit extensions suite (with database)") is not proofed on CI that everything would run, but that is still a risk for all the Wikibase dependency.
After math runs so far, the comments in zuul config could be changed to point to Wikibase.
Comment Actions
Change #1017425 merged by jenkins-bot:
[mediawiki/extensions/Math@master] Declare properties statically
Comment Actions
Change #1017474 had a related patch set uploaded (by Reedy; author: Physikerwelt):
[mediawiki/extensions/Math@REL1_41] Declare properties statically
Comment Actions
Change #1017475 had a related patch set uploaded (by Reedy; author: Physikerwelt):
[mediawiki/extensions/Math@REL1_40] Declare properties statically
Comment Actions
Change #1017476 had a related patch set uploaded (by Reedy; author: Physikerwelt):
[mediawiki/extensions/Math@REL1_39] Declare properties statically
Comment Actions
Change #1017476 abandoned by Reedy:
[mediawiki/extensions/Math@REL1_39] Declare properties statically
Reason:
Comment Actions
Change #1017474 merged by jenkins-bot:
[mediawiki/extensions/Math@REL1_41] Declare properties statically
Comment Actions
Change #1017475 merged by jenkins-bot:
[mediawiki/extensions/Math@REL1_40] Declare properties statically
Comment Actions
Change #1024786 had a related patch set uploaded (by Jforrester; author: Jforrester):
[integration/config@master] Zuul: [mediawiki/extensions/Math] Re-enable PHP 8.2 CI voting
Comment Actions
Change #1024786 merged by jenkins-bot:
[integration/config@master] Zuul: [mediawiki/extensions/Math] Re-enable PHP 8.2 CI voting
Comment Actions
Mentioned in SAL (#wikimedia-releng) [2024-04-26T18:55:19Z] <James_F> Zuul: [mediawiki/extensions/Math] Re-enable PHP 8.2 CI voting for T360709
Comment Actions
Reopening, as it was still disabled. See for example https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Math/+/1035851 it failed only in the gate-and-submit phase because the php8 tests were not run in the test phase.
Comment Actions
Sorry, my local copy of the integration config was outdated... php8 tests only run when running check experimental and the behavior is the same for all extensions... so we can just wait until the PHP8 tests run by default
Comment Actions
Right now, PHP 7.4, PHP 8.1, and PHP 8.2 tests all run by default. Only the first of these runs in the test pipeline, the other two run in php, and all three run in the gate_and_submit pipeline.