[cfe-dev] codechecker into clang/LLVM?
Anna Zaks via cfe-dev
cfe-dev at lists.llvm.org
Tue Dec 8 11:32:07 PST 2015
Hi Daniel,
I’ve looked at the project in a bit more detail and here are some other general comments.
It is very desirable to allow CodeChecker to work with scan-build (the current one and/or the python rewrite). This would ensure that all projects that we can analyze now (on all platforms such as OS X and Windows) will be supported. We agree that certain parts of current scan-build could be improved; however, getting a modular design will allow us to stage that process. Keeping code checker build interposition separate from the current scan-build would cause fragmentation between the users and developers. Is it possible the integrate CodeChecker with scan-build?
Is it possible to have something like "quick check" mode but with the enhanced HTML viewer? The database is needed for bug tracking, but requiring users to set it up to view bugs does not seem necessary. The easier it is to run/setup the tool for basic usage the more people will use it!
We’d need to integrate the automated tests into lit and add documentation.
Suppression in code should be handled by the compiler/analyzer, hopefully, using a more familiar syntax. (We can discuss this separately later.)
All command line options should be compatible/make sense in the potential future world of whole project analysis. (This might already be the case.)
Thank you,
Anna.
> On Nov 12, 2015, at 3:44 PM, Anna Zaks via cfe-dev <cfe-dev at lists.llvm.org> wrote:
>
>>
>> On Nov 11, 2015, at 9:07 AM, Dániel Krupp <daniel.krupp at ericsson.com <mailto:daniel.krupp at ericsson.com>> wrote:
>>
>> Hi Anna,
>>
>> First, thanks for looking into this.
>> We are open to any suggestions that you feel necessary to get this accepted to LLVM/Clang as a bug-tracking solution…
>>
>> >- What would it take for this to replace scan-build?...
>> We’ve been mainly targeting (I mean test it on ) Linux, but Mac and Windows support can be easily added too.
>> Since the whole thing is in python, the only issue here could be the “build interposition”, as you pointed out. Other than that, it’s pretty much platform independent.
>>
>> Regarding the “bug interposition”: CodeChecker uses the standard clang JSON compilation database <http://clang.llvm.org/docs/JSONCompilationDatabase.html> format as an input which you can pass like this (CodeChecker check -l <build_log.json>).
>
> Do we have to use the JSON compilation database or can this be made to work with the existing scan-build?
>
>> You can generate this log, by any other tools, using bear <https://github.com/rizsotto/Bear> tool for example on Mac.
>
> CC-ing Laszlo who is working on a scan-build rewrite in Python. This is the rewrite I've mentioned in one of the previous emails. It is flexible so that it could use either build interposition (bear) or ccc-analyzer style build.
>
> Ideally, we would have a component that would have the parity in capabilities with scan-build (or better). Laszlo has made a lot of progress on this. We could use that component for interposition and have a bug management system on top of it.
>
>> Currently the built in logger (based on LD_PRELOAD) supports compilation logging on Linux smoothly. We could add a (bash script) based logger too, that would be platform independent.
>
> How would the bash script logger work?
>
>>
>> We can do some testing on windows, but any help testing on Mac is welcome (as we don’t use Macs).
>>
>> > Is licensing compatible?
>> It should be. Except for psycopg <http://initd.org/psycopg/license/> (the postgres database connector) we not relying on any GPL or LGPL stuff.
>>
>> If psycopg is a problem this could be replaced to another postgres connector, such as pg8000 <https://pypi.python.org/pypi/pg8000> with BSD license.
>>
>
> It is a problem. (I cannot test this until it's free of LGPL.) Good to hear that we can switch to using an alternate method.
>
>> I collected here the licenses of the dependencies. All dependencies are run-time dependencies (except for the thrift compiler) and are used without modification.
>>
>> Javascript dependencies
>> *codemirror (https://codemirror.net/ <https://codemirror.net/> MIT licence)
>> *jsplumb (community edition, MIT https://jsplumbtoolkit.com/license#community <https://jsplumbtoolkit.com/license#community>)
>> *marked (BSD like https://github.com/chjj/marked/blob/master/LICENSE <https://github.com/chjj/marked/blob/master/LICENSE>)
>> *dojotoolkit (new BSD license https://dojotoolkit.org/license.html <https://dojotoolkit.org/license.html>)
>>
>> Python dependencies
>> · Python2 <https://www.python.org/> (> 2.7) (Python Software Foundation ) https://www.python.org/download/releases/2.7/license/ <https://www.python.org/download/releases/2.7/license/>)
>> · Alembic <https://pypi.python.org/pypi/alembic> (>=0.8.2) (MIT)
>> · SQLAlchemy <http://www.sqlalchemy.org/> (> 1.0.2) (MIT)
>> · psycopg2 <http://initd.org/psycopg/> (> 2.5.4) (LGPL http://initd.org/psycopg/license/ <http://initd.org/psycopg/license/>)
>> Other dependencies
>> · Clang Static analyzer <http://clang-analyzer.llvm.org/>
>> Postgresql <http://www.postgresql.org/> (> 9.3.5) (BSD like http://www.postgresql.org/about/licence/ <http://www.postgresql.org/about/licence/>)
>> · Thrift (compilation dependency) (Apache v2.0 https://thrift.apache.org/ <https://thrift.apache.org/>)
>> · Bzip2 is used for test project only (can be removed) (BSD like http://www.bzip.org/ <http://www.bzip.org/>)
>>
>>
>> Could you suggest which dependencies are problematic? We will investigate how to replace those.
>
> I do not know if there are other licensing issues. They all look OK to me but I am not an expert.
>
> Ideally, we'd also want to smooth out the installation process as much as possible.
>
>> Could you help in testing and/or making it Mac compatible? I suggest first running it on a standard JSON build db using the –l option.
>>
>> Regards,
>> Daniel
>>
>>
>> From: ganna at apple.com <mailto:ganna at apple.com> [mailto:ganna at apple.com <mailto:ganna at apple.com>]
>> Sent: 2015. november 10. 19:20
>> To: Dániel Krupp
>> Cc: cfe-dev at lists.llvm.org <mailto:cfe-dev at lists.llvm.org>
>> Subject: Re: [cfe-dev] codechecker into clang/LLVM?
>>
>> Hi Daniel,
>>
>> Sorry for taking so long to reply!
>>
>> The clang static analyzer is definitely missing a bug tracking system and I believe this project has a good potential to fill that need. Here are a couple of concerns that immediately jump into mind:
>>
>> - What would it take for this to replace scan-build? Can scan-build be used instead of the interposition module you use? For example, can we control the build interposition method by some option and the bug tracking would be an add-on on top of that? I suspect that your solution does not work on all platforms that scan-build currently supports (Mac and Windows come to mind). That is the main concern here. There are also projects that might not build with the type of interposition you use. I am not sure if you are aware of the scan-build rewrite (in Python) effort, where all these issues were raised as well.
>>
>> - Is licensing compatible? The llvm codebase tries to stay clear of any dependencies on GPL or LGPL licenses because there are companies who are involved with the project and cannot use software tainted with those licenses.
>>
>> - The list of dependencies is large, which is a concern if this was to replace scan-build.
>>
>> Anna.
>>
>> On Oct 22, 2015, at 7:14 AM, Dániel Krupp via cfe-dev <cfe-dev at lists.llvm.org <mailto:cfe-dev at lists.llvm.org>> wrote:
>>
>> Hello All,
>>
>> Scan-build, the current bug viewer Clang Static Analyzer front-end tool has some scalability issues and limitations.
>> For example, scan-build creates static HTML reports, storing whole source files as many times as they are included in a report.
>> Incremental bug reporting (show only new bugs compared to a baseline) and false positive suppression is not supported either.
>>
>> To address these issues, back in July we published CodeChecker on GitHub ( https://github.com/Ericsson/codechecker <https://github.com/Ericsson/codechecker> ),
>> a new defect storage and management infrastructure for Clang Static Analyzer (written in python). We also gave a talk about this in Euro LLVM 2015 (http://llvm.org/devmtg/2015-04/ <http://llvm.org/devmtg/2015-04/>).
>>
>> The most important features are the following:
>> - scalable dynamic web based defect viewer (instead of static html)
>> - a new command line tool for analyzing projects which is usable in CI scripts
>> - a PostgreSQL based defect storage & management
>> - incremental bug reporting (show only new bugs compared to a baseline)
>> - suppression of false positives
>> - better integration with build systems (through the LD_PRELOAD mechanism)
>> - Apache Thrift API based server-client model for storing bugs and viewing results.
>> - It is possible to connect multiple bug viewers. Currently a web-based viewer and a command line viewer are provided.
>>
>> Since its publication we have fixed many errors, addressed user-feedbacks and now I think it is mature enough.
>>
>>
>> We could release the tool under LLVM license.
>>
>> If you agree, this tool could be part of the llvm/clang source tree, possibly besides scan-build (or a separate llvm repository?).
>> I am not sure about the official process.
>> Can anyone help with this?
>>
>> Regards,
>> Daniel
>> _______________________________________________
>> cfe-dev mailing list
>> cfe-dev at lists.llvm.org <mailto:cfe-dev at lists.llvm.org>
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev <http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev>
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at lists.llvm.org <mailto:cfe-dev at lists.llvm.org>
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev <http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20151208/b2daf3c4/attachment.html>
More information about the cfe-dev
mailing list