Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 511494 - [kde overlay] kde-base/plasma-workspace:5 missing DEPEND on kde-frameworks/kdewebkit?
Summary: [kde overlay] kde-base/plasma-workspace:5 missing DEPEND on kde-frameworks/kd...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] KDE (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo KDE team
URL:
Whiteboard:
Keywords:
: 516554 (view as bug list)
Depends on:
Blocks:
 
Reported: 2014-05-25 20:01 UTC by vltg0903
Modified: 2014-07-22 10:12 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description vltg0903 2014-05-25 20:01:03 UTC
Otherwise either of these missing cmake files trips up the build:

KF5WebKitConfig.cmake
kf5webkit-config.cmake


Reproducible: Always
Comment 1 Johannes Huber (RETIRED) gentoo-dev 2014-05-26 12:20:42 UTC
It could be that the upstream build system is wrong here.
Comment 2 Michael Palimaka (kensington) gentoo-dev 2014-05-26 12:22:42 UTC
It's on my list of bogus deps to follow up with upstream, I just didn't have the time yet.
Comment 3 vltg0903 2014-05-26 13:56:57 UTC
A quick search through the plasma-workspace sources shows that there is some real dependency here on headers provided by kdewebkit:

# ~/tmp/plasma-workspace-4.96.0 $ grep -in include * -R | grep 'k.*web.*\.h' 
applets/webbrowser/webbrowser.cpp:45:#include <kwebwallet.h>
applets/webbrowser/webbrowserpage.cpp:27:#include <kwebwallet.h>


(Might be possible to make it OPTIONAL though...)
Comment 4 Malte E. 2014-06-02 02:10:09 UTC
same here, with plasma-workspace-9999:

CMake Warning at /usr/share/ECM/find-modules/FindKF5.cmake:75 (find_package):
  Could not find a package configuration file provided by "KF5WebKit" with
  any of the following names:

    KF5WebKitConfig.cmake
    kf5webkit-config.cmake

  Add the installation prefix of "KF5WebKit" to CMAKE_PREFIX_PATH or set
  "KF5WebKit_DIR" to a directory containing one of the above files.  If
  "KF5WebKit" provides a separate development package or SDK, be sure it has
  been installed.
Call Stack (most recent call first):
  CMakeLists.txt:11 (find_package)


-- Could NOT find KF5WebKit: found neither KF5WebKitConfig.cmake nor kf5webkit-config.cmake 
-- Found KF5KDELibs4Support: /usr/lib64/cmake/KF5KDELibs4Support/KF5KDELibs4SupportConfig.cmake (found version "4.100.0") 
CMake Error at /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:108 (message):
  Could NOT find KF5 (missing: WebKit) (found version "4.100.0")
Call Stack (most recent call first):
  /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:315 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/ECM/find-modules/FindKF5.cmake:111 (find_package_handle_standard_args)
  CMakeLists.txt:11 (find_package)
Comment 5 Michael Palimaka (kensington) gentoo-dev 2014-06-26 15:18:07 UTC
KF5WebKit is only used if drkonqi is built with (optional) KF5XmlRpcClient (which has no release and isn't packaged yet. I have a pending patch upstream to address this issue.
Comment 6 Malte E. 2014-07-01 02:03:37 UTC
in the meantime, how about adding the dependency back to plasma-workspace? It just doesn't compile without it, whether it's actually needed or not.
Comment 7 Michael Palimaka (kensington) gentoo-dev 2014-07-01 12:45:34 UTC
We could backport the patch for now I guess, but the ebuilds are deliberately a work in progress - keep in mind even upstream it's still beta. :-)
Comment 8 Andreas Sturmlechner gentoo-dev 2014-07-06 21:49:06 UTC
*** Bug 516554 has been marked as a duplicate of this bug. ***
Comment 9 Malte E. 2014-07-09 11:24:46 UTC
The RC still depends on kdewebkit. I doubt that is a mistake.
Comment 10 Michael Palimaka (kensington) gentoo-dev 2014-07-09 11:25:56 UTC
It is a mistake, I just didn't have a chance to merge the patch upstream yet.
Comment 11 Michael Palimaka (kensington) gentoo-dev 2014-07-22 10:12:56 UTC
Due to upstream changes, kdewebkit is now (possibly temporarily) unconditionally required. See commit message for more info.

http://git.overlays.gentoo.org/gitweb/?p=proj/kde.git;a=commit;h=e360c1ed09b5fd76ede2e95dce5c2b6a18f5a61a