Change JarPublish.fingerprint to JarPublish.entry_fingerprint to ensure task fingerprint can change

Review Request #3078 - Created Nov. 5, 2015 and submitted

Information
Nick Howard (Twitter)
pants
2502
4cc1efc...
Reviewers
pants-reviews
benjyw, jsirois, stuhood
While doing some debugging in intellij, I noticed that JarPublish's fingerprint method was hilighted with a warning that it overrode a parent with a different signature. This lead me to realize that its fingerprint is now going to be a function rather than a string containing a hash. That didn't seem very good.

This renames the JarPublish fingerprint method so it is no longer overriding the property

Saw that it doesn't override Task.fingerprint. CI away on the PR.

Stu Hood
John Sirois
Nick Howard (Twitter)
Review request changed

Status: Closed (submitted)

Change Summary:

submitted at https://github.com/pantsbuild/pants/commit/c9177d42755b0c6b9fbd75aad44009d4cc9693e3

Loading...