Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 666692

Summary: app-doc/doxygen-1.8.14-r1 wont compile with clang useflag after an upgrade to llvm/clang-7
Product: Gentoo Linux Reporter: mael <mae.lippert>
Component: Current packagesAssignee: Matthias Maier <tamiko>
Status: RESOLVED FIXED    
Severity: normal CC: awkravchuk, candrews, drmccoy, dschridde+gentoobugs, jstein, musv, steven
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: Build Log
"emerge --info" output
Proposed patch

Description mael 2018-09-21 12:06:33 UTC
Created attachment 547478 [details]
Build Log

it seems that after an upgrade to llvm/clang-7 doxygen-1.8.14-r1 would not longer successfully compile with set clang useflag. emerge --info and build.log will be provided!
Comment 1 mael 2018-09-21 12:07:57 UTC
Created attachment 547480 [details]
"emerge --info" output
Comment 2 Stephen Newell 2018-09-22 04:39:23 UTC
Created attachment 547512 [details, diff]
Proposed patch

Two issues:
  1) a missing include in clangparser.cpp
  2) Missing libraries while linking against clang/llvm-7.0.0

Tested by adding the patch to /etc/portage/patches/app-doc/doxygen-1.8.14-r1/ and by compiling manually.  While compiling manually, I also verified doxygen builds against llvm-6 (I didn't test this through portage).  

I didn't test against anything older than llvm-6.0.1-r1.
Comment 3 Mark Wright gentoo-dev 2018-09-22 07:12:16 UTC
(In reply to Stephen Newell from comment #2)
> Created attachment 547512 [details, diff] [details, diff]
> Proposed patch
> 
> Two issues:
>   1) a missing include in clangparser.cpp
>   2) Missing libraries while linking against clang/llvm-7.0.0
> 
> Tested by adding the patch to
> /etc/portage/patches/app-doc/doxygen-1.8.14-r1/ and by compiling manually. 
> While compiling manually, I also verified doxygen builds against llvm-6 (I
> didn't test this through portage).  
> 
> I didn't test against anything older than llvm-6.0.1-r1.

Thanks, I reproduced the problem building app-doc/doxygen-1.8.14-r1 with sys-devel/clang-7.0.0.  Then I applied this patch and it builds fine.
Comment 4 Larry the Git Cow gentoo-dev 2018-10-02 02:14:27 UTC
The bug has been closed via the following commit(s):

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

commit 4126e1d5bc12eddbbafaab0e2fb6530526170500
Author:     Matthias Maier <tamiko@gentoo.org>
AuthorDate: 2018-10-02 02:10:59 +0000
Commit:     Matthias Maier <tamiko@gentoo.org>
CommitDate: 2018-10-02 02:13:51 +0000

    app-doc/doxygen: apply llvm patches, fix subslot
    
     - apply llvm patches, thanks to Stephen Newell for the patch.
     - add missing subslot operator
    
    Closes: https://bugs.gentoo.org/666692
    Closes: https://bugs.gentoo.org/657556
    Package-Manager: Portage-2.3.50, Repoman-2.3.11
    Signed-off-by: Matthias Maier <tamiko@gentoo.org>

 app-doc/doxygen/doxygen-1.8.14-r1.ebuild         |  5 +++--
 app-doc/doxygen/files/doxygen-1.8.14-llvm7.patch | 25 ++++++++++++++++++++++++
 2 files changed, 28 insertions(+), 2 deletions(-)