[ 
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)

Reply via email to