Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[net7.0] Update dependencies from dotnet/installer #15657

Merged
merged 27 commits into from
Aug 19, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
cec51a8
Update dependencies from https://github.com/dotnet/installer build 20…
dotnet-maestro[bot] Aug 8, 2022
801194f
Re-generate global.json
Aug 8, 2022
8ddd002
Update dependencies from https://github.com/dotnet/installer build 20…
dotnet-maestro[bot] Aug 9, 2022
9be4d21
Re-generate global.json
Aug 9, 2022
6a29fab
Update dependencies from https://github.com/dotnet/installer build 20…
dotnet-maestro[bot] Aug 10, 2022
e5b7391
Re-generate global.json
Aug 10, 2022
764d052
Update dependencies from https://github.com/dotnet/installer build 20…
dotnet-maestro[bot] Aug 11, 2022
e5d6c9c
Re-generate global.json
Aug 11, 2022
f07190d
Update dependencies from https://github.com/dotnet/installer build 20…
dotnet-maestro[bot] Aug 12, 2022
4bbc52b
Re-generate global.json
Aug 12, 2022
ad44963
Update dependencies from https://github.com/dotnet/installer build 20…
dotnet-maestro[bot] Aug 13, 2022
f7b083e
Re-generate global.json
Aug 13, 2022
415a2d2
Update dependencies from https://github.com/dotnet/installer build 20…
dotnet-maestro[bot] Aug 15, 2022
5dba5c1
Re-generate global.json
Aug 15, 2022
c47f5d3
Update dependencies from https://github.com/dotnet/installer build 20…
dotnet-maestro[bot] Aug 16, 2022
b351372
Re-generate global.json
Aug 16, 2022
dccf5bc
Update dependencies from https://github.com/dotnet/installer build 20…
dotnet-maestro[bot] Aug 17, 2022
3bfbdd1
Re-generate global.json
Aug 17, 2022
4bd22e3
[dotnet] Include the 'marshal-ilgen' component. Fixes #15668.
rolfbjarne Aug 17, 2022
4fa5a49
[dotnet] Enable serialization discovery in the linker. Fixes #15676.
rolfbjarne Aug 17, 2022
8f9e149
[tests] Make the dont link tests actually not link for macOS.
rolfbjarne Aug 18, 2022
639ac43
[tests] The 'trimmode copy' test needs an adjustment after recent lin…
rolfbjarne Aug 18, 2022
c108ddd
[dotnet] Don't set a default 'TrimMode' value.
rolfbjarne Aug 18, 2022
0add8a1
[tests] Workaround a bug in 'dotnet build'.
rolfbjarne Aug 18, 2022
1f8917f
[dotnet] Update expected bundle contents according to updated runtime.
rolfbjarne Aug 18, 2022
de7e738
[tests] Update expected error message texts.
rolfbjarne Aug 18, 2022
e2a34fb
Bump to RC 2.
rolfbjarne Aug 19, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Make.config
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,7 @@ endif
## change *first*, otherwise we'll produce builds with the same version from
## two different branches (which is very, very bad).
##
NUGET_HARDCODED_PRERELEASE_IDENTIFIER=rc.1
NUGET_HARDCODED_PRERELEASE_IDENTIFIER=rc.2
NUGET_HARDCODED_PRERELEASE_BRANCH=net7.0

# compute the alphanumeric version of branch names
Expand Down
2 changes: 0 additions & 2 deletions dotnet/targets/Xamarin.Shared.Sdk.props
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,6 @@
<_XamarinTaskAssembly Condition="'$(_PlatformName)' != 'macOS'">$(_XamarinSdkRootDirectory)\tools\msbuild\iOS\Xamarin.iOS.Tasks.dll</_XamarinTaskAssembly>
<_XamarinTaskAssembly Condition="'$(_PlatformName)' == 'macOS'">$(_XamarinSdkRootDirectory)\tools\msbuild\macOS\Xamarin.Mac.Tasks.dll</_XamarinTaskAssembly>
<SuppressTrimAnalysisWarnings Condition=" '$(SuppressTrimAnalysisWarnings)' == '' ">true</SuppressTrimAnalysisWarnings>
<!-- For compat with user code not marked trimmable, only trim opt-in by default. -->
<TrimMode Condition="'$(TrimMode)' == ''">partial</TrimMode>
<AfterMicrosoftNETSdkTargets>$(AfterMicrosoftNETSdkTargets);$(MSBuildThisFileDirectory)Microsoft.$(_PlatformName).Sdk.targets</AfterMicrosoftNETSdkTargets>

<!-- _XamarinSdkRoot is used by the existing MSBuild targets files -->
Expand Down
4 changes: 4 additions & 0 deletions dotnet/targets/Xamarin.Shared.Sdk.targets
Original file line number Diff line number Diff line change
Expand Up @@ -395,6 +395,7 @@
<_MonoComponent Include="hot_reload" Condition="'$(MtouchInterpreter)' == 'true'" />
<_MonoComponent Include="debugger" Condition="'$(_BundlerDebug)' == 'true'" />
<_MonoComponent Include="diagnostics_tracing" Condition="'$(_BundlerDebug)' == 'true'" />
<_MonoComponent Include="marshal-ilgen" />
</ItemGroup>
</Target>

Expand Down Expand Up @@ -554,6 +555,9 @@
<_ExtraTrimmerArgs Condition="('$(_PlatformName)' == 'iOS' Or '$(_PlatformName)' == 'tvOS') And '$(_SdkIsSimulator)' == 'true'">$(_ExtraTrimmerArgs) --feature ObjCRuntime.Runtime.Arch.IsSimulator true</_ExtraTrimmerArgs>
<_ExtraTrimmerArgs Condition="('$(_PlatformName)' == 'iOS' Or '$(_PlatformName)' == 'tvOS') And '$(_SdkIsSimulator)' != 'true'">$(_ExtraTrimmerArgs) --feature ObjCRuntime.Runtime.Arch.IsSimulator false</_ExtraTrimmerArgs>

<!-- Enable serialization discovery. Ref: https://github.com/xamarin/xamarin-macios/issues/15676 -->
<_ExtraTrimmerArgs>$(_ExtraTrimmerArgs) --enable-serialization-discovery</_ExtraTrimmerArgs>

<!-- We always want the linker to process debug symbols, even when building in Release mode, because the AOT compiler uses the managed debug symbols to output DWARF debugging symbols -->
<TrimmerRemoveSymbols Condition="'$(TrimmerRemoveSymbols)' == ''">false</TrimmerRemoveSymbols>

Expand Down
16 changes: 8 additions & 8 deletions eng/Version.Details.xml
Original file line number Diff line number Diff line change
@@ -1,21 +1,21 @@
<Dependencies>
<ProductDependencies>
<Dependency Name="Microsoft.Dotnet.Sdk.Internal" Version="7.0.100-rc.1.22405.9">
<Dependency Name="Microsoft.Dotnet.Sdk.Internal" Version="7.0.100-rc.2.22417.1">
<Uri>https://github.com/dotnet/installer</Uri>
<Sha>785bddf6a55f6e826f92a46e0464c01ff41481c1</Sha>
<Sha>f3da421a6e81382d6d2b25b236697f33e210092c</Sha>
</Dependency>
<Dependency Name="Microsoft.NET.ILLink.Tasks" Version="7.0.100-1.22377.1" CoherentParentDependency="Microsoft.Dotnet.Sdk.Internal">
<Dependency Name="Microsoft.NET.ILLink.Tasks" Version="7.0.100-1.22415.4" CoherentParentDependency="Microsoft.Dotnet.Sdk.Internal">
<Uri>https://github.com/dotnet/linker</Uri>
<Sha>f09bacf09ef10b61cf9f19825f8782171a816dab</Sha>
<Sha>6252a2194dd32911db2c0669fc818555687d5570</Sha>
</Dependency>
<Dependency Name="Microsoft.NETCore.App.Ref" Version="7.0.0-rc.1.22403.8" CoherentParentDependency="Microsoft.Dotnet.Sdk.Internal">
<Dependency Name="Microsoft.NETCore.App.Ref" Version="7.0.0-rc.1.22411.12" CoherentParentDependency="Microsoft.Dotnet.Sdk.Internal">
<Uri>https://github.com/dotnet/runtime</Uri>
<Sha>26a71c61fbda229f151afb14e274604b4926df5c</Sha>
<Sha>f52d8c59bb49360eb2cbeeb863c5856ebd62adda</Sha>
</Dependency>
<!-- This is required for our test apps to build; in some cases Microsoft.AspNetCore.App is pulled in, and when building test apps the build needs to be able to resolve that -->
<Dependency Name="Microsoft.AspNetCore.App.Ref" Version="7.0.0-rc.1.22404.6" CoherentParentDependency="Microsoft.Dotnet.Sdk.Internal">
<Dependency Name="Microsoft.AspNetCore.App.Ref" Version="7.0.0-rc.1.22415.4" CoherentParentDependency="Microsoft.Dotnet.Sdk.Internal">
<Uri>https://github.com/dotnet/aspnetcore</Uri>
<Sha>80ea57377a90fa2750f46662be3642b59ad0feed</Sha>
<Sha>0438e7ec04808230c67b00caa3d584bd2b991f57</Sha>
</Dependency>
</ProductDependencies>
<ToolsetDependencies>
Expand Down
6 changes: 3 additions & 3 deletions eng/Versions.props
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
<Project>
<!--Package versions-->
<PropertyGroup>
<MicrosoftDotnetSdkInternalPackageVersion>7.0.100-rc.1.22405.9</MicrosoftDotnetSdkInternalPackageVersion>
<MicrosoftNETILLinkTasksPackageVersion>7.0.100-1.22377.1</MicrosoftNETILLinkTasksPackageVersion>
<MicrosoftDotnetSdkInternalPackageVersion>7.0.100-rc.2.22417.1</MicrosoftDotnetSdkInternalPackageVersion>
<MicrosoftNETILLinkTasksPackageVersion>7.0.100-1.22415.4</MicrosoftNETILLinkTasksPackageVersion>
<MicrosoftDotNetBuildTasksFeedPackageVersion>6.0.0-beta.21212.6</MicrosoftDotNetBuildTasksFeedPackageVersion>
<MicrosoftNETILStripTasksPackageVersion>6.0.0-rc.2.21468.3</MicrosoftNETILStripTasksPackageVersion>
<MicrosoftNETCoreAppRefPackageVersion>7.0.0-rc.1.22403.8</MicrosoftNETCoreAppRefPackageVersion>
<MicrosoftNETCoreAppRefPackageVersion>7.0.0-rc.1.22411.12</MicrosoftNETCoreAppRefPackageVersion>
<MicrosoftTemplateEngineTasksVersion>7.0.100-alpha.1.21601.1</MicrosoftTemplateEngineTasksVersion>
</PropertyGroup>
</Project>
2 changes: 1 addition & 1 deletion global.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"sdk": {
"version": "7.0.100-rc.1.22405.9"
"version": "7.0.100-rc.2.22417.1"
}
}
26 changes: 25 additions & 1 deletion tests/common/DotNet.cs
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,7 @@ public static ExecutionResult Execute (string verb, string project, Dictionary<s
args.Add (verb);
args.Add (project);
if (properties != null) {
Dictionary<string, string> generatedProps = null;
foreach (var prop in properties) {
if (prop.Value.IndexOfAny (new char [] { ';' }) >= 0) {
// https://github.com/dotnet/msbuild/issues/471
Expand All @@ -128,11 +129,34 @@ public static ExecutionResult Execute (string verb, string project, Dictionary<s
// This means that a task that takes a "string[] RuntimeIdentifiers" will get an array with
// a single element, where that single element is the whole RuntimeIdentifiers string.
// Example task: https://github.com/dotnet/sdk/blob/ffca47e9a36652da2e7041360f2201a2ba197194/src/Tasks/Microsoft.NET.Build.Tasks/ProcessFrameworkReferences.cs#L45
args.Add ($"/p:{prop.Key}=\"{prop.Value}\"");
// args.Add ($"/p:{prop.Key}=\"{prop.Value}\"");

// Setting a property with a semicolon from the command line doesn't work anymore.
// Ref: https://github.com/dotnet/sdk/issues/27059#issuecomment-1219319513
// So write these properties in a file instead. This is a behavioural difference, because
// they'll be project-specific instead of global, but I don't see a better workaround.
if (generatedProps is null)
generatedProps = new Dictionary<string, string> ();
generatedProps.Add (prop.Key, prop.Value);
} else {
args.Add ($"/p:{prop.Key}={prop.Value}");
}
}
if (generatedProps is not null) {
var sb = new StringBuilder ();
sb.AppendLine ("<?xml version=\"1.0\" encoding=\"utf-8\"?>");
sb.AppendLine ("<Project>");
sb.AppendLine ("\t<PropertyGroup>");
foreach (var prop in generatedProps) {
sb.AppendLine ($"\t\t<{prop.Key}>{prop.Value}</{prop.Key}>");
}
sb.AppendLine ("\t</PropertyGroup>");
sb.AppendLine ("</Project>");

var generatedProjectFile = Path.Combine (Cache.CreateTemporaryDirectory (), "GeneratedProjectFile.props");
File.WriteAllText (generatedProjectFile, sb.ToString ());
args.Add ($"/p:GeneratedProjectFile={generatedProjectFile}");
}
}
if (!string.IsNullOrEmpty (target))
args.Add ("/t:" + target);
Expand Down
2 changes: 2 additions & 0 deletions tests/common/shared-dotnet.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -64,4 +64,6 @@
</ItemGroup>

<Import Project="$(MSBuildThisFileDirectory)/../nunit.framework.targets" />

<Import Project="$(GeneratedProjectFile)" Condition="'$(GeneratedProjectFile)' != ''" />
</Project>
2 changes: 2 additions & 0 deletions tests/dotnet/UnitTests/BundleStructureTest.cs
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ void CheckAppBundleContents (ApplePlatform platform, string appPath, string[] ru
var fn = Path.GetFileName (v!);

switch (fn) {
case "libclrgc.dylib":
case "libclrjit.dylib":
case "libcoreclr.dylib":
case "libdbgshim.dylib":
Expand All @@ -57,6 +58,7 @@ void CheckAppBundleContents (ApplePlatform platform, string appPath, string[] ru
case "libmono-component-debugger.dylib":
case "libmono-component-diagnostics_tracing.dylib":
case "libmono-component-hot_reload.dylib":
case "libmono-component-marshal-ilgen.dylib":
Copy link
Member

@jonathanpeppers jonathanpeppers Aug 25, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dalexsoto @rolfbjarne I think this is probably the change (plus the @(_MonoComponent) above), we're hitting that here:

dotnet/maui#9636 (review)

case "libmonosgen-2.0.dylib":
return platform != ApplePlatform.MacOSX;
case "libSystem.Native.dylib":
Expand Down
6 changes: 3 additions & 3 deletions tests/dotnet/UnitTests/ProjectTest.cs
Original file line number Diff line number Diff line change
Expand Up @@ -487,10 +487,10 @@ public void InvalidRuntimeIdentifier (ApplePlatform platform, string runtimeIden
}

[Test]
[TestCase (ApplePlatform.iOS, "win10-x86", null)]
[TestCase (ApplePlatform.TVOS, "win10-x64", null)]
[TestCase (ApplePlatform.iOS, "win10-x86", "Error getting pack version: Pack 'Microsoft.NETCore.App.Runtime.Mono.win-x86' was not present in workload manifests.")]
[TestCase (ApplePlatform.TVOS, "win10-x64", "Error getting pack version: Pack 'Microsoft.NETCore.App.Runtime.Mono.win-x64' was not present in workload manifests.")]
[TestCase (ApplePlatform.MacOSX, "win10-arm", null)]
[TestCase (ApplePlatform.MacCatalyst, "win10-arm64", "Unable to find package Microsoft.NETCore.App.Runtime.Mono.win-arm64. No packages exist with this id in source[(]s[)]:.*")]
[TestCase (ApplePlatform.MacCatalyst, "win10-arm64", "Error getting pack version: Pack 'Microsoft.NETCore.App.Runtime.Mono.win-arm64' was not present in workload manifests.")]
public void InvalidRuntimeIdentifier_Restore (ApplePlatform platform, string runtimeIdentifier, string? failureMessagePattern)
{
var project = "MySimpleApp";
Expand Down
1 change: 1 addition & 0 deletions tests/linker/ios/dont link/dotnet/shared.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
<RootNamespace>dontlink</RootNamespace>
<AssemblyName>dont link</AssemblyName>
<MtouchLink>None</MtouchLink>
<LinkMode>$(MtouchLink)</LinkMode>
<RootTestsDirectory>$([System.IO.Path]::GetFullPath('$(MSBuildThisFileDirectory)\..\..\..\..'))</RootTestsDirectory>
<ThisTestDirectory>$(RootTestsDirectory)\linker\ios\dont link</ThisTestDirectory>
</PropertyGroup>
Expand Down
1 change: 1 addition & 0 deletions tests/linker/ios/trimmode copy/dotnet/shared.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
<AssemblyName>trimmode copy</AssemblyName>
<ApplicationTitle>TrimMode Copy</ApplicationTitle>
<TrimMode>copy</TrimMode>
<_TrimmerDefaultAction>copy</_TrimmerDefaultAction> <!-- https://github.com/dotnet/linker/issues/2981 -->
<RootTestsDirectory>$([System.IO.Path]::GetFullPath('$(MSBuildProjectDirectory)\..\..\..\..\..'))</RootTestsDirectory>
<ThisTestDirectory>$(RootTestsDirectory)\linker\ios\dont link</ThisTestDirectory>
</PropertyGroup>
Expand Down