Bug 16814 - sqlite3 is built without ICU support
Summary: sqlite3 is built without ICU support
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 5
Hardware: All Linux
Priority: Normal enhancement
Target Milestone: ---
Assignee: Mageia Bug Squad
QA Contact:
URL:
Whiteboard:
Keywords: PATCH
Depends on: 16842
Blocks:
  Show dependency tree / graph
 
Reported: 2015-09-23 21:21 CEST by Oleg Bosis
Modified: 2015-10-27 13:48 CET (History)
6 users (show)

See Also:
Source RPM:
CVE:
Status comment:

Attachments
Patch to enable ICU support (967 bytes, patch)
2015-09-23 21:23 CEST, Oleg Bosis 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Description Oleg Bosis 2015-09-23 21:21:49 CEST 
Description of problem:
sqlite3 is currently built without ICU support. This support is required to enable case-insensitive search for/select non-Latin strings in databases. Without ICU support only Latin-1 characters/strings can be searched/selected in case-insensitive manner. Both sqlite3's upper() and lower() functions do not support non-Latin character conversion to upper/lower case without ICU support.

Version-Release number of selected component (if applicable):


How reproducible:
always

Steps to Reproduce:
1.
2.
3.


Reproducible: 

Steps to Reproduce:
Comment 1 Oleg Bosis 2015-09-23 21:23:18 CEST 
Created attachment 7046 [details]
Patch to enable ICU support
Oleg Bosis 2015-09-23 21:23:29 CEST

CC: (none) => olegbosis

Comment 2 Oleg Bosis 2015-09-23 21:40:38 CEST 
The simplest wey to test I've found so far:

sqlite> SELECT "s" LIKE "S";
1
sqlite> SELECT "Ñ" LIKE "Ы";
1
sqlite>

If the second select statement with Russian "Ñ" character return '0' then the ICU support is missing.
Comment 3 Samuel Verschelde 2015-09-24 09:35:09 CEST 
Thanks for the report. sqlite3 has no dedicated maintainer but several packagers have touched it in the past, so I'm adding them in CC.

Packagers: sqlite3 is an important package. If any of you would accept to be the maintainer, this would help assigning bug reports to the right person.

Keywords: (none) => PATCH
CC: (none) => fundawang, luigiwalser, n54, thierry.vignaud

Comment 4 David Walser 2015-09-24 23:02:29 CEST 
Just FYI, Fedora and OpenSuSE do not build sqlite3 with ICU support.
Comment 5 Oleg Bosis 2015-09-25 07:20:33 CEST 
(In reply to David Walser from comment #4)
> Just FYI, Fedora and OpenSuSE do not build sqlite3 with ICU support.

So what? I've rebuilt the package with ICU for all my PCs and I do not see any regressions. FF and TB do work OK. My own sqlite3-based projects (using Qt5) are able to do proper DB search (and there were no need to rebuild Qt5). I do not see any reason to not enable this feature in cauldron and test it more thoroughly. And if everything would be OK then backport it to MGA5.
Comment 6 David Walser 2015-09-25 14:09:20 CEST 
Fixed in sqlite3-3.8.11.1-2.mga6.

For Mageia 5, we'll need to update sqlite3 when we update to the next stable Firefox branch (45esr).  I'll try to remember to fix this then.
Comment 7 Christiaan Welvaart 2015-10-04 14:43:21 CEST 
Iceape 2.38 already needs the sqlite3 change so I checked that into svn. ICU support now also added in svn. I guess sqlite3 should be included in the iceape advisory.

CC: (none) => cjw

Comment 8 David Walser 2015-10-27 13:48:27 CET 
Fixed in http://advisories.mageia.org/MGASA-2015-0414.html

Status: NEW => RESOLVED
Depends on: (none) => 16842
Resolution: (none) => FIXED
Note You need to log in before you can comment on or make changes to this bug.