Skip to content
This repository has been archived by the owner on Apr 20, 2023. It is now read-only.

Commit

Permalink
code review feedback and clean-up
Browse files Browse the repository at this point in the history
  • Loading branch information
analogrelay committed Oct 7, 2015
1 parent 78858b0 commit 6395ffb
Show file tree
Hide file tree
Showing 9 changed files with 73 additions and 63 deletions.
60 changes: 60 additions & 0 deletions Microsoft.DotNet.Cli.sln
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,36 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Solution Items", "Solution
NuGet.Config = NuGet.Config
EndProjectSection
EndProject
Project("{8BB2217D-0F2D-49D1-97BC-3654ED321F3B}") = "Microsoft.DotNet.Cli.Utils", "src\Microsoft.DotNet.Cli.Utils\Microsoft.DotNet.Cli.Utils.xproj", "{61B7C351-C77D-43F7-B56F-BB1440178E10}"
EndProject
Project("{8BB2217D-0F2D-49D1-97BC-3654ED321F3B}") = "Microsoft.DotNet.Tools.Compiler", "src\Microsoft.DotNet.Tools.Compiler\Microsoft.DotNet.Tools.Compiler.xproj", "{0A309227-A9D8-4DDF-88DD-326B57B04378}"
EndProject
Project("{8BB2217D-0F2D-49D1-97BC-3654ED321F3B}") = "Microsoft.DotNet.Tools.DependencyResolver", "src\Microsoft.DotNet.Tools.DependencyResolver\Microsoft.DotNet.Tools.DependencyResolver.xproj", "{4298DF4A-E04C-4ED9-95F8-FBB7CB32721F}"
EndProject
Project("{8BB2217D-0F2D-49D1-97BC-3654ED321F3B}") = "Microsoft.DotNet.Tools.Publish", "src\Microsoft.DotNet.Tools.Publish\Microsoft.DotNet.Tools.Publish.xproj", "{391D6AD6-0FD5-4E1E-A4C8-6B5439D197B5}"
EndProject
Project("{8BB2217D-0F2D-49D1-97BC-3654ED321F3B}") = "Microsoft.DotNet.Tools.Run", "src\Microsoft.DotNet.Tools.Run\Microsoft.DotNet.Tools.Run.xproj", "{8B23B5EC-6740-447C-92D9-1F10692F232D}"
EndProject
Project("{8BB2217D-0F2D-49D1-97BC-3654ED321F3B}") = "Microsoft.DotNet.Tools.SourceResolver", "src\Microsoft.DotNet.Tools.SourceResolver\Microsoft.DotNet.Tools.SourceResolver.xproj", "{7108E4FB-E2F1-4682-8464-1DD6EFB17035}"
EndProject
Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "samples", "samples", "{13F8C30C-1011-459C-82B2-0ACDD73EDA18}"
ProjectSection(SolutionItems) = preProject
scripts\dotnet = scripts\dotnet
scripts\dotnet-compile = scripts\dotnet-compile
scripts\dotnet-compile.cmd = scripts\dotnet-compile.cmd
scripts\dotnet-publish = scripts\dotnet-publish
scripts\dotnet-publish.cmd = scripts\dotnet-publish.cmd
scripts\dotnet-resolve-references = scripts\dotnet-resolve-references
scripts\dotnet-resolve-references.cmd = scripts\dotnet-resolve-references.cmd
scripts\dotnet-resolve-sources = scripts\dotnet-resolve-sources
scripts\dotnet-resolve-sources.cmd = scripts\dotnet-resolve-sources.cmd
scripts\dotnet-restore = scripts\dotnet-restore
scripts\dotnet-restore.cmd = scripts\dotnet-restore.cmd
scripts\dotnet-run = scripts\dotnet-run
scripts\dotnet-run.cmd = scripts\dotnet-run.cmd
scripts\dotnet.cmd = scripts\dotnet.cmd
EndProjectSection
EndProject
Global
GlobalSection(SolutionConfigurationPlatforms) = preSolution
Debug|Any CPU = Debug|Any CPU
Expand All @@ -23,11 +53,41 @@ Global
{60CF7E6C-D6C8-439D-B7B7-D8A27E29BE2C}.Debug|Any CPU.Build.0 = Debug|Any CPU
{60CF7E6C-D6C8-439D-B7B7-D8A27E29BE2C}.Release|Any CPU.ActiveCfg = Release|Any CPU
{60CF7E6C-D6C8-439D-B7B7-D8A27E29BE2C}.Release|Any CPU.Build.0 = Release|Any CPU
{61B7C351-C77D-43F7-B56F-BB1440178E10}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
{61B7C351-C77D-43F7-B56F-BB1440178E10}.Debug|Any CPU.Build.0 = Debug|Any CPU
{61B7C351-C77D-43F7-B56F-BB1440178E10}.Release|Any CPU.ActiveCfg = Release|Any CPU
{61B7C351-C77D-43F7-B56F-BB1440178E10}.Release|Any CPU.Build.0 = Release|Any CPU
{0A309227-A9D8-4DDF-88DD-326B57B04378}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
{0A309227-A9D8-4DDF-88DD-326B57B04378}.Debug|Any CPU.Build.0 = Debug|Any CPU
{0A309227-A9D8-4DDF-88DD-326B57B04378}.Release|Any CPU.ActiveCfg = Release|Any CPU
{0A309227-A9D8-4DDF-88DD-326B57B04378}.Release|Any CPU.Build.0 = Release|Any CPU
{4298DF4A-E04C-4ED9-95F8-FBB7CB32721F}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
{4298DF4A-E04C-4ED9-95F8-FBB7CB32721F}.Debug|Any CPU.Build.0 = Debug|Any CPU
{4298DF4A-E04C-4ED9-95F8-FBB7CB32721F}.Release|Any CPU.ActiveCfg = Release|Any CPU
{4298DF4A-E04C-4ED9-95F8-FBB7CB32721F}.Release|Any CPU.Build.0 = Release|Any CPU
{391D6AD6-0FD5-4E1E-A4C8-6B5439D197B5}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
{391D6AD6-0FD5-4E1E-A4C8-6B5439D197B5}.Debug|Any CPU.Build.0 = Debug|Any CPU
{391D6AD6-0FD5-4E1E-A4C8-6B5439D197B5}.Release|Any CPU.ActiveCfg = Release|Any CPU
{391D6AD6-0FD5-4E1E-A4C8-6B5439D197B5}.Release|Any CPU.Build.0 = Release|Any CPU
{8B23B5EC-6740-447C-92D9-1F10692F232D}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
{8B23B5EC-6740-447C-92D9-1F10692F232D}.Debug|Any CPU.Build.0 = Debug|Any CPU
{8B23B5EC-6740-447C-92D9-1F10692F232D}.Release|Any CPU.ActiveCfg = Release|Any CPU
{8B23B5EC-6740-447C-92D9-1F10692F232D}.Release|Any CPU.Build.0 = Release|Any CPU
{7108E4FB-E2F1-4682-8464-1DD6EFB17035}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
{7108E4FB-E2F1-4682-8464-1DD6EFB17035}.Debug|Any CPU.Build.0 = Debug|Any CPU
{7108E4FB-E2F1-4682-8464-1DD6EFB17035}.Release|Any CPU.ActiveCfg = Release|Any CPU
{7108E4FB-E2F1-4682-8464-1DD6EFB17035}.Release|Any CPU.Build.0 = Release|Any CPU
EndGlobalSection
GlobalSection(SolutionProperties) = preSolution
HideSolutionNode = FALSE
EndGlobalSection
GlobalSection(NestedProjects) = preSolution
{60CF7E6C-D6C8-439D-B7B7-D8A27E29BE2C} = {ED2FE3E2-F7E7-4389-8231-B65123F2076F}
{61B7C351-C77D-43F7-B56F-BB1440178E10} = {ED2FE3E2-F7E7-4389-8231-B65123F2076F}
{0A309227-A9D8-4DDF-88DD-326B57B04378} = {ED2FE3E2-F7E7-4389-8231-B65123F2076F}
{4298DF4A-E04C-4ED9-95F8-FBB7CB32721F} = {ED2FE3E2-F7E7-4389-8231-B65123F2076F}
{391D6AD6-0FD5-4E1E-A4C8-6B5439D197B5} = {ED2FE3E2-F7E7-4389-8231-B65123F2076F}
{8B23B5EC-6740-447C-92D9-1F10692F232D} = {ED2FE3E2-F7E7-4389-8231-B65123F2076F}
{7108E4FB-E2F1-4682-8464-1DD6EFB17035} = {ED2FE3E2-F7E7-4389-8231-B65123F2076F}
EndGlobalSection
EndGlobal
44 changes: 0 additions & 44 deletions dotnet-cli.sln

This file was deleted.

7 changes: 3 additions & 4 deletions src/Microsoft.DotNet.Tools.Compiler/Program.cs
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
using System;
using System.IO;
using System.Linq;
using System.Reflection;
using Microsoft.Dnx.Runtime.Common.CommandLine;
using Microsoft.DotNet.Cli.Utils;

Expand Down Expand Up @@ -110,10 +109,10 @@ private static int Compile(string path, string framework, DirectoryInfo projectD
// Build csc args
var cscArgs = new[]
{
"/nostdlib",
$"/out:{Path.Combine(outputPath, projectDir.Name + ".dll")}"
"-nostdlib",
$"-out:{Path.Combine(outputPath, projectDir.Name + ".dll")}"
}
.Concat(references.Select(r => $"/r:{r}"))
.Concat(references.Select(r => $"-r:{r}"))
.Concat(sources);
var rsp = Path.Combine(outputPath, "csc.rsp");
if(File.Exists(rsp))
Expand Down
5 changes: 3 additions & 2 deletions src/Microsoft.DotNet.Tools.DependencyResolver/Program.cs
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,16 @@
using Microsoft.Dnx.Runtime.Common.CommandLine;
using NuGet.Frameworks;

namespace DotNet.Tools.DependencyResolver
namespace Microsoft.DotNet.Tools.DependencyResolver
{
public class Program
{
public void Main(string[] args)
{
var app = new CommandLineApplication();
app.HelpOption("-h|--help");
app.Name = "dotnet resolve-references";
app.Description = "Resolves the absolute path of all dependencies for a project";
app.HelpOption("-h|--help");

var packages = app.Option("-p|--packages <PACKAGES_DIRECTORY>", "Path to the directories containing packages to resolve.", CommandOptionType.MultipleValue);
var framework = app.Option("-f|--framework <FRAMEWORK_NAME>", "The framework to resolve dependencies for.", CommandOptionType.SingleValue);
Expand Down
3 changes: 1 addition & 2 deletions src/Microsoft.DotNet.Tools.DependencyResolver/Resolver.cs
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
using System;
using System.Collections.Generic;
using System.IO;
using System.Threading.Tasks;
using Newtonsoft.Json.Linq;

namespace DotNet.Tools.DependencyResolver
namespace Microsoft.DotNet.Tools.DependencyResolver
{
public static class Resolver
{
Expand Down
4 changes: 1 addition & 3 deletions src/Microsoft.DotNet.Tools.Publish/Program.cs
Original file line number Diff line number Diff line change
@@ -1,11 +1,9 @@
using System;
using System.IO;
using System.Linq;
using System.Reflection;
using Microsoft.Dnx.Runtime.Common.CommandLine;
using Microsoft.DotNet.Cli.Utils;

namespace Microsoft.DotNet.Tools.Compiler
namespace Microsoft.DotNet.Tools.Publish
{
public class Program
{
Expand Down
4 changes: 1 addition & 3 deletions src/Microsoft.DotNet.Tools.Run/Program.cs
Original file line number Diff line number Diff line change
@@ -1,12 +1,10 @@
using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Reflection;
using Microsoft.Dnx.Runtime.Common.CommandLine;
using Microsoft.DotNet.Cli.Utils;

namespace Microsoft.DotNet.Tools.Compiler
namespace Microsoft.DotNet.Tools.Run
{
public class Program
{
Expand Down
5 changes: 3 additions & 2 deletions src/Microsoft.DotNet.Tools.SourceResolver/Program.cs
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,16 @@
using System.IO;
using Microsoft.Dnx.Runtime.Common.CommandLine;

namespace DotNet.Tools.DependencyResolver
namespace Microsoft.DotNet.Tools.SourceResolver
{
public class Program
{
public void Main(string[] args)
{
var app = new CommandLineApplication();
app.HelpOption("-h|--help");
app.Name = "dotnet resolve-sources";
app.Description = "Resolves the absolute path of all source files used by a project";
app.HelpOption("-h|--help");

var output = app.Option("-o|--output <OUTPUT_FILE>", "The path in which to write the output file (formatted as text with one line per dependency)", CommandOptionType.SingleValue);
var project = app.Argument("PROJECT", "The project to resolve. A directory or a path to a project.json may be used. Defaults to the current directory");
Expand Down
4 changes: 1 addition & 3 deletions src/Microsoft.DotNet.Tools.SourceResolver/Resolver.cs
Original file line number Diff line number Diff line change
@@ -1,11 +1,9 @@
using System;
using System.Collections.Generic;
using System.IO;
using System.Threading.Tasks;
using Microsoft.Dnx.Runtime;
using Newtonsoft.Json.Linq;

namespace DotNet.Tools.DependencyResolver
namespace Microsoft.DotNet.Tools.SourceResolver
{
public static class Resolver
{
Expand Down

0 comments on commit 6395ffb

Please sign in to comment.