Failing RTL test with ICU on Fedora #1429

Closed
springmeyer opened this Issue Aug 22, 2012 · 3 comments

Comments

Projects
None yet
2 participants
Owner

springmeyer commented Aug 22, 2012

Built Mapnik on Fedora(https://gist.github.com/3427021) and seeing a curious RTL visual test failure that likely has to do with ICU version. This is important because the main OSM server runs Fedora I think.

Visual text rendering summary: Failed: 183 different pixels:
    /tmp/mapnik-visual-images/rtl-point-200-agg.png (actual)
    tests/visual_tests/images/rtl-point-200-reference.png (expected)

Expected:

Actual:

Owner

springmeyer commented Aug 22, 2012

yum info libicu gives:


Installed Packages
Name        : libicu
Arch        : x86_64
Version     : 4.2.1
Release     : 9.9.amzn1
Size        : 19 M
Repo        : installed
From repo   : amzn-main
Summary     : International Components for Unicode - libraries
URL         : http://www.icu-project.org/
License     : MIT and UCD and Public Domain
Description : The International Components for Unicode (ICU) libraries provide
            : robust and full-featured Unicode services on a wide variety of
            : platforms. ICU supports the most current version of the Unicode
            : standard, and they provide support for supplementary Unicode
            : characters (needed for GB 18030 repertoire support).
            : As computing environments become more heterogeneous, software
            : portability becomes more important. ICU lets you produce the same
            : results across all the various platforms you support, without
            : sacrificing performance. It offers great flexibility to extend and
            : customize the supplied services.
Member

herm commented Aug 22, 2012

This might happen when the "rtl" property in string_info is incorrectly initalized.
The code depends on ICU version, but your version should be recent enough.
You can try adding debug output here:
https://github.com/mapnik/mapnik/blob/master/src/font_engine_freetype.cpp#L314-319

Member

herm commented Apr 1, 2013

This problem is almost certainly fixed by the RTL-aware rewrite of the line breaking code.
Refs #1428.

herm closed this Apr 1, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment