This change is pipeline'd on top of the PNaCl VS integration.
lgtm, but make sure this compiles! http://codereview.chromium.org/11348050/diff/4001/visual_studio/NativeClientV... File visual_studio/NativeClientVSAddIn/NaCl.Build.CPPTasks/NaClLink.cs (right): http://codereview.chromium.org/11348050/diff/4001/visual_studio/NativeClientV... visual_studio/NativeClientVSAddIn/NaCl.Build.CPPTasks/NaClLink.cs:175: string dirname = Path.GetDirectoryName(GenerateFullPathToTool(); doesn't look like this will compile -- missing close paren http://codereview.chromium.org/11348050/diff/4001/visual_studio/NativeClientV... visual_studio/NativeClientVSAddIn/NaCl.Build.CPPTasks/NaClLink.cs:249: string nmfPath = Path.Combine(outputRoot, Path.ChangeExtension(ProjectName, ".nmf")); just over 100 chars... :-/ http://codereview.chromium.org/11348050/diff/4001/visual_studio/NativeClientV... visual_studio/NativeClientVSAddIn/NaCl.Build.CPPTasks/NaClLink.cs:292: protected override int ExecuteTool(string pathToTool, string responseFileCommands, string commandLineCommands) nit: wrap at 100 http://codereview.chromium.org/11348050/diff/4001/visual_studio/NativeClientV... visual_studio/NativeClientVSAddIn/NaCl.Build.CPPTasks/NaClLink.cs:296: Log.LogMessage(MessageImportance.High, pathToTool + " " + responseFileCommands + " " + commandLineCommands); nit: wrap at 100