Last modified: 2013-02-20 21:56:57 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T45802, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 43802 - Do not bundle first-letters-root.ser with MediaWiki since it's only valid for one version of ICU
Do not bundle first-letters-root.ser with MediaWiki since it's only valid for...
Status: RESOLVED WONTFIX
Product: MediaWiki
Classification: Unclassified
Categories (Other open bugs)
1.21.x
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: i18n
Depends on:
Blocks: 43740
  Show dependency treegraph
 
Reported: 2013-01-09 23:17 UTC by Bartosz Dziewoński
Modified: 2013-02-20 21:56 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Bartosz Dziewoński 2013-01-09 23:17:25 UTC
+++ This bug was initially created as a clone of Bug #43740 +++

We need to have versions of first-letters-root.ser that correspond to different versions of icu library. These are fairly easy to make (I believe). Simply need to feed maintinance/languages/generateCollationData.php different data files corresponding to different versions of unicode. (Need to double check that icu library is using the unicode files we think it is. I have a vague memory of it changing to some file from the CLDR stuff a couple versions ago)
Comment 1 Bawolff (Brian Wolff) 2013-01-10 21:03:36 UTC
(In reply to comment #0)
>. I have a vague memory of
> it
> changing to some file from the CLDR stuff a couple versions ago)

Specificly I'm thinking of the issue reported in bug 30675 - need to use allkeys_CLDR.txt instead for later versions.
Comment 2 Bartosz Dziewoński 2013-02-09 21:08:55 UTC
It seems like we'd need at least five versions of this file, for Unicode 5.1, 5.2, 6.0, 6.1, and 6.2. (These are the version of Unicode generateCollationData.php supports after my changes, corresponding to ICU 4.0 .. 50.)

Therefore it seems like it would be a better idea not to bundle it, and just document that people wishing to use it should generate them themselves - the process is easy and just works.

And if we ever decide to start using ICU collation by default, these could be generated by the installer.

Thoughts?
Comment 3 Bartosz Dziewoński 2013-02-09 22:09:41 UTC
[Summary: Generate first-letters-root.ser files for different versions of the ICU library --> Do not bundle first-letters-root.ser with MediaWiki since it's only valid for one version of ICU]

Submitted I5d2a4e7e, removing this from the the git repo.

CC-ing Tim.
Comment 4 Bartosz Dziewoński 2013-02-20 21:56:57 UTC
Changeset abandoned. Wontfixing this; supposedly bug 43740 can and should be fixed in a different way.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links