mirrors/dotnet-installer
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Add --self-contained to publish.

Browse source 
This flows to the $(SelfContained) property added in https://github.com/dotnet/sdk/pull/1053
This commit is contained in:
Eric Erhardt 2017-04-03 18:09:50 -05:00
parent a616c04a38
commit ab15200500
6 changed files with 95 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
build/DependencyVersions.props
View file

@ -4,7 +4,7 @@
<CLI_SharedFrameworkVersion>2.0.0-preview1-001887-00</CLI_SharedFrameworkVersion>
<CLI_MSBuild_Version>15.2.0-preview-000047-02</CLI_MSBuild_Version>
<CLI_Roslyn_Version>2.0.0-rc4-61325-08</CLI_Roslyn_Version>
<CLI_NETSDK_Version>2.0.0-alpha-20170323-1</CLI_NETSDK_Version>
<CLI_NETSDK_Version>2.0.0-alpha-20170403-2</CLI_NETSDK_Version>
<CLI_NuGet_Version>4.3.0-beta1-2418</CLI_NuGet_Version>
<CLI_WEBSDK_Version>1.0.0-alpha-20170130-3-281</CLI_WEBSDK_Version>
<CLI_TestPlatform_Version>15.1.0-preview-20170316-05</CLI_TestPlatform_Version>

2
src/dotnet/commands/dotnet-publish/LocalizableStrings.cs
View file

@ -20,5 +20,7 @@ namespace Microsoft.DotNet.Tools.Publish
public const string FilterProjOption = "profile.xml";
public const string FilterProjOptionDescription = "The XML file that contains the list of packages to be excluded from publish step.";
public const string SelfContainedOptionDescription = "Publish the .NET Core runtime with your application so the runtime doesn't need to be installed on the target machine. Defaults to 'true' if a runtime identifier is specified.";
}
}

18
src/dotnet/commands/dotnet-publish/PublishCommandParser.cs
View file

@ -21,16 +21,26 @@ namespace Microsoft.DotNet.Cli
"-o|--output",
LocalizableStrings.OutputOptionDescription,
Accept.ExactlyOneArgument()
.With(name: LocalizableStrings.OutputOption)
.ForwardAsSingle(o => $"/p:PublishDir={o.Arguments.Single()}")),
.With(name: LocalizableStrings.OutputOption)
.ForwardAsSingle(o => $"/p:PublishDir={o.Arguments.Single()}")),
CommonOptions.ConfigurationOption(),
CommonOptions.VersionSuffixOption(),
Create.Option(
"--filter",
LocalizableStrings.FilterProjOptionDescription,
Accept.OneOrMoreArguments()
.With(name: LocalizableStrings.FilterProjOption)
.ForwardAsSingle(o => $"/p:FilterProjectFiles={string.Join("%3B", o.Arguments)}")),
.With(name: LocalizableStrings.FilterProjOption)
.ForwardAsSingle(o => $"/p:FilterProjectFiles={string.Join("%3B", o.Arguments)}")),
Create.Option(
"--self-contained",
LocalizableStrings.SelfContainedOptionDescription,
Accept.ZeroOrOneArgument()
.WithSuggestionsFrom("true", "false")
.ForwardAsSingle(o =>
{
string value = o.Arguments.Any() ? o.Arguments.Single() : "true";
return $"/p:SelfContained={value}";
})),
CommonOptions.VerbosityOption());
}
}

16
test/Microsoft.DotNet.Tools.Tests.Utilities/Commands/PublishCommand.cs
View file

@ -13,7 +13,8 @@ namespace Microsoft.DotNet.Tools.Test.Utilities
private string _output;
private string _runtime;
private List<string> _profileFilterProject = new List<string>();
private bool? _selfContained;
public PublishCommand WithFramework(string framework)
{
_framework = framework;
@ -37,12 +38,18 @@ namespace Microsoft.DotNet.Tools.Test.Utilities
return this;
}
public PublishCommand WithProFileProject(string profileproj)
public PublishCommand WithProfileProject(string profileproj)
{
_profileFilterProject.Add( $" --filter {profileproj}");
return this;
}
public PublishCommand WithSelfContained(bool value)
{
_selfContained = value;
return this;
}
public override CommandResult Execute(string args = "")
{
args = $"publish {BuildArgs()} {args}";
@ -61,7 +68,8 @@ namespace Microsoft.DotNet.Tools.Test.Utilities
FrameworkOption,
OutputOption,
ProfileProjOption,
RuntimeOption);
RuntimeOption,
SelfContainedOption);
}
private string FrameworkOption => string.IsNullOrEmpty(_framework) ? "" : $"-f {_framework}";
@ -71,5 +79,7 @@ namespace Microsoft.DotNet.Tools.Test.Utilities
private string RuntimeOption => string.IsNullOrEmpty(_runtime) ? "" : $"-r {_runtime}";
private string ProfileProjOption => string.Join(" ", _profileFilterProject);
private string SelfContainedOption => _selfContained.HasValue ? $"--self-contained:{_selfContained.Value}" : "";
}
}

8
test/dotnet-cache.Tests/GivenDotnetCachesAndPublishesProjects.cs
View file

@ -57,7 +57,7 @@ namespace Microsoft.DotNet.Cli.Publish.Tests
new PublishCommand()
.WithFramework(_tfm)
.WithWorkingDirectory(testProjectDirectory)
.WithProFileProject(profileFilter)
.WithProfileProject(profileFilter)
.Execute()
.Should().Pass();
@ -95,7 +95,7 @@ namespace Microsoft.DotNet.Cli.Publish.Tests
new PublishCommand()
.WithFramework(_tfm)
.WithWorkingDirectory(testProjectDirectory)
.WithProFileProject(profileFilter)
.WithProfileProject(profileFilter)
.Execute()
.Should().Pass();
@ -153,8 +153,8 @@ namespace Microsoft.DotNet.Cli.Publish.Tests
new PublishCommand()
.WithFramework(_tfm)
.WithWorkingDirectory(testProjectDirectory)
.WithProFileProject(profileFilter)
.WithProFileProject(profileFilter1)
.WithProfileProject(profileFilter)
.WithProfileProject(profileFilter1)
.Execute()
.Should().Pass();

65
test/dotnet-publish.Tests/GivenDotnetPublishPublishesProjects.cs
View file

@ -27,7 +27,7 @@ namespace Microsoft.DotNet.Cli.Publish.Tests
new RestoreCommand()
.WithWorkingDirectory(testProjectDirectory)
.Execute("/p:SkipInvalidConfigurations=true")
.Execute()
.Should().Pass();
new PublishCommand()
@ -62,9 +62,7 @@ namespace Microsoft.DotNet.Cli.Publish.Tests
.WithFramework("netcoreapp2.0")
.WithRuntime(rid)
.WithWorkingDirectory(testProjectDirectory)
//Workaround for https://github.com/dotnet/cli/issues/4501
.WithEnvironmentVariable("SkipInvalidConfigurations", "true")
.Execute("/p:SkipInvalidConfigurations=true")
.Execute()
.Should().Pass();
var configuration = Environment.GetEnvironmentVariable("CONFIGURATION") ?? "Debug";
@ -85,6 +83,65 @@ namespace Microsoft.DotNet.Cli.Publish.Tests
.And.HaveStdOutContaining("Hello World");
}
[Theory]
[InlineData(true)]
[InlineData(false)]
public void ItPublishesAnAppExplicitlySpecifyingSelfContained(bool selfContained)
{
var testAppName = "MSBuildTestApp";
var testInstance = TestAssets.Get(testAppName)
.CreateInstance($"PublishesSelfContained{selfContained}")
.WithSourceFiles()
.WithRestoreFiles();
var testProjectDirectory = testInstance.Root;
var rid = DotnetLegacyRuntimeIdentifiers.InferLegacyRestoreRuntimeIdentifier();
new PublishCommand()
.WithRuntime(rid)
.WithSelfContained(selfContained)
.WithWorkingDirectory(testProjectDirectory)
.Execute()
.Should().Pass();
var configuration = Environment.GetEnvironmentVariable("CONFIGURATION") ?? "Debug";
var outputPath = testProjectDirectory
.GetDirectory("bin", configuration, "netcoreapp2.0", rid, "publish")
.FullName;
var selfContainedProgram = Path.Combine(outputPath, $"{testAppName}{Constants.ExeSuffix}");
var selfContainedProgramFile = new FileInfo(selfContainedProgram);
TestCommand testCommand;
string testArgs;
if (selfContained)
{
selfContainedProgramFile.Should().Exist();
if (!RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
{
//Workaround for https://github.com/dotnet/corefx/issues/15516
Process.Start("chmod", $"u+x {selfContainedProgram}").WaitForExit();
}
testCommand = new TestCommand(selfContainedProgram);
testArgs = null;
}
else
{
selfContainedProgramFile.Should().NotExist();
testCommand = new DotnetCommand();
testArgs = Path.Combine(outputPath, $"{testAppName}.dll");
}
testCommand
.ExecuteWithCapturedOutput(testArgs)
.Should().Pass()
.And.HaveStdOutContaining("Hello World");
}
[Fact]
public void ItPublishesAppWhenRestoringToSpecificPackageDirectory()
{