Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1109)

Unified Diff: visual_studio/NativeClientVSAddIn/NaCl.Build.CPPTasks/NaClLink.cs

Issue 11375004: [NaCl Addin] Fix to PNaCl lib creation from MSVS (Closed) Base URL: http://nativeclient-sdk.googlecode.com/svn/trunk/src
Patch Set: Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: visual_studio/NativeClientVSAddIn/NaCl.Build.CPPTasks/NaClLink.cs
diff --git a/visual_studio/NativeClientVSAddIn/NaCl.Build.CPPTasks/NaClLink.cs b/visual_studio/NativeClientVSAddIn/NaCl.Build.CPPTasks/NaClLink.cs
index 8283afb4554ee0d1158ac632de0f0e96201032da..e0d6fe0b6f4cb34bb0fff6902fcf0d738ee47dc3 100644
--- a/visual_studio/NativeClientVSAddIn/NaCl.Build.CPPTasks/NaClLink.cs
+++ b/visual_studio/NativeClientVSAddIn/NaCl.Build.CPPTasks/NaClLink.cs
@@ -15,8 +15,6 @@ namespace NaCl.Build.CPPTasks
{
public class NaClLink : NaClToolTask
{
- public bool BuildingInIDE { get; set; }
-
/// <summary>
/// Property set only in PNaCl builds to signal that the translator
/// should be run post-link.
@@ -36,9 +34,6 @@ namespace NaCl.Build.CPPTasks
public bool TranslateX64 { get; set; }
[Required]
- public bool OutputCommandLine { get; set; }
-
- [Required]
public bool CreateNMF { get; set; }
[Required]
@@ -51,84 +46,14 @@ namespace NaCl.Build.CPPTasks
public string ToolchainName { get; set; }
[Required]
- public string Platform { get; set; }
-
- [Required]
public string CreateNMFPath { get; set; }
[Required]
- public virtual string OutputFile { get; set; }
-
- [Required]
public string PropertiesFile { get; set; }
[Required]
public string ConfigurationType { get; set; }
- protected override CanonicalTrackedOutputFiles OutputWriteTLog(ITaskItem[] inputs)
- {
- string path = Path.Combine(TlogDirectory, WriteTLogFilename);
- TaskItem item = new TaskItem(path);
- CanonicalTrackedOutputFiles trackedFiles =
- new CanonicalTrackedOutputFiles(new TaskItem[] { item });
-
- foreach (ITaskItem sourceItem in Sources)
- {
- //remove this entry associated with compiled source which is about to be recomputed
- trackedFiles.RemoveEntriesForSource(sourceItem);
-
- //add entry with updated information
- string upper = Path.GetFullPath(sourceItem.ItemSpec).ToUpperInvariant();
- trackedFiles.AddComputedOutputForSourceRoot(upper, OutputFile);
- }
-
- //output tlog
- trackedFiles.SaveTlog();
-
- return trackedFiles;
- }
-
- protected override void OutputReadTLog(ITaskItem[] compiledSources, CanonicalTrackedOutputFiles outputs)
- {
- string trackerPath = Path.GetFullPath(TlogDirectory + ReadTLogFilenames[0]);
-
- using (StreamWriter writer = new StreamWriter(trackerPath, false, Encoding.Unicode))
- {
- string sourcePath = "";
- foreach (ITaskItem source in Sources)
- {
- if (sourcePath != "")
- sourcePath += "|";
- sourcePath += Path.GetFullPath(source.ItemSpec).ToUpperInvariant();
- }
-
- writer.WriteLine("^" + sourcePath);
- foreach (ITaskItem source in Sources)
- {
- writer.WriteLine(Path.GetFullPath(source.ItemSpec).ToUpperInvariant());
- }
- writer.WriteLine(Path.GetFullPath(OutputFile).ToUpperInvariant());
- }
- }
-
- protected override void OutputCommandTLog(ITaskItem[] compiledSources)
- {
- using (StreamWriter writer = new StreamWriter(TLogCommandFile.GetMetadata("FullPath"), false, Encoding.Unicode))
- {
- string cmds = GenerateResponseFileCommands();
- string sourcePath = "";
- foreach (ITaskItem source in Sources)
- {
- if (sourcePath != "")
- sourcePath += "|";
- sourcePath += Path.GetFullPath(source.ItemSpec).ToUpperInvariant();
- }
-
- writer.WriteLine("^" + sourcePath);
- writer.WriteLine(cmds);
- }
- }
-
public NaClLink()
: base(new ResourceManager("NaCl.Build.CPPTasks.Properties.Resources", Assembly.GetExecutingAssembly()))
{
@@ -187,22 +112,8 @@ namespace NaCl.Build.CPPTasks
return true;
}
- private bool IsPNaCl()
- {
- return Platform.Equals("pnacl", StringComparison.OrdinalIgnoreCase);
- }
-
public override bool Execute()
{
- if (IsPNaCl())
- {
- if (!SDKUtilities.FindPython())
- {
- Log.LogError("PNaCl linking requires python in your executable path.");
- return false;
- }
- }
-
xamlParser = new XamlParser(PropertiesFile);
if (!Setup())
return false;
@@ -301,11 +212,6 @@ namespace NaCl.Build.CPPTasks
return base.ExecuteTool(pathToTool, responseFileCommands, commandLineCommands);
}
- protected override string GenerateFullPathToTool()
- {
- return this.ToolName;
- }
-
protected override Encoding ResponseFileEncoding
{
get
@@ -345,15 +251,5 @@ namespace NaCl.Build.CPPTasks
return BaseTool() + ".link.write.1.tlog";
}
}
-
- public virtual string PlatformToolset
- {
- get
- {
- return "GCC";
- }
- set
- {}
- }
}
}

Powered by Google App Engine
This is Rietveld 408576698