Page MenuHomePhabricator

Bundle Math extension with MediaWiki
Closed, ResolvedPublic

Description

Extension Math does not have any additional dependencies, but php-curl. We might want to consider adding it to the default bundle.

  • Passed security review or already Wikimedia deployed
  • Voting CI structure tests
  • Runs MediaWiki-CodeSniffer
  • Runs phan
  • Runs seccheck
  • Supports MySQL, SQLite, and Postgres (if there are schema changes)
  • GPL v2 or later compatible license
  • Extension's default configuration provides optimal experience
  • Tested with web installer
  • Resolve dependency upon php-curl, if necessary

Event Timeline

Physikerwelt renamed this task from Add extension math to the default modules in REL 1.34 to Add extension math to the default extensions in REL 1.34.Sep 15 2019, 1:14 PM

Also consider T137926 (Support running MediaWiki without 'curl' PHP extension)

CCicalese_WMF renamed this task from Add extension math to the default extensions in REL 1.34 to Bundle Math extension with MediaWiki.Sep 17 2019, 5:24 PM
CCicalese_WMF removed a project: Release.
CCicalese_WMF updated the task description. (Show Details)

Is there a direct dependency of Math upon php-curl, or is it an indirect dependency through MultiHttpClient? If the latter, perhaps T202352 could eliminate that dependency. Since I do not believe there is a way in the web installer to prevent enabling Math if php-curl is not present, this may be a blocker for bundling this extension at this point.

CCicalese_WMF edited projects, added MW-1.35-release; removed MW-1.34-release.

I'm bumping this to 1.35 for now until the php-curl dependency is clarified. If this can be resolved in time for the 1.34 release, we can bump it back.

Is there a direct dependency of Math upon php-curl, or is it an indirect dependency through MultiHttpClient? If the latter, perhaps T202352 could eliminate that dependency.

It comes with MultiHttpClient.

https://github.com/wikimedia/mediawiki-extensions-Math/blob/54d1b77401a250d24a6923193c5c73459a10c2c2/src/MathRestbaseInterface.php#L170

Since I do not believe there is a way in the web installer to prevent enabling Math if php-curl is not present, this may be a blocker for bundling this extension at this point.

Yes, one either needs the MultiHttpClient or to install nodejs, and mathoid and do configuration, which is some effort. See
https://github.com/physikerwelt/mediawiki-docker/commit/b2f700ecb2f6fb9753c50e21ca70981ff3eff457#diff-3254677a7917c6c01f55212f86c57fbfR94-R99

Extension's default configuration provides optimal experience

Not sure what this means, but there is nothing one can do to improve the default configuration unless one has special needs.

I guess the strict definition of optimal as the minimum of a well-defined loss function, is not meant here.

Hey there. This task is proposed as a blocker to MediaWiki 1.37, which will be cut in less than three weeks' time. Please consider whether this will make that deadline, and if not, move it to block the MediaWiki 1.38 release (MW-1.38-release) or remove as a blocker entirely.

As far as I can see this should be resolved. The issue was the Math uses MultiHttpClient.

Now that things work without cURL, including for the MultiHttpClient.

I could test without php-curl if that is uncertain.

@Jdforrester-WMF is there any work to be done to get this extension included?

Change 715082 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/tools/release@master] Bundle Math extension with MediaWIki

https://gerrit.wikimedia.org/r/715082

Change 715085 had a related patch set uploaded (by Jforrester; author: Jforrester):

[integration/config@master] parameter_functions: Math extension is now tarballed

https://gerrit.wikimedia.org/r/715085

@Jdforrester-WMF is there any work to be done to get this extension included?

I've made the config patches. I'll leave the final call on this to whomever is running the release process.

Change 715144 had a related patch set uploaded (by Jforrester; author: Jforrester):

[integration/config@master] Zuul: [mediawiki/extensions/Math] Add Math to the main gate

https://gerrit.wikimedia.org/r/715144

I was trying to figure out why this task was moved by clicking on the linked tasks. However, I wasn't able to get it. Can someone explain it or link the related discussion?

Change 715144 merged by jenkins-bot:

[integration/config@master] Zuul: [mediawiki/extensions/Math] Add Math to the main gate

https://gerrit.wikimedia.org/r/715144

Mentioned in SAL (#wikimedia-releng) [2022-01-25T15:57:38Z] <James_F> Zuul: [mediawiki/extensions/Math] Add Math to the main gate for T232948

Change 715082 merged by jenkins-bot:

[mediawiki/tools/release@master] Bundle Math extension with MediaWiki

https://gerrit.wikimedia.org/r/715082

Change 715085 merged by jenkins-bot:

[integration/config@master] parameter_functions: Math extension is now tarballed

https://gerrit.wikimedia.org/r/715085

Mentioned in SAL (#wikimedia-releng) [2022-01-25T16:39:08Z] <James_F> Zuul: Mark Math extension as now tarballed in parameter_functions for T232948

Jdforrester-WMF claimed this task.

This is now done.

@Physikerwelt: T279842 was declined :-/

Yes, procedurally Declined rather than Resolved because nothing got added.