Bug 88138

Summary:	ICU libs should be checked using pkg-config/.pc instead of icu-config
Product:	LibreOffice	Reporter:	Rémi Cardona <remi>
Component:	Libreoffice	Assignee:	Not Assigned <libreoffice-bugs>
Status:	NEW ---	QA Contact:
Severity:	normal
Priority:	medium	CC:	dtardon, libreoffice
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:	EasyHack DifficultyBeginner SkillScript TopicCleanup
i915 platform:		i915 features:

Description Rémi Cardona 2015-01-07 07:49:49 UTC

Before hitting bug #85214, I found that icu-config was not POSIX-compliant either. While fixing the issue and reporting it upstream[1], one of the ICU devs pointed out that the icu-config script is deprecated and that pkg-config should be used instead[2].

Cheers

[1] http://bugs.icu-project.org/trac/ticket/11460
[2] http://userguide.icu-project.org/howtouseicu#TOC-C-Makefiles

Comment 1 Adolfo Jayme 2015-01-07 12:55:09 UTC

Thanks for the report. Marking it NEW.

Comment 2 David Tardon 2015-01-07 13:30:47 UTC

We need to get the ICU version, which cannot be done using pkg-config with the existing .pc files (IOW, there is no variable containing version that we could query).

Comment 3 Rémi Cardona 2015-01-07 21:57:59 UTC

From LO's configure.ac around line 8795:

$ icu-config --noverify --version
53.1

$ icu-config --version
53.1

Using pkg-config, one can call the following:

$ pkg-config --modversion icu-uc
53.1
$ pkg-config --variable=UNICODE_VERSION icu-uc
6.3

Is more needed? That's all I could find in configure.ac.

Comment 4 David Tardon 2015-01-08 11:20:41 UTC

Eh, right. pkg-config --modversion is sufficient.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.