Bug 27995 - Request xxdiff ebuild allow optional KDE support
Bug#: 27995 Product:  Gentoo Linux Version: unspecified Platform: All
OS/Version: Linux Status: RESOLVED Severity: enhancement Priority: P2
Resolution: FIXED Assigned To: kde@gentoo.org Reported By: carl@hudkins.org
Component: Development
URL: 
Summary: Request xxdiff ebuild allow optional KDE support
Keywords:  
Status Whiteboard: 
Opened: 2003-09-05 06:17 0000
Description:   Opened: 2003-09-05 06:17 0000 
With version 2.6, xxdiff now may be compiled against kdelibs, allowing some
sort 
of KDE support in the application. As "kde" is a USE flag, I think xxdiff
should be 
built with KDE support if that flag is enabled. 

Reproducible: Always
Steps to Reproduce:
1.
2.
3.


Expected Results:  


Adding KDE support is done by changing 3 lines in src/xxdiff.pro before
building the 
Makefile. I have been attempting to get this to work, but the build currently 
bombs because ld cannot find -lkdeui. If I get this working, I will post my
ebuild.

------- Comment #1 From Theofilos Intzoglou 2003-09-06 15:48:34 0000 ------- 
Created an attachment (id=17193) [details]
xxdiff-2.9.2.ebuild.patch

Apply this patch to the ebuild

------- Comment #2 From Theofilos Intzoglou 2003-09-06 15:50:04 0000 ------- 
Created an attachment (id=17194) [details]
kdesupport.patch

Add this in /usr/portage/dev-util/xxdiff/files/ dir

------- Comment #3 From Theofilos Intzoglou 2003-09-06 15:51:30 0000 ------- 
Now these patches allow the program to be compiled with kde support. The binary
that is created seems to work just fine. Just don't ask me what kind of extra
things it adds :-) I leave that to you ;-)

------- Comment #4 From Carl Hudkins 2003-09-06 18:28:12 0000 ------- 
Rats, you beat me to the ebuild submission!  :) 
That's essentially what I did here, except I was trying to make the ebuild change xxdiff.pro using 
"dosed" instead of an external patch file.  (Oh, and I had removed some redundant things from 
the ebuild, like IUSE="" and S=${WORKDIR}/$P.)  However, since I don't seem to understand 
"dosed" (it doesn't actually *do* anything here), it's back to the old-fashioned way! 
Thanks for your work; now everyone else can enjoy the lovely KDE support of xxdiff -- which is 
basically use of kfiledialog and making the widgets follow the user's theme.  I really miss the 
extra up-arrow at the bottom of scrollbars, so of course I had to have it.  :)

------- Comment #5 From Carl Hudkins 2003-09-06 19:47:00 0000 ------- 
Created an attachment (id=17205) [details]
Optional KDE support & ebuild cleanup

Also removes superfluous items from ebuild, such as IUSE="".  Such things are
implied if not defined, so defining them to their default values is pointless.

------- Comment #6 From Caleb Tennis 2003-09-15 19:53:25 0000 ------- 
put your patch stuff in - -r1.  Let me know if there's any problems.  Thanks!

------- Comment #7 From Martin Blais 2003-10-16 22:54:32 0000 ------- 
whoah... sexy.  i had never even seen it myself with kde widgets.  i had
merely patched up someone else's contribution.   how cool!  now that i run
gentoo myself, all i had to do was emerge! :-)

(the author of xxdiff)