Fix a bug where all OptionValue objects had the same underlying forwardings.

Review Request #1229 - Created Oct. 28, 2014 and submitted

Information
Benjy Weinberger
pants
f2583d2...
Reviewers
pants-reviews
dturner-tw, ity, patricklaw

This wasn't seen earlier because in practice all tasks migrated so far had
disjoint option names.

This was exposed when I migrated two unrelated tasks that happened to have
an identically named option (--skip).

CI passed.

Verified that this fixes the name collision issue that triggered its discovery in the first place.

Benjy Weinberger
Ity Kaul
David Turner
Benjy Weinberger
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted as 3d9a7ba8a5448ab345365659d15022bdf7c103ff.
Patrick Lawson

Ship It!

Loading...