Merge 3.1.4xx branches to 5.0.100/master

eng/Version.Details.xml

@@ -33,13 +33,13 @@
       <Uri>https://github.com/dotnet/CliCommandLineParser</Uri>
       <Sha>0e89c2116ad28e404ba56c14d1c3f938caa25a01</Sha>
     </Dependency>
-    <Dependency Name="Microsoft.Build" Version="16.7.0-preview-20359-01">
+    <Dependency Name="Microsoft.Build" Version="16.7.0-preview-20360-03">
       <Uri>https://github.com/microsoft/msbuild</Uri>
-      <Sha>29609438b47acb32744163fa618a60f3aa478435</Sha>
+      <Sha>188921e2ffecf6c74dd4583cf01b62465fcbee3c</Sha>
     </Dependency>
-    <Dependency Name="Microsoft.Build.Localization" Version="16.7.0-preview-20359-01">
+    <Dependency Name="Microsoft.Build.Localization" Version="16.7.0-preview-20360-03">
       <Uri>https://github.com/microsoft/msbuild</Uri>
-      <Sha>29609438b47acb32744163fa618a60f3aa478435</Sha>
+      <Sha>188921e2ffecf6c74dd4583cf01b62465fcbee3c</Sha>
     </Dependency>
     <Dependency Name="Microsoft.FSharp.Compiler" Version="10.10.0-beta.20329.5">
       <Uri>https://github.com/dotnet/fsharp</Uri>

eng/Versions.props

@@ -83,10 +83,10 @@
   </PropertyGroup>
   <PropertyGroup>
     <!-- Dependencies from https://github.com/Microsoft/msbuild -->
-    <MicrosoftBuildPackageVersion>16.7.0-preview-20359-01</MicrosoftBuildPackageVersion>
+    <MicrosoftBuildPackageVersion>16.7.0-preview-20360-03</MicrosoftBuildPackageVersion>
     <MicrosoftBuildFrameworkPackageVersion>$(MicrosoftBuildPackageVersion)</MicrosoftBuildFrameworkPackageVersion>
     <MicrosoftBuildRuntimePackageVersion>$(MicrosoftBuildPackageVersion)</MicrosoftBuildRuntimePackageVersion>
-    <MicrosoftBuildLocalizationPackageVersion>16.7.0-preview-20359-01</MicrosoftBuildLocalizationPackageVersion>
+    <MicrosoftBuildLocalizationPackageVersion>16.7.0-preview-20360-03</MicrosoftBuildLocalizationPackageVersion>
     <MicrosoftBuildUtilitiesCorePackageVersion>$(MicrosoftBuildPackageVersion)</MicrosoftBuildUtilitiesCorePackageVersion>
     <MicrosoftBuildVersion>$(MicrosoftBuildPackageVersion)</MicrosoftBuildVersion>
     <MicrosoftBuildFrameworkVersion>$(MicrosoftBuildFrameworkPackageVersion)</MicrosoftBuildFrameworkVersion>

src/Cli/Microsoft.DotNet.Configurer/DotnetFirstRunConfiguration.cs

@@ -8,16 +8,21 @@ public class DotnetFirstRunConfiguration
         public bool GenerateAspNetCertificate { get; }
 
         public bool TelemetryOptout { get; }
+
         public bool AddGlobalToolsToPath { get; }
 
+        public bool NoLogo { get; }
+
         public DotnetFirstRunConfiguration(
             bool generateAspNetCertificate,
             bool telemetryOptout,
-            bool addGlobalToolsToPath)
+            bool addGlobalToolsToPath, 
+            bool nologo)
         {
             GenerateAspNetCertificate = generateAspNetCertificate;
             TelemetryOptout = telemetryOptout;
             AddGlobalToolsToPath = addGlobalToolsToPath;
+            NoLogo = nologo;
         }
     }
 }

src/Cli/Microsoft.DotNet.Configurer/DotnetFirstTimeUseConfigurer.cs

@@ -50,13 +50,17 @@ public void Configure()
 
             if (ShouldPrintFirstTimeUseNotice())
             {
-                PrintFirstTimeMessageWelcome();
-                if (ShouldPrintTelemetryMessageWhenFirstTimeUseNoticeIsEnabled())
+                if (!_dotnetFirstRunConfiguration.NoLogo)
                 {
-                    PrintTelemetryMessage();
+                    PrintFirstTimeMessageWelcome();
+                    if (ShouldPrintTelemetryMessageWhenFirstTimeUseNoticeIsEnabled())
+                    {
+                        PrintTelemetryMessage();
+                    }
+
+                    PrintFirstTimeMessageMoreInformation();
                 }
 
-                PrintFirstTimeMessageMoreInformation();
                 _firstTimeUseNoticeSentinel.CreateIfNotExists();
             }
 

src/Cli/dotnet/Program.cs

@@ -148,6 +148,8 @@ internal static int ProcessArgs(string[] args, ITelemetry telemetryClient = null
                           environmentProvider.GetEnvironmentVariableAsBool("DOTNET_CLI_TELEMETRY_OPTOUT", defaultValue: false);
                         bool addGlobalToolsToPath =
                             environmentProvider.GetEnvironmentVariableAsBool("DOTNET_ADD_GLOBAL_TOOLS_TO_PATH", defaultValue: true);
+                        bool nologo =
+                            environmentProvider.GetEnvironmentVariableAsBool("DOTNET_NOLOGO", defaultValue: false);
 
                         ReportDotnetHomeUsage(environmentProvider);
 
@@ -158,13 +160,14 @@ internal static int ProcessArgs(string[] args, ITelemetry telemetryClient = null
                             aspNetCertificateSentinel = new NoOpAspNetCertificateSentinel();
                             firstTimeUseNoticeSentinel = new NoOpFirstTimeUseNoticeSentinel();
                             toolPathSentinel = new NoOpFileSentinel(exists: false);
-                            isDotnetBeingInvokedFromNativeInstaller  = true;
+                            isDotnetBeingInvokedFromNativeInstaller = true;
                         }
 
                         var dotnetFirstRunConfiguration = new DotnetFirstRunConfiguration(
                             generateAspNetCertificate: generateAspNetCertificate,
                             telemetryOptout: telemetryOptout,
-                            addGlobalToolsToPath: addGlobalToolsToPath);
+                            addGlobalToolsToPath: addGlobalToolsToPath,
+                            nologo: nologo);
 
                         ConfigureDotNetForFirstTimeUse(
                             firstTimeUseNoticeSentinel,

src/Cli/dotnet/commands/dotnet-nuget/NuGetCommandParser.cs

@@ -54,7 +54,8 @@ public static Command NuGet() =>
                     Create.Option("-d|--disable-buffering", Parser.CompletionOnlyDescription),
                     Create.Option("-n|--no-symbols", Parser.CompletionOnlyDescription),
                     Create.Option("--no-service-endpoint", Parser.CompletionOnlyDescription),
-                    Create.Option("--interactive", Parser.CompletionOnlyDescription)
+                    Create.Option("--interactive", Parser.CompletionOnlyDescription),
+                    Create.Option("--skip-duplicate", Parser.CompletionOnlyDescription)
                     ));
     }
 }

src/Layout/redist/minimumMSBuildVersion

@@ -1 +1 @@
-16.5.0
+16.7.0

src/Microsoft.DotNet.MSBuildSdkResolver/Microsoft.DotNet.MSBuildSdkResolver.csproj

@@ -8,6 +8,8 @@
     <WarningsAsErrors>true</WarningsAsErrors>
     <StrongNameKeyId>MicrosoftAspNetCore</StrongNameKeyId>
     <IsPackable>true</IsPackable>
+    <!-- Create FileDefinitions items for ResolveHostfxrCopyLocalContent target -->
+    <EmitLegacyAssetsFileItems>true</EmitLegacyAssetsFileItems>
   </PropertyGroup>
 
   <ItemGroup>

src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.NET.Publish.targets

@@ -450,6 +450,7 @@ Copyright (c) .NET Foundation. All rights reserved.
                                         _ComputeResolvedFilesToPublishTypes
 
     Splits ResolvedFileToPublish items into 'PreserveNewest' and 'Always' buckets.
+    Then further splits those into 'Unbundled' buckets based on the single file setting.
     ============================================================
     -->
   <Target Name="_ComputeResolvedFilesToPublishTypes">
@@ -461,6 +462,19 @@ Copyright (c) .NET Foundation. All rights reserved.
       <_ResolvedFileToPublishAlways Include="@(ResolvedFileToPublish)"
                                     Condition="'%(ResolvedFileToPublish.CopyToPublishDirectory)'=='Always'" />
     </ItemGroup>
+
+    <ItemGroup>
+
+      <_ResolvedUnbundledFileToPublishPreserveNewest
+                    Include="@(_ResolvedFileToPublishPreserveNewest)"
+                    Condition="'$(PublishSingleFile)' != 'true' or 
+                               '%(_ResolvedFileToPublishPreserveNewest.ExcludeFromSingleFile)'=='true'" />
+
+      <_ResolvedUnbundledFileToPublishAlways
+              Include="@(_ResolvedFileToPublishAlways)"
+              Condition="'$(PublishSingleFile)' != 'true' or 
+                         '%(_ResolvedFileToPublishAlways.ExcludeFromSingleFile)'=='true'" />
+    </ItemGroup>
   </Target>
 
   <!--
@@ -541,7 +555,7 @@ Copyright (c) .NET Foundation. All rights reserved.
 
       <!-- Copy generated COM References. -->
       <ResolvedFileToPublish Include="@(ReferenceComWrappersToCopyLocal)">
-        <RelativePath>@(ReferenceComWrappersToCopyLocal->'%(Filename)%(Extension)')</RelativePath>
+        <RelativePath>%(Filename)%(Extension)</RelativePath>
         <CopyToPublishDirectory>PreserveNewest</CopyToPublishDirectory>
       </ResolvedFileToPublish>
 
@@ -1112,11 +1126,11 @@ Copyright (c) .NET Foundation. All rights reserved.
     ============================================================
                                             ComputeFilesCopiedToPublishDir
 
-    Gathers all the files that were copied to the publish directory.  This is used by wapproj and is required for back compat.
+    Gathers all the files that will be copied to the publish directory.  This is used by wapproj and is required for back compat.
     ============================================================
     -->
   <Target Name="ComputeFilesCopiedToPublishDir"
-          DependsOnTargets="ComputeResolvedFilesToPublishList;
+          DependsOnTargets="_ComputeResolvedFilesToPublishTypes;
                             _ComputeFilesToBundle">
     <ItemGroup>
       <FilesCopiedToPublishDir Include="@(ResolvedFileToPublish)"/>

src/Tests/Microsoft.DotNet.Configurer.UnitTests/GivenADotnetFirstTimeUseConfigurer.cs

@@ -46,7 +46,8 @@ public void It_does_not_print_the_first_time_use_notice_if_the_sentinel_exists()
                 (
                     generateAspNetCertificate: true,
                     telemetryOptout: false,
-                    addGlobalToolsToPath: true
+                    addGlobalToolsToPath: true,
+                    nologo: false
                 ),
                 _reporterMock.Object,
                 CliFallbackFolderPath,
@@ -72,7 +73,8 @@ public void It_prints_the_telemetry_if_the_sentinel_does_not_exist()
                 (
                     generateAspNetCertificate: true,
                     telemetryOptout: false,
-                    addGlobalToolsToPath: true
+                    addGlobalToolsToPath: true,
+                    nologo: false
                 ),
                 _reporterMock.Object,
                 CliFallbackFolderPath,
@@ -98,7 +100,8 @@ public void It_adds_the_tool_path_to_the_environment_if_the_tool_path_sentinel_d
                 (
                     generateAspNetCertificate: true,
                     telemetryOptout: false,
-                    addGlobalToolsToPath: true
+                    addGlobalToolsToPath: true,
+                    nologo: false
                 ),
                 _reporterMock.Object,
                 CliFallbackFolderPath,
@@ -124,7 +127,8 @@ public void It_does_not_add_the_tool_path_to_the_environment_if_the_tool_path_se
                 (
                     generateAspNetCertificate: true,
                     telemetryOptout: false,
-                    addGlobalToolsToPath: true
+                    addGlobalToolsToPath: true,
+                    nologo: false
                 ),
                 _reporterMock.Object,
                 CliFallbackFolderPath,
@@ -150,7 +154,8 @@ public void It_does_not_generate_the_aspnet_https_development_certificate_if_the
                 (
                     generateAspNetCertificate: true,
                     telemetryOptout: false,
-                    addGlobalToolsToPath: true
+                    addGlobalToolsToPath: true, 
+                    nologo: false
                 ),
                 _reporterMock.Object,
                 CliFallbackFolderPath,
@@ -175,7 +180,8 @@ public void It_does_not_generate_the_aspnet_https_development_certificate_when_t
                 (
                     generateAspNetCertificate: false,
                     telemetryOptout: false,
-                    addGlobalToolsToPath: true
+                    addGlobalToolsToPath: true,
+                    nologo: false
                 ),
                 _reporterMock.Object,
                 CliFallbackFolderPath,
@@ -200,7 +206,8 @@ public void It_generates_the_aspnet_https_development_certificate_if_the_sentine
                 (
                     generateAspNetCertificate: true,
                     telemetryOptout: false,
-                    addGlobalToolsToPath: true
+                    addGlobalToolsToPath: true,
+                    nologo: false
                 ),
                 _reporterMock.Object,
                 CliFallbackFolderPath,
@@ -224,7 +231,8 @@ public void It_adds_the_tool_path_to_the_environment_if_addGlobalToolsToPath_is_
                 (
                     generateAspNetCertificate: true,
                     telemetryOptout: false,
-                    addGlobalToolsToPath: true
+                    addGlobalToolsToPath: true,
+                    nologo: false
                 ),
                 _reporterMock.Object,
                 CliFallbackFolderPath,
@@ -247,7 +255,8 @@ public void It_does_not_add_the_tool_path_to_the_environment_if_addGlobalToolsTo
                 (
                     generateAspNetCertificate: true,
                     telemetryOptout: false,
-                    addGlobalToolsToPath: false
+                    addGlobalToolsToPath: false,
+                    nologo: false
                 ),
                 _reporterMock.Object,
                 CliFallbackFolderPath,

src/Tests/Microsoft.DotNet.Configurer.UnitTests/GivenADotnetFirstTimeUseConfigurerWIthStateSetup.cs

@@ -48,18 +48,27 @@ private void ResetObjectState()
         }
 
         [Theory]
-        [InlineData(false, false, false, Never, FirstRun, FirstRun, true, true)]
-        [InlineData(true, false, false, FirstRun, FirstRun, FirstRun, true, true)]
-        [InlineData(false, true, false, Never, FirstRun, Never, false, false)]
-        [InlineData(true, true, false, FirstRun, FirstRun, Never, false, false)]
-        [InlineData(false, false, true, Never, SecondRun, SecondRun, true, true)]
-        [InlineData(true, false, true, SecondRun, SecondRun, SecondRun, true, true)]
-        [InlineData(false, true, true, Never, SecondRun, Never, false, false)]
-        [InlineData(true, true, true, SecondRun, SecondRun, Never, false, false)]
+        [InlineData(false, false, false, false, Never, FirstRun, FirstRun, true, true)]
+        [InlineData(true, false, false, false, FirstRun, FirstRun, FirstRun, true, true)]
+        [InlineData(false, true, false, false, Never, FirstRun, Never, false, false)]
+        [InlineData(true, true, false, false, FirstRun, FirstRun, Never, false, false)]
+        [InlineData(false, false, true, false, Never, Never, Never, true, true)]
+        [InlineData(true, false, true, false, FirstRun, Never, Never, true, true)]
+        [InlineData(false, true, true, false, Never, Never, Never, false, false)]
+        [InlineData(true, true, true, false, FirstRun, Never, Never, false, false)]
+        [InlineData(false, false, false, true, Never, SecondRun, SecondRun, true, true)]
+        [InlineData(true, false, false, true, SecondRun, SecondRun, SecondRun, true, true)]
+        [InlineData(false, true, false, true, Never, SecondRun, Never, false, false)]
+        [InlineData(true, true, false, true, SecondRun, SecondRun, Never, false, false)]
+        [InlineData(false, false, true, true, Never, Never, Never, true, true)]
+        [InlineData(true, false, true, true, SecondRun, Never, Never, true, true)]
+        [InlineData(false, true, true, true, Never, Never, Never, false, false)]
+        [InlineData(true, true, true, true, SecondRun, Never, Never, false, false)]
         public void FlagsCombinationAndAction(
             // Inputs
             bool DOTNET_GENERATE_ASPNET_CERTIFICATE,
             bool DOTNET_CLI_TELEMETRY_OPTOUT,
+            bool DOTNET_NOLOGO,
             //   true to simulate install via installer. The first run is during installer,
             //   silent but has sudo permission
             //   false to simulate install via zip/tar.gz
@@ -75,11 +84,14 @@ bool telemetrySecondRunShouldBeEnabled
             ResetObjectState();
 
             _environmentProvider
-            .Setup(p => p.GetEnvironmentVariableAsBool("DOTNET_GENERATE_ASPNET_CERTIFICATE", It.IsAny<bool>()))
-            .Returns(DOTNET_GENERATE_ASPNET_CERTIFICATE);
+                .Setup(p => p.GetEnvironmentVariableAsBool("DOTNET_GENERATE_ASPNET_CERTIFICATE", It.IsAny<bool>()))
+                .Returns(DOTNET_GENERATE_ASPNET_CERTIFICATE);
             _environmentProvider
                 .Setup(p => p.GetEnvironmentVariableAsBool("DOTNET_CLI_TELEMETRY_OPTOUT", It.IsAny<bool>()))
                 .Returns(DOTNET_CLI_TELEMETRY_OPTOUT);
+            _environmentProvider
+                .Setup(p => p.GetEnvironmentVariableAsBool("DOTNET_NOLOGO", It.IsAny<bool>()))
+                .Returns(DOTNET_NOLOGO);
             _environmentProvider
                 .Setup(p => p.GetEnvironmentVariableAsBool("DOTNET_ADD_GLOBAL_TOOLS_TO_PATH", It.IsAny<bool>()))
                 .Returns(true);
@@ -206,6 +218,8 @@ private Telemetry RunConfigUsingMocks(bool isInstallerRun)
                 _environmentProviderObject.GetEnvironmentVariableAsBool("DOTNET_CLI_TELEMETRY_OPTOUT", false);
             bool addGlobalToolsToPath =
                 _environmentProviderObject.GetEnvironmentVariableAsBool("DOTNET_ADD_GLOBAL_TOOLS_TO_PATH", defaultValue: true);
+            bool nologo =
+                _environmentProviderObject.GetEnvironmentVariableAsBool("DOTNET_NOLOGO", defaultValue: false);
 
             IAspNetCertificateSentinel aspNetCertificateSentinel;
             IFirstTimeUseNoticeSentinel firstTimeUseNoticeSentinel;
@@ -233,7 +247,8 @@ private Telemetry RunConfigUsingMocks(bool isInstallerRun)
                  (
                      generateAspNetCertificate: generateAspNetCertificate,
                      telemetryOptout: telemetryOptout,
-                     addGlobalToolsToPath: addGlobalToolsToPath
+                     addGlobalToolsToPath: addGlobalToolsToPath,
+                     nologo: nologo
                  ),
                  reporter: _reporterMock,
                  cliFallbackFolderPath: CliFallbackFolderPath,