Page MenuHomePhabricator

ReplaceText Doxygen failure
Closed, ResolvedPublic

Description

19:40:41 /src/extensions/ReplaceText/maintenance/replaceAll.php:24: warning: missing argument after '\category'.
19:40:41 /src/extensions/ReplaceText/replaceAll.php:24: warning: missing argument after '\category'.
19:40:44 input buffer overflow, can't enlarge buffer because scanner uses REJECT
19:40:44 ---------------------------------------------------
19:40:44 Doxygen execution finished.
19:40:44 Check above for possible errors.

Event Timeline

@Reedy - looking at the code, I don't understand how this could have happened. Is this still an issue, do you know? And how can I see this for myself?

It'll be from one of the Doxygen jobs... I don't know if it's from one of the MW release branches, but I'm guessing it probably was...

Oh, interesting - I haven't seen any of this before. Sorry to keep pestering you, but do you have any idea where I can see the Replace Text output there? I don't see it anywhere.

I don't imagine we're specifically generating them for ReplaceText, as there's no Doxyfile, and nothing on https://doc.wikimedia.org/

Hence why I was questioning if it was on a release branch, as there's extension submodules for ReplaceText and other extensions, so it potentially would've been generated on one of those...

Even if I'd reference the URL from the CI job, it wouldn't still exist so long after :/

I get it - thanks for the information. Well, I guess I'll have to wait until the 1.40 release, then...

In theory any change merged to mediawiki core in a release branch should trigger it, and if it's still present, should be findable...

Oh. Where can I see such a change? I don't see any branch changes in the "mediawiki-core-doxygen-docker" listing.

Change 1009821 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/core@master] build: Restore Doxygen output for MediaWiki release tags

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

Change 1009821 merged by jenkins-bot:

[mediawiki/core@master] build: Restore Doxygen output for MediaWiki release tags

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

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

[mediawiki/core@REL1_41] build: Restore Doxygen output for MediaWiki release tags

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

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

[mediawiki/core@REL1_40] build: Restore Doxygen output for MediaWiki release tags

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

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

[mediawiki/core@REL1_39] build: Restore Doxygen output for MediaWiki release tags

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

Change 1010211 merged by jenkins-bot:

[mediawiki/core@REL1_39] build: Restore Doxygen output for MediaWiki release tags

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

Change 1010210 merged by jenkins-bot:

[mediawiki/core@REL1_40] build: Restore Doxygen output for MediaWiki release tags

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

Change 1010209 merged by jenkins-bot:

[mediawiki/core@REL1_41] build: Restore Doxygen output for MediaWiki release tags

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

@Reedy (or anyone else) - can this be closed?

I've no idea if the failure is fixed, but we don't seem to be publishing anything for ReplaceText

Yaron_Koren claimed this task.
Yaron_Koren added a subscriber: Krinkle.

I guess that's good enough evidence for me... I'm closing this as "Resolved". @Krinkle and everyone else - thank you for looking into this! Of course, if this is still an issue, feel free to re-open it.