Ah, thanks for tracking that down and fixed it, Greg!

On 4/5/25 8:29 AM, Greg Hellings wrote:
The issue was a bit more arcane.

Sword's configure.ac <http://configure.ac> unilaterally refuses to even consider linking ICU if it cannot find icu-config, re-gardles of options passed in. For the cross build ICU package in Fedora there is no icu-config file. So I had to patch configure.ac <http://configure.ac> to check for the ICU packages with pkg-config if icu-config cannot be found, rather than blindly refuse to include ICU.

So the issue was that we were building the library without ICU support but thought we were building it with, because of the configure options we used.

--Greg

On Sat, Apr 5, 2025, 10:13 AM Troy A. Griffitts <scr...@crosswire.org> wrote:

    Hi Karl,

    I wonder if an ICU data dll is not present on the path. Just a
    first thought.


    On 4/1/25 10:40 AM, Karl Kleinpaste wrote:
    We have a Xiphos bug saying that the new release does not support
    transliteration in the Windows build. It works fine in Linux.

    Xiphos itself knows next to nothing about transliteration, just
    the option on the module context menu which provides for turning
    the feature on/off. There is nothing deeper in Xiphos that knows
    or cares, it just tells the engine what's wanted, and depends on
    the engine to render text accordingly.

    So the problem seems to be that it isn't enabled in how Sword is
    built for Xiphos. And a couple of us looking at this are fairly
    mystified.

    It seems to be the case that this is driven by #define of
    "_ICU_", based on its use in src/mgr/swmgr.cpp lines 300 and
    1200. But the Windows build provides for ICU (config summary
    shows it having been found) so we see no reason why this would be
    a question.

    Am I misssing the mark? Is transliteration enabled by some other
    configurable aspect?

    Transliteration worked in Windows build of Xiphos 4.2.1 (I
    checked), so I got tag "sword-1-8-1" in order to diff swmgr.cpp
    versus today's 1.9. Nothing of consequence; "ICU" does not appear
    in the diff at all.

    We could really use a pointer as to what we are missing.
    Thoughts, theories, clues, and wild ass guesses are all welcome.

    _______________________________________________
    sword-devel mailing list:sword-devel@crosswire.org
    http://crosswire.org/mailman/listinfo/sword-devel
    Instructions to unsubscribe/change your settings at above page
    _______________________________________________
    sword-devel mailing list: sword-devel@crosswire.org
    http://crosswire.org/mailman/listinfo/sword-devel
    Instructions to unsubscribe/change your settings at above page


_______________________________________________
sword-devel mailing list:sword-devel@crosswire.org
http://crosswire.org/mailman/listinfo/sword-devel
Instructions to unsubscribe/change your settings at above page
_______________________________________________
sword-devel mailing list: sword-devel@crosswire.org
http://crosswire.org/mailman/listinfo/sword-devel
Instructions to unsubscribe/change your settings at above page

Reply via email to