Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 730170 - app-doc/doxygen-1.8.18 with clang-10 fails to link against/find libclangFormat.so libclangTooling.so libclangRewrite.so libclangLex.so libclangBasic.so
Summary: app-doc/doxygen-1.8.18 with clang-10 fails to link against/find libclangForma...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Matthias Maier
URL:
Whiteboard:
Keywords: PATCH, UPSTREAM
Depends on:
Blocks: llvm-10
  Show dependency tree
 
Reported: 2020-06-29 19:04 UTC by Joakim Tjernlund
Modified: 2020-12-10 19:14 UTC (History)
2 users (show)

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


Attachments
Proposed patch (0001-Fix-7883.patch,1.77 KB, patch)
2020-07-19 11:38 UTC, Joakim Tjernlund
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joakim Tjernlund 2020-06-29 19:04:38 UTC
Where did they go?
Comment 1 Joakim Tjernlund 2020-06-29 20:11:39 UTC
Fond out they are relaced with clang-cpp

Anyhow, doxygen wants them but I didn't want llvm-9 just for doxygen so I did
this hack:
--- ./src/CMakeLists.txt.org	2020-06-29 21:21:52.635402469 +0200
+++ ./src/CMakeLists.txt	2020-06-29 21:53:59.387750730 +0200
@@ -306,9 +306,11 @@
     endif()
     include_directories(${LLVM_INCLUDE_DIRS})
     add_definitions(${LLVM_DEFINITIONS})
-    llvm_map_components_to_libnames(llvm_libs support core option)
+    #llvm_map_components_to_libnames(llvm_libs support core option)
+    llvm_map_components_to_libnames(llvm_libs support)
     target_compile_definitions(doxygen PRIVATE ${LLVM_DEFINITIONS})
-    set(CLANG_LIBS libclang clangTooling clangBasic clangLex ${llvm_libs})
+    #set(CLANG_LIBS libclang clangTooling clangBasic clangLex ${llvm_libs})
+    set(CLANG_LIBS libclang clang-cpp ${llvm_libs})
 endif()
 
 target_link_libraries(doxygen

Maybe someone wants the same, you can use the above as a start
Comment 2 Joakim Tjernlund 2020-06-30 08:37:00 UTC
Upstream doxygen bug:
https://github.com/doxygen/doxygen/issues/7883
Comment 3 Jeroen Roovers (RETIRED) gentoo-dev 2020-07-01 12:23:14 UTC
Please describe the actual error you're seeing and not just the solution.
Comment 4 Joakim Tjernlund 2020-07-19 11:38:30 UTC
Created attachment 649838 [details, diff]
Proposed patch

There is a PR(by me) upstream with this patch too
Comment 5 Joakim Tjernlund 2020-07-23 18:58:30 UTC
Fixed upstream in master, no release yet
Comment 6 Joakim Tjernlund 2020-07-31 11:47:53 UTC
Would be nice to get doxygen-1.8.18 with the clang-10 patch in tree so that
people don't need to keep clang-9 around
Comment 7 Joakim Tjernlund 2020-08-11 16:12:41 UTC
app-doc/doxygen-1.8.19 has been released which fixes this problem.
Comment 8 Joakim Tjernlund 2020-09-16 10:28:02 UTC
(In reply to Joakim Tjernlund from comment #7)
> app-doc/doxygen-1.8.19 has been released which fixes this problem.

Ping?
Comment 9 Joakim Tjernlund 2020-11-09 14:20:19 UTC
ping ?
Comment 10 Joakim Tjernlund 2020-11-11 17:58:05 UTC
app-doc/doxygen-1.8.20 is out
Comment 11 Joakim Tjernlund 2020-11-16 15:38:27 UTC
doxygen-1.8.20 build for me with LLVM=11 with this minor change:

@@ -27,7 +27,7 @@
 	dev-lang/perl
 	media-libs/libpng:0=
 	virtual/libiconv
-	clang? ( <sys-devel/clang-10:= )
+	clang? ( <sys-devel/clang-12:= )
 	dot? (
 		media-gfx/graphviz
 		media-libs/freetype
@@ -59,13 +59,12 @@
 PATCHES=(
 	"${FILESDIR}/${PN}-1.8.9.1-empty-line-sigsegv.patch" #454348
 	"${FILESDIR}/${PN}-1.8.16-link_with_pthread.patch"
-	"${FILESDIR}/${PN}-1.8.15-llvm7.patch" #666692
 	"${FILESDIR}/${PN}-1.8.17-ensure_static_support_libraries.patch"
 )
 
 DOCS=( LANGUAGE.HOWTO README.md )
 
-LLVM_MAX_SLOT=9
+LLVM_MAX_SLOT=11
 
 pkg_setup() {
 	use clang && llvm_pkg_setup
Comment 12 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2020-12-10 08:56:43 UTC
Maintainer ping.
Comment 13 Larry the Git Cow gentoo-dev 2020-12-10 19:14:06 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=9029f41ab66a980289f88eb31c7eb8cd15331416

commit 9029f41ab66a980289f88eb31c7eb8cd15331416
Author:     Matthias Maier <tamiko@gentoo.org>
AuthorDate: 2020-12-10 19:13:12 +0000
Commit:     Matthias Maier <tamiko@gentoo.org>
CommitDate: 2020-12-10 19:13:53 +0000

    app-doc/doxygen: add LLVM 10/11 compatibility patch
    
     * apply in revision bump and mark as stable
    
    Closes: https://bugs.gentoo.org/730170
    Package-Manager: Portage-3.0.11, Repoman-3.0.2
    RepoMan-Options: --force
    Signed-off-by: Matthias Maier <tamiko@gentoo.org>

 app-doc/doxygen/doxygen-1.8.17-r1.ebuild          | 137 ++++++++++++++++++++++
 app-doc/doxygen/files/doxygen-1.8.17-llvm10.patch |  42 +++++++
 2 files changed, 179 insertions(+)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=dc8c2f03346c169253098f4519780bf64f24dbb9

commit dc8c2f03346c169253098f4519780bf64f24dbb9
Author:     Matthias Maier <tamiko@gentoo.org>
AuthorDate: 2020-12-10 18:59:44 +0000
Commit:     Matthias Maier <tamiko@gentoo.org>
CommitDate: 2020-12-10 19:13:50 +0000

    app-doc/doxygen: version bump to 1.8.20
    
     * some QA warnings unresolved
     * remove LLVM-9 constraint
    
    Bug: https://bugs.gentoo.org/730170
    Package-Manager: Portage-3.0.11, Repoman-3.0.2
    Signed-off-by: Matthias Maier <tamiko@gentoo.org>

 app-doc/doxygen/Manifest              |   1 +
 app-doc/doxygen/doxygen-1.8.20.ebuild | 135 ++++++++++++++++++++++++++++++++++
 2 files changed, 136 insertions(+)