Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NSString/test08.m tests fail with ICU 65 #104

Closed
doko42 opened this issue Feb 23, 2020 · 4 comments
Closed

NSString/test08.m tests fail with ICU 65 #104

doko42 opened this issue Feb 23, 2020 · 4 comments

Comments

@doko42
Copy link

doko42 commented Feb 23, 2020

base/NSString/test08.m:
Failed test: test08.m:47 ... 11 is greater than 2
Failed test: test08.m:61 ... numeric sort

Testing test08.m...
Running base/NSString/test08.m...
Passed test: test08.m:27 ... 1 is less than 2
Passed test: test08.m:32 ... 1 is less than 10
Passed test: test08.m:37 ... 1 is less than 11
Passed test: test08.m:42 ... 11 is greater than 1
Failed test: test08.m:47 ... 11 is greater than 2
Passed test: test08.m:56 ... text sort
Failed test: test08.m:61 ... numeric sort
Expected '(1, 2, 3, 10, 11, 20)' and got '(1, 10, 11, 2, 20, 3)'
Completed file: test08.m

@fredkiefer
Copy link
Member

I get the same results on my OpenSuse Tumbleweed. Looks like ICU requires different values there. I never got around to inspect this.
What OS are you using?

@doko42
Copy link
Author

doko42 commented Feb 23, 2020

that's trying to build the Ubuntu packages:
https://launchpad.net/ubuntu/+source/gnustep-base/1.26.0-6build3

@fredkiefer
Copy link
Member

I started this pull request #109 which fixes the issue for me locally. Could you please give it a try and comment on the pull request?

@rfm
Copy link
Contributor

rfm commented Aug 25, 2022

Closing due to lack of feedback

@rfm rfm closed this as completed Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants