Skip to content

Commit 69bd75e

Browse files
committed
Merge branch 'main' into blazor-hang
* main: [One .NET] fix 'dotnet publish' with no arguments (dotnet#8137) [build] consider `$NUGET_PACKAGES` for `$(XAPackagesDir)` (dotnet#8136) Bump external/xamarin-android-tools from `44885bc` to `3cee10b` (dotnet#8121)
2 parents 33c9ac3 + 0cd963a commit 69bd75e

File tree

4 files changed

+19
-1
lines changed

4 files changed

+19
-1
lines changed

Configuration.props

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,7 @@
151151
<XAPlatformToolsVersion>34.0.1</XAPlatformToolsVersion>
152152
<XAIncludeProprietaryBits Condition="'$(XAIncludeProprietaryBits)' == ''">False</XAIncludeProprietaryBits>
153153
<XABundleToolVersion Condition="'$(XABundleToolVersion)' == ''">1.8.1</XABundleToolVersion>
154+
<XAPackagesDir Condition=" '$(XAPackagesDir)' == '' And '$(NUGET_PACKAGES)' != ''">$(NUGET_PACKAGES)</XAPackagesDir>
154155
<XAPackagesDir Condition=" '$(XAPackagesDir)' == '' And '$(HostOS)' == 'Windows'">$(userprofile)\.nuget\packages</XAPackagesDir>
155156
<XAPackagesDir Condition=" '$(XAPackagesDir)' == '' And '$(HostOS)' != 'Windows'">$(HOME)/.nuget/packages</XAPackagesDir>
156157
<PathSeparator>$([System.IO.Path]::PathSeparator)</PathSeparator>

external/xamarin-android-tools

src/Xamarin.Android.Build.Tasks/Microsoft.Android.Sdk/targets/Microsoft.Android.Sdk.DefaultProperties.targets

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,11 @@
8181
<UseCurrentRuntimeIdentifier>false</UseCurrentRuntimeIdentifier>
8282
<PublishTrimmed Condition=" '$(PublishTrimmed)' == '' and ('$(AndroidLinkMode)' == 'SdkOnly' or '$(AndroidLinkMode)' == 'Full') ">true</PublishTrimmed>
8383
<PublishTrimmed Condition=" '$(PublishTrimmed)' == '' and '$(Configuration)' == 'Release' and '$(AndroidLinkMode)' != 'None' ">true</PublishTrimmed>
84+
<!--
85+
This prevents an early error message during 'dotnet publish'.
86+
We handle $(SelfContained) in a custom way where it is forced to be true.
87+
-->
88+
<PublishSelfContained Condition=" '$(PublishSelfContained)' == '' ">false</PublishSelfContained>
8489
<AndroidLinkMode Condition=" '$(AndroidLinkMode)' == '' and '$(PublishTrimmed)' == 'true' ">SdkOnly</AndroidLinkMode>
8590
<AndroidLinkMode Condition=" '$(AndroidLinkMode)' == '' ">None</AndroidLinkMode>
8691
<!-- For compat with user code not marked trimmable, only trim opt-in by default. -->

src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/XASdkTests.cs

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -874,6 +874,18 @@ static bool IsPreviewFrameworkVersion (string targetFramework)
874874
&& XABuildConfig.AndroidLatestUnstableApiLevel != XABuildConfig.AndroidLatestStableApiLevel);
875875
}
876876

877+
[Test]
878+
public void DotNetPublishDefaultValues([Values (false, true)] bool isRelease)
879+
{
880+
var proj = new XamarinAndroidApplicationProject {
881+
IsRelease = isRelease
882+
};
883+
var builder = CreateDllBuilder ();
884+
builder.Save (proj);
885+
var dotnet = new DotNetCLI (Path.Combine (Root, builder.ProjectDirectory, proj.ProjectFilePath));
886+
Assert.IsTrue (dotnet.Publish (), "`dotnet publish` should succeed");
887+
}
888+
877889
[Test]
878890
public void DotNetPublish ([Values (false, true)] bool isRelease, [ValueSource(nameof(DotNetTargetFrameworks))] object[] data)
879891
{

0 commit comments

Comments
 (0)