Merge pull request #820 from brendanzagaeski/master

[mono.git] / mcs / class / Microsoft.Build.Tasks / Microsoft.Build.Tasks / ResolveAssemblyReference.cs

diff --git a/mcs/class/Microsoft.Build.Tasks/Microsoft.Build.Tasks/ResolveAssemblyReference.cs b/mcs/class/Microsoft.Build.Tasks/Microsoft.Build.Tasks/ResolveAssemblyReference.cs
index 730216cf8c12fa5e020ba7cf9cdab06214095098..8ac434bb9a418471e8338bf738f43155fdc411fa 100644 (file)
--- a/mcs/class/Microsoft.Build.Tasks/Microsoft.Build.Tasks/ResolveAssemblyReference.cs
+++ b/mcs/class/Microsoft.Build.Tasks/Microsoft.Build.Tasks/ResolveAssemblyReference.cs
@@ -162,9 +162,6 @@ namespace Microsoft.Build.Tasks {
                                        Log.LogWarning ("Reference '{0}' not resolved", item.ItemSpec);
                                        assembly_resolver.LogSearchLoggerMessages (MessageImportance.Normal);
                                } else {
-                                       if (Environment.GetEnvironmentVariable ("XBUILD_LOG_REFERENCE_RESOLVER") != null)
-                                               assembly_resolver.LogSearchLoggerMessages (MessageImportance.Low);
-
                                        Log.LogMessage (MessageImportance.Low,
                                                        "\tReference {0} resolved to {1}. CopyLocal = {2}",
                                                        item.ItemSpec, resolved_ref.TaskItem,
@@ -174,6 +171,8 @@ namespace Microsoft.Build.Tasks {
                                                        "\tReference found at search path {0}",
                                                        resolved_ref.FoundInSearchPathAsString);
 
+                                       assembly_resolver.LogSearchLoggerMessages (MessageImportance.Low);
+
                                        if (TryAddNewReference (tempResolvedFiles, resolved_ref) &&
                                                !IsFromGacOrTargetFramework (resolved_ref) &&
                                                resolved_ref.FoundInSearchPath != SearchPath.PkgConfig) {
@@ -289,8 +288,7 @@ namespace Microsoft.Build.Tasks {
                                        continue;
                                }
 
-                               if (Environment.GetEnvironmentVariable ("XBUILD_LOG_REFERENCE_RESOLVER") != null)
-                                       assembly_resolver.LogSearchLoggerMessages (MessageImportance.Low);
+                               assembly_resolver.LogSearchLoggerMessages (MessageImportance.Low);
 
                                ResolvedReference rr = assembly_resolver.GetResolvedReference (item, item.ItemSpec, aname, true,
                                                SearchPath.RawFileName);
@@ -364,9 +362,6 @@ namespace Microsoft.Build.Tasks {
                        resolved_ref = ResolveReference (item, dependency_search_paths, false);
 
                        if (resolved_ref != null) {
-                               if (Environment.GetEnvironmentVariable ("XBUILD_LOG_REFERENCE_RESOLVER") != null)
-                                               assembly_resolver.LogSearchLoggerMessages (MessageImportance.Low);
-
                                Log.LogMessage (MessageImportance.Low, "\tReference {0} resolved to {1}.",
                                        aname, resolved_ref.TaskItem.ItemSpec);
 
@@ -374,6 +369,8 @@ namespace Microsoft.Build.Tasks {
                                                "\tReference found at search path {0}",
                                                resolved_ref.FoundInSearchPathAsString);
 
+                               assembly_resolver.LogSearchLoggerMessages (MessageImportance.Low);
+
                                if (resolved_ref.FoundInSearchPath == SearchPath.Directory) {
                                        // override CopyLocal with parent's val
                                        SetCopyLocal (resolved_ref.TaskItem, parent_copy_local);
@@ -389,14 +386,14 @@ namespace Microsoft.Build.Tasks {
                                } else {
                                        //gac or tgtfmwk
                                        Log.LogMessage (MessageImportance.Low,
-                                                       "\tThis is CopyLocal false as it is in the gac," +
+                                                       "\tThis is CopyLocal false as it is in the GAC," +
                                                        "target framework directory or provided by a package.");
 
                                        TryAddNewReference (tempResolvedFiles, resolved_ref);
                                }
                        } else {
-                               Log.LogWarning ("Reference '{0}' not resolved", aname);
-                               assembly_resolver.LogSearchLoggerMessages (MessageImportance.Normal);
+                               Log.LogMessage (MessageImportance.Low, "Could not resolve the assembly \"{0}\".", aname);
+                               assembly_resolver.LogSearchLoggerMessages (MessageImportance.Low);
                        }
 
                        return resolved_ref;
@@ -408,7 +405,7 @@ namespace Microsoft.Build.Tasks {
                                return;
 
                        foreach (string ext in allowedRelatedFileExtensions) {
-                               string rfile = filename + ext;
+                               string rfile = Path.ChangeExtension (filename, ext);
                                if (File.Exists (rfile)) {
                                        ITaskItem item = new TaskItem (rfile);
                                        SetCopyLocal (item, parent_copy_local);