error-prone icon indicating copy to clipboard operation
error-prone copied to clipboard

non-matched rules are reported as patched

Open delanym opened this issue 3 years ago • 4 comments
trafficstars

The plugin is reporting patches written to a file, but no file writes are occurring.

<arg>-Xplugin:ErrorProne -XepAllErrorsAsWarnings -XepPatchChecks:DeadException -XepPatchLocation:${project.build.directory}</arg>

These are also logged as warnings, but they result in writes to the patch file

<arg>-Xplugin:ErrorProne -XepAllErrorsAsWarnings -XepPatchChecks:MissingOverride -XepPatchLocation:${project.build.directory}</arg>

<arg>-Xplugin:ErrorProne -XepAllErrorsAsWarnings -XepPatchChecks:DefaultCharset -XepPatchLocation:${project.build.directory}</arg>

delanym avatar Jul 31 '22 18:07 delanym

Today none of the patches are applying, even though they are logged e.g. Changes were written to /git/walla/walla-maven-plugin/target/error-prone.patch. Please inspect the file and apply with: patch -p0 -u -i error-prone.patch

delanym avatar Aug 01 '22 06:08 delanym

@delanym this sounds similar to #1562, which was fixed in #3236. IIUC that change hasn't been released yet.

Stephan202 avatar Aug 01 '22 07:08 Stephan202

@Stephan202 it looks like it hasn't been merged either?? Im not sure its the same though. Patches to DefaultCharset were working, and today they aren't. Its very strange.

delanym avatar Aug 01 '22 07:08 delanym

@Stephan202 it looks like it hasn't been merged either??

It was merged; see ad0289b0251293d73c5ea62a5f165bba8577b3a7, which is on master. (The Github PR status is misleading; that's merely an artifact of the way in which Google syncs private and OSS code.)

Im not sure its the same though. Patches to DefaultCharset were working, and today they aren't. Its very strange.

Can't help further with that, I'm afraid. (A complete reproduction case would help.)

Stephan202 avatar Aug 01 '22 08:08 Stephan202