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

Review Request #1229 — Created Oct. 27, 2014 and submitted

benjyw
pants
f2583d2...
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.

BE
IT
  1. Ship It!

  2. 
      
DT
  1. Ship It!

  2. 
      
BE
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted as 3d9a7ba8a5448ab345365659d15022bdf7c103ff.
PA
  1. Ship It!

  2. 
      
Loading...