Bug 519032 – app-i18n/ibus-qt-1.3.3 should depend on app-doc/doxygen

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Johannes Janssen 2014-08-04 14:13:03 UTC]

With ibus-qt-1.3.3 bug 297567 was reintroduced. CMake build scripts have a hard dependency on doxygen. But it is only required of course when building with USE="doc". I append a fixed ebuild and a patch to CMakeLists.txt.

Reproducible: Always

Steps to Reproduce:
1. Not having doxygen installed.
2. USE="-doc" emerge -1 "=app-i18n/ibus-qt-1.3.3"

Actual Results:  
relevant part of build.log:

CMake Error at /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:108 (message):
  Could NOT find Doxygen (missing: DOXYGEN_EXECUTABLE)
Call Stack (most recent call first):
  /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:315 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake/Modules/FindDoxygen.cmake:85 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:80 (find_package)


-- Configuring incomplete, errors occurred!

[Comment 1 Johannes Janssen 2014-08-04 14:14:21 UTC]

Created attachment 382236 [details]
fixed ebuild

[Comment 2 Johannes Janssen 2014-08-04 14:15:59 UTC]

Created attachment 382238 [details, diff]
ibus-qt-1.3.3-doc.patch

[Comment 3 Jeroen Roovers (RETIRED) 2014-08-05 09:51:51 UTC]

Comment on attachment 382236 [details]
fixed ebuild

--- ibus-qt-1.3.3.ebuild        2014-08-01 09:33:54.000000000 +0200
+++ -   2014-08-05 11:51:42.873003966 +0200
@@ -29,6 +29,10 @@

 mycmakeargs="-DLIBDIR=$(get_libdir) all"

+src_prepare() {
+       epatch "${FILESDIR}/${P}-doc.patch"
+}
+
 src_compile() {
        cmake-utils_src_compile

[Comment 4 Yixun Lan 2014-08-05 10:13:45 UTC]

+  05 Aug 2014; Yixun Lan <[e-mail]> ibus-qt-1.3.3.ebuild,
+  +files/ibus-qt-1.3-doc.patch:
+  fix bug 519032, remove hard doxygen dep, thanks Johannes Janssen

[Comment 5 Yixun Lan 2014-08-05 10:14:41 UTC]

please re-sync, I've fixed this without revision bump.