Bug 55121 - French (fr) sorting does not use 'backward' rule any more with ICU 4.6
Summary: French (fr) sorting does not use 'backward' rule any more with ICU 4.6
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Jungshik Shin
URL:
Keywords:
: 55118 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-02-24 00:46 PST by Jungshik Shin
Modified: 2011-02-28 13:43 PST (History)
3 users (show)

See Also:


Attachments
add chromium-specific expectation file (1.57 KB, patch)
2011-02-25 17:25 PST, Jungshik Shin
no flags Details | Formatted Diff | Diff
same as above but a diff from the main expectation to be made clear using 'svn cp' (1.65 KB, patch)
2011-02-25 17:37 PST, Jungshik Shin
tony: review+
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jungshik Shin 2011-02-24 00:46:59 PST
French (fr_FR and many fr_FOO other than fr_CA) sorting does not use 'backward first accent weighting' rules any more. Thus, CLDR 1.9 and ICU 4.6 have different sorting for fr_FOO (where FOO !=CA) and fr_CA. 

fast/xsl/sort-locale has the expectation that treats both the same. Because other ports of Webkit still use old rules while Chrome with ICU 4.6 uses new rules, the expected result should go in platform/chromium. 

I'm making a patch.
Comment 1 Hajime Morrita 2011-02-24 01:07:36 PST
Thank you for filing this! I've also filed Bug 55118 so Could you close it after fix this?
Comment 2 Tony Chang 2011-02-24 11:06:50 PST
*** Bug 55118 has been marked as a duplicate of this bug. ***
Comment 3 Jungshik Shin 2011-02-25 17:25:25 PST
Created attachment 83903 [details]
add chromium-specific expectation file
Comment 4 Jungshik Shin 2011-02-25 17:37:17 PST
Created attachment 83904 [details]
same as above but a diff from the main expectation to be made clear using 'svn cp'
Comment 5 Tony Chang 2011-02-28 09:44:07 PST
Should there also be an updated to test_expectations.txt?
Comment 6 WebKit Commit Bot 2011-02-28 10:02:22 PST
Comment on attachment 83904 [details]
same as above but a diff from the main expectation to be made clear using 'svn cp' 

Rejecting attachment 83904 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-8', 'apply-..." exit_code: 2

Last 500 characters of output:
ebkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'Tony Chang', u'--force']" exit_code: 2

Parsed 3 diffs from patch file(s).
cp: fast/xsl/sort-locale-expected.txt: No such file or directory
Failed to copy fast/xsl/sort-locale-expected.txt platform/chromium/fast/xsl/sort-locale-expected.txt. at /mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply line 427.

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'Tony Chang', u'--force']" exit_code: 2

Full output: http://queues.webkit.org/results/8070423
Comment 7 Tony Chang 2011-02-28 11:46:03 PST
(In reply to comment #6)
> Parsed 3 diffs from patch file(s).
> cp: fast/xsl/sort-locale-expected.txt: No such file or directory
> Failed to copy fast/xsl/sort-locale-expected.txt platform/chromium/fast/xsl/sort-locale-expected.txt. at /mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply line 427.
> 
> Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'Tony Chang', u'--force']" exit_code: 2
> 
> Full output: http://queues.webkit.org/results/8070423

The diff around the copy looks weird (it's relative to a different path than the other 2 files).
Comment 8 Jungshik Shin 2011-02-28 13:43:52 PST
Thank you for the review. 

The patch creating script does something strange in this case. Anyway, I manually landed the patch with the correct path.  ( http://trac.webkit.org/changeset/79912 ). 

Later, I'll update test_expectations.txt as well. 


































Thank you for the review. 

The patch creating script does something strange in this case. Anyway, I manually landed the patch with the correct path ( http://trac.webkit.org/changeset/79912 ). 

I forgot