Bugzilla – Bug 113259
SuSEconfig.tuxpaint using excesive time
Last modified: 2005-08-26 20:06:56 UTC
I timed the usage of SuSEconfig modules on a multiple run (taking a bit of time on a firest run is tolerated, a rerun should better not be above 1 second). Executing /sbin/conf.d/SuSEconfig.tuxpaint... Using "Verdana" (/usr/X11R6/lib/X11/fonts/truetype/verdanaz.ttf) for "el" locale. No font available for "zh" locale. No font available for "ko" locale. Using "Arial" (/usr/X11R6/lib/X11/fonts/truetype/arialbi.ttf) for "he" locale. Using "Lucida Sans" (/usr/lib/jvm/java-1.4.2-sun-1.4.2.06/jre/lib/fonts/LucidaSansRegular.ttf) for "hi" locale. Using "Tuxpaint font" (/usr/share/tuxpaint/fonts/locale/ta-orig.ttf) for "ta" locale. real 0m2.541s user 0m0.780s sys 0m0.116s 2.5 seconds on a really fast machine are far too much. Why is this hack needed anyway ?
Probably this is related to the fc-match/fc-list calls to get optimized fonts for asian languages. Since tuxpaint is not in default selection I don't think this really hurts.
Let's try to fix it.
I don't think this would be possible. I need to call fc-match for each language (el, zh, ko, he, hi, ta) which is supported by tuxpaint and call fc-list afterwards to match the filename of the font. Question is if we do care about good i18n support or if we do not. If we don't I can remove the SuSEconfig script.
It would be possible to split of a subpackage tuxpaint-suseconfig, which is only installed for el, zh, ko, he, hi, ta. IMHO overkill, but possible ...
Do we need to run this script with every SuSEconfig invocation? Is there no way to cache the information so that a second run will be fast? Otherwise let's keep the status quo.
Reimplementing the cache mechanism SuSEconfig.fonts is using to see if any new fonts were installed somewhere? I think this is not that easy and for sure more overkill than to split off a tuxpaint-suseconfig subpackage.
selectio.changes: ------------------------------------------------------------------- Fri Aug 26 22:01:26 CEST 2005 - sndirsch@suse.de - SuSEconfig.tuxpaint: * only check for fonts when "--force" is specified (Bug #113259) fixed for Beta4.