Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 666692 - app-doc/doxygen-1.8.14-r1 wont compile with clang useflag after an upgrade to llvm/clang-7
Summary: app-doc/doxygen-1.8.14-r1 wont compile with clang useflag after an upgrade to...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal with 2 votes (vote)
Assignee: Matthias Maier
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-09-21 12:06 UTC by mael
Modified: 2018-10-02 02:14 UTC (History)
7 users (show)

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


Attachments
Build Log (doxygen-build.log,176.28 KB, text/x-log)
2018-09-21 12:06 UTC, mael
Details
"emerge --info" output (emerge-info.txt,18.64 KB, text/plain)
2018-09-21 12:07 UTC, mael
Details
Proposed patch (bug-666692.patch,858 bytes, patch)
2018-09-22 04:39 UTC, Stephen Newell
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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(-)