[GRADLE-3577] REPLACED file change type breaks compatibility with existing plugin Created: 26/Oct/16  Updated: 31/Oct/16  Resolved: 26/Oct/16

Status: Resolved
Project: Gradle
Affects Version/s: 3.1
Fix Version/s: 3.2-rc-2

Type: Bug
Reporter: Lóránt Pintér Assignee: Lóránt Pintér
Resolution: Fixed Votes: 0


 Description   

The change adds a new ChangeType value REPLACED (reference: 0a4a3bb). This leads to two issues:

FileChange has not been updated to provide a isReplaced() method so plugins cannot query for this kind of file change. Because OrderSensitiveTaskFilePropertyCompareStrategy has been changed to exclusively use the new status instead of MODIFIED some file changes cannot be detected by new task implementations.

However a bigger problems is that existing plugins that go through IncrementalTaskInputs.outofDate looking at isModified() will also always miss the files that use the new status. The only fix is to release a new version of the plugin (after the new API is fixed with a new isReplaced() method).

In order to be compatible, I believe Gradle should make these replaced files return true for both isModified() and isReplaced(). This require some more refactoring as FileChange only support a single ChangeType.

The Gradle plugin for Android is failing some of its tests on 3.1 because of this.



 Comments   
Comment by Lóránt Pintér [ 26/Oct/16 ]

Ended up reverting the change. REPLACED changes are now reported as a REMOVED and an ADDED change, just as with the unordered case.

Generated at Wed Jun 30 12:52:25 CDT 2021 using Jira 8.4.2#804003-sha1:d21414fc212e3af190e92c2d2ac41299b89402cf.