search

facebook / infer

A static analyzer for Java, C, C++, and Objective-C
http://fbinfer.com/
MIT License
14.93k stars 2.01k forks source link

[java] Bump maven-compiler-plugin version in capture profile #1784

Closed kohlschuetter closed 1 year ago

kohlschuetter commented 1 year ago

The "infer-capture" Maven profile specified a version for maven-compiler-plugin to fix #990. This may interfere with the inspected project's configuration.

In this particular case, version 3.8.0 was specified. This prevented compilation of some projects, resulting in the ominous error 'Cannot invoke "java.lang.Throwable.getCause()" because "cause" is null'.

Removing the declared version for maven-compiler-plugin is a potential fix, which would automatically use the version defined in the origin Maven project. However that may break Maven 2.x projects because "forceJavacCompilerUse" is only available since version 3.0.

Bumping version to the latest available 3.11.0 instead.

Fixes https://github.com/facebook/infer/issues/1783

facebook-github-bot commented 1 year ago

@geralt-encore has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot commented 1 year ago

@geralt-encore merged this pull request in facebook/infer@9304c05a1dd672effb9fcbe92a36c18d12e2ca99.