Bug 113259

Summary:	SuSEconfig.tuxpaint using excesive time
Product:	[openSUSE] SUSE LINUX 10.0	Reporter:	Ruediger Oertel <ro>
Component:	X11 Applications	Assignee:	Stefan Dirsch <sndirsch>
Status:	RESOLVED FIXED	QA Contact:	Stefan Dirsch <sndirsch>
Severity:	Critical
Priority:	P2 - High
Version:	Beta 3
Target Milestone:	---
Hardware:	Other
OS:	All
Whiteboard:
Found By:	Other	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---

Description Ruediger Oertel 2005-08-26 10:41:17 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 ?

Comment 1 Stefan Dirsch 2005-08-26 10:49:58 UTC

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.

Comment 2 Andreas Jaeger 2005-08-26 11:17:45 UTC

Let's try to fix it.

Comment 3 Stefan Dirsch 2005-08-26 12:48:00 UTC

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.

Comment 4 Stefan Dirsch 2005-08-26 16:11:07 UTC

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 ...

Comment 5 Andreas Jaeger 2005-08-26 17:59:38 UTC

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.

Comment 6 Stefan Dirsch 2005-08-26 19:08:06 UTC

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.

Comment 7 Stefan Dirsch 2005-08-26 20:06:56 UTC

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.