First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 27995
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Gentoo KDE team <kde@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Carl Hudkins <carl@hudkins.org>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
xxdiff-2.9.2.ebuild.patch xxdiff-2.9.2.ebuild.patch patch Theofilos Intzoglou 2003-09-06 15:48 0000 628 bytes Details | Diff
kdesupport.patch kdesupport.patch patch Theofilos Intzoglou 2003-09-06 15:50 0000 590 bytes Details | Diff
kxxdiff-2.9.2.patch Optional KDE support & ebuild cleanup patch Carl Hudkins 2003-09-06 19:47 0000 959 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 27995 depends on: Show dependency tree
Bug 27995 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug

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)
First Last Prev Next    No search results available      Search page      Enter new bug