Page MenuHomePhabricator

Release utfnormal 3.0.0
Closed, ResolvedPublic

Description

The main change is https://gerrit.wikimedia.org/r/plugins/gitiles/utfnormal/+/1be6644a4b9f995f818a91f3fa5b2c41725b7ee3%5E%21/#F0 which replaces some methods with their native equivalents.

But since we have to do a breaking change anyways, should we just drop those functions, update usage everywhere (https://codesearch.wmcloud.org/search/?q=(codepointToUtf8%7Cutf8ToCodepoint)&i=nope&files=&excludeFiles=&repos=) and release 3.0.0 without them?

Otherwise we drop them in a future 4.0.0.

Event Timeline

Dropping the wrappers immediately with no cut-over feels a little unfriendly. Just because we can doesn't mean we should. Of course in practice I doubt anyone is using this outwith the Wikimedia universe; Packagist doesn't list anything public, for instance.

Change 660066 had a related patch set uploaded (by Legoktm; owner: Legoktm):
[utfnormal@master] Release 3.0.0

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

Change 660066 merged by jenkins-bot:
[utfnormal@master] Release 3.0.0

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

Change 660067 had a related patch set uploaded (by Legoktm; owner: Legoktm):
[mediawiki/libs/RemexHtml@master] Drop wikimedia/utfnormal in favor of mb_chr()

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

RemexHtml's test suite exposed an issue with mb_chr on PHP 7.2, see https://gerrit.wikimedia.org/r/c/utfnormal/+/660083/ as potential fix.

Change 660290 had a related patch set uploaded (by Legoktm; owner: Legoktm):
[utfnormal@master] Release 3.0.1

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

Change 660290 merged by jenkins-bot:
[utfnormal@master] Release 3.0.1

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

Change 660295 had a related patch set uploaded (by Legoktm; owner: Legoktm):
[mediawiki/libs/Equivset@master] Support wikimedia/utfnormal ^3.0.1

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

Change 660296 had a related patch set uploaded (by Legoktm; owner: Legoktm):
[css-sanitizer@master] Support wikimedia/utfnormal ^3.0.1

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

Change 660295 merged by jenkins-bot:
[mediawiki/libs/Equivset@master] Support wikimedia/utfnormal ^3.0.1

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

Change 660296 merged by jenkins-bot:
[css-sanitizer@master] Support wikimedia/utfnormal ^3.0.1

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

Change 660067 merged by jenkins-bot:
[mediawiki/libs/RemexHtml@master] Support wikimedia/utfnormal ^3.0.1

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

Change 660404 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/libs/RemexHtml@master] Release 2.2.2

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

Change 660404 merged by jenkins-bot:
[mediawiki/libs/RemexHtml@master] Release 2.2.2

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

Change 660420 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/vendor@master] Upgrade wikimedia/utfnormal from 2.0.0 to 3.0.1 (and concomitant libraries)

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

Change 660421 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/core@master] Upgrade wikimedia/utfnormal from 2.0.0 to 3.0.1 (and concomitant remex-html)

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

Change 660420 merged by jenkins-bot:
[mediawiki/vendor@master] Upgrade wikimedia/utfnormal from 2.0.0 to 3.0.1 (and concomitant libraries)

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

Change 660421 merged by jenkins-bot:
[mediawiki/core@master] Upgrade wikimedia/utfnormal from 2.0.0 to 3.0.1 (and concomitant remex-html)

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

Jdforrester-WMF assigned this task to Legoktm.
Jdforrester-WMF updated the task description. (Show Details)