You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "Francois Saint-Jacques (JIRA)" <ji...@apache.org> on 2019/02/13 21:01:00 UTC
[jira] [Comment Edited] (ARROW-4340) [C++] Update IWYU version in
the `lint` dockerfile
[ https://issues.apache.org/jira/browse/ARROW-4340?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16767587#comment-16767587 ]
Francois Saint-Jacques edited comment on ARROW-4340 at 2/13/19 9:00 PM:
------------------------------------------------------------------------
Note that ubuntu provided iwyu (the package) is still linked with llvm-5 and the compiler used within arrow has no effect on this. Having support for the latest IWYU would imply compiling from source locally, unless someone can point to a binary release, I do not have time to add this for now as I fear I'll enter rabit whole.
was (Author: fsaintjacques):
Note that ubuntu provided iwyu (the package) is still linked with llvm-5 and the compiler used within arrow has no effect on this. Having support for the latest IWYU would imply compiling from source locally, unless someone can point to a binary release, I do not have time to add this for now as I fear I'll enter rabit wholes.
> [C++] Update IWYU version in the `lint` dockerfile
> --------------------------------------------------
>
> Key: ARROW-4340
> URL: https://issues.apache.org/jira/browse/ARROW-4340
> Project: Apache Arrow
> Issue Type: Improvement
> Components: C++
> Reporter: Krisztian Szucs
> Priority: Major
> Fix For: 0.13.0
>
>
> I was trying to cleanup the c++ imports based on the current docker-iwyu suggestions, but it requires to be customized (symbol maps and pragmas) more than it is currently. It'd also help a lot to use the latest IWYU version (see the changelog https://include-what-you-use.org/)
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)