Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 247649

Summary: kde-misc/kim for KDE-4
Product: Gentoo Linux Reporter: Bjoern Olausson <contactme>
Component: New packagesAssignee: Gentoo Linux bug wranglers <bug-wranglers>
Status: RESOLVED DUPLICATE    
Severity: enhancement    
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: kim-0.9.5.ebuild for KDE-4
kim4.patch
kim4.patch

Description Bjoern Olausson 2008-11-20 00:30:39 UTC
Porting KIM to KDE-4

Reproducible: Always




I am starting this new bug to handle KIM for KDE-4 independent from KIM for KDE-3 (http://bugs.gentoo.org/show_bug.cgi?id=154115)

http://www.kde-apps.org/content/show.php/kim%2BSet+serv+menus%2Bright+click+ACTIONS+?content=37277

Kind regards
Bjoern
Comment 1 Bjoern Olausson 2008-11-20 00:31:49 UTC
Created attachment 172427 [details]
kim-0.9.5.ebuild for KDE-4

kim-0.9.5.ebuild ported to work with KDE-4
Comment 2 Bjoern Olausson 2008-11-20 00:43:42 UTC
Created attachment 172428 [details, diff]
kim4.patch

There is a lot to optimize like

--> removing unnecessary external programs and use bash build in tools

-		RANDOM=`perl -e 'print int(rand(1000000))'`;
+		RAND="$RANDOM";

--> Moving duplicate code int one function that can be reused
--> Make the coding-style nicer
--> Making some features more usable (Like changing black text to white text with a black border so it can be seen on white and black background)

...

CHANGELOG:
- Optimized some more code (still lot to do)
- Modyfied the "Add text annotation" to produce white text with black border instead of plain black text

I am working on it when I got time... so expect more patches to come.

Kind regards
Bjoern
Comment 3 Bjoern Olausson 2008-11-20 00:44:21 UTC
Created attachment 172430 [details, diff]
kim4.patch

There is a lot to optimize like

--> removing unnecessary external programs and use bash build in tools

-		RANDOM=`perl -e 'print int(rand(1000000))'`;
+		RAND="$RANDOM";

--> Moving duplicate code int one function that can be reused
--> Make the coding-style nicer
--> Making some features more usable (Like changing black text to white text with a black border so it can be seen on white and black background)

...

CHANGELOG:
- Optimized some more code (still lot to do)
- Modyfied the "Add text annotation" to produce white text with black border instead of plain black text

I am working on it when I got time... so expect more patches to come.

Kind regards
Bjoern
Comment 4 Jeroen Roovers (RETIRED) gentoo-dev 2008-11-20 22:13:17 UTC
There's absolutely no need for more than one bug per new package.

*** This bug has been marked as a duplicate of bug 154115 ***