Issue 153163002: [VSAddin] Don't require both 2010 and 2012 to be installed.

lgtm https://codereview.chromium.org/153163002/diff/1/visual_studio/NativeClientVSAddIn/InstallerResources/install.py File visual_studio/NativeClientVSAddIn/InstallerResources/install.py (right): https://codereview.chromium.org/153163002/diff/1/visual_studio/NativeClientVSAddIn/InstallerResources/install.py#newcode119 visual_studio/NativeClientVSAddIn/InstallerResources/install.py:119: if not os.path.exists(platform_root): This is superfluous now, right?

6 years, 10 months ago (2014-02-03 21:46:41 UTC) #2

Sam Clegg

https://codereview.chromium.org/153163002/diff/1/visual_studio/NativeClientVSAddIn/InstallerResources/install.py File visual_studio/NativeClientVSAddIn/InstallerResources/install.py (right): https://codereview.chromium.org/153163002/diff/1/visual_studio/NativeClientVSAddIn/InstallerResources/install.py#newcode119 visual_studio/NativeClientVSAddIn/InstallerResources/install.py:119: if not os.path.exists(platform_root): On 2014/02/03 21:46:41, binji wrote: > ...

6 years, 10 months ago (2014-02-03 22:44:54 UTC) #3

Sam Clegg

6 years, 10 months ago (2014-02-04 18:30:34 UTC) #4

Message was sent while issue was closed.

Committed patchset #1 manually as r1570 (presubmit successful).

Expand Messages | Collapse Messages