Summary: | kde-misc/kdiff3 supports building without KDE | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Matthew Turnbull <sparky> |
Component: | New packages | Assignee: | Bernard Cafarelli <voyageur> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | CC: | chetanreddy, gentoo, kde, sparky |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
Ebuild changes
kdiff3 patch Patch against ebuild for Qt-only installation Patch against ebuild for Qt-only installation (v2) full ebuild ebuild build.log |
Description
Matthew Turnbull
2010-01-31 02:32:19 UTC
Created attachment 217969 [details]
Ebuild changes
Created attachment 217971 [details]
kdiff3 patch
If upstream releases version that has optional kde4 we will gladly accept it. We won't alter the buildsystem of the current package tho. Feel free to sent the build patch to the upstream maintainer if you want it included. I was hoping that, at least, this would at least get the patch some testing/review (regardless of whether it made it into an ebuild) before pushing upstream.. But ok, I'll just push upstream and see what happens. I don't quite understand why this was WONTFIXed, but I'm interested to see this in Gentoo myself. With new Gentoo installations I otherwise have to build it outside of portage from the original sources in /usr/local. (It doesn't need any patches then.) Was something finally submitted to upstream kdiff3? Just asking because I don't find it in the SF tracker of kdiff3. The place to discuss this is at upstream bug tracking system, we won't maintain this in Portage if they don't accept it first. I don't actually think any change upstream is necessary because the package from upstream already installs flawlessly without KDE. I'm not good at writing ebuilds (yet) but I'm going to attach a first try in a minute which can do without any changes to the kdiff3 package. Created attachment 220841 [details, diff]
Patch against ebuild for Qt-only installation
I'm sure one can improve things with regard to the documentation, but this works with Qt-only. I don't have a box with KDE installed, so I cannot check if that still works (but according to the docs I found, it should).
I finally got around to pushing this up-stream https://sourceforge.net/tracker/?func=detail&aid=2957347&group_id=58666&atid=488548 (In reply to comment #8) > Created an attachment (id=220841) [details] > Patch against ebuild for Qt-only installation At the very least, the konquorer depend should be inside the KDE depend.Take a look at my ebuild patch for how I handle DEPEND and RDEPEND. I'm sorry I don't have more time to take a closer look. I've been really busy with my course work recently. Created attachment 221215 [details, diff]
Patch against ebuild for Qt-only installation (v2)
Good point. How about this then?
Is this at least good enough to reopen this bug? Or should I open a new one?
Worth reopening indeed. Just FYI, debian has a qt only version of kdiff3 that i've been using for a while without facing any bugs. So upstream definitely supports building without kde. Created attachment 245469 [details, diff] full ebuild Here's the full ebuild file that still works for me. Compared to the patch that I submitted half a year ago, this only removes the blank between >= and kde-base (which made some portage tools give weird results) and adds amd64-linux keyword (that is also part of the official ebuild). What is the way to proceed with this? Do I need to submit the patch for review somehow? http://www.gentoo.org/doc/en/ebuild-submit.xml does not give any hints. The ebuild diff does not work for -kde, but anyway, kdiff3 itself doesn't behave correct without KDE (eg missing icons). I'm following the upstream patch for the cmake patch though, we may add non-kde support in the future. For now I'm closing the bug, thank you all for your efforts though (In reply to comment #14) > The ebuild diff does not work for -kde Eh, Theo, can you please specify _what_ does not work? I have been using this ebuild since I submitted it, on several machines with -kde. It works and I have all the icons that I need. if it behaves ok then i'm fine in adding it in overlay. I'm attaching the ebuild i tried (a slightly modified version of yours, nothing special) and my build.log. When compiling with -kde it calls qmake (as expected) and then it calls cmake for some reason Created attachment 258438 [details]
ebuild
Created attachment 258440 [details]
build.log
OK, for you it fails because of FEATURES=test. I never had that enabled anywhere. But with that (and your ebuild) it fails for my, too, even earlier than for you: [... snip ...] -- Detecting CXX compiler ABI info -- Detecting CXX compiler ABI info - done CMake Error at /usr/share/cmake/Modules/FindKDE4.cmake:58 (MESSAGE): ERROR: Could not find KDE4 kde4-config Call Stack (most recent call first): CMakeLists.txt:4 (find_package) I have no idea what magic src_test or kde4-base_src_test do by default, so I cannot really help solve this failure (yours or mine). fixed in 0.9.95-r1 and -9999, enjoy |