Have all JvmToolMixins share the same --jvm-options option registration.

Review Request #3684 - Created April 11, 2016 and submitted

Information
Benjy Weinberger
pants
Reviewers
pants-reviews
nhoward_tw, patricklaw, stuhood, zundel

It was pointlessly repeated everywhere. This change means that
(in a followup change) we can set a generally useful default once
for all tasks (e.g., memory settings, encoding and timezone), with
subclasses overriding as needed.

Also, took this opportunity to clarify in docstrings the distinction
between JvmTask and JvmToolTaskMixin.

CI passes: https://github.com/pantsbuild/pants/pull/3178

Nick Howard (Twitter)
Eric Ayers
Benjy Weinberger
Benjy Weinberger
Review request changed

Status: Closed (submitted)

Change Summary:

a68290cf748d4871fc80721c0b127320d5861895

Benjy Weinberger

Submitted! Thanks Nick and Eric.

Loading...