[net10.0] Merge main into net10.0.

Directory.Build.props

@@ -9,6 +9,10 @@
 		<MicrosoftBuildTasksCorePackageVersion>$(MicrosoftBuildPackageVersion)</MicrosoftBuildTasksCorePackageVersion>
 		<MicrosoftBuildUtilitiesCorePackageVersion>$(MicrosoftBuildPackageVersion)</MicrosoftBuildUtilitiesCorePackageVersion>
 		<NUnitXmlTestLoggerPackageVersion>3.1.15</NUnitXmlTestLoggerPackageVersion>
+		<!-- Fix transient dependency issue found by component governance 4.7.0 -> 4.7.2 brought by Microsoft.Build package -->
+		<SystemDrawingCommonPackageVersion>4.7.2</SystemDrawingCommonPackageVersion>
+		<!-- Fix transient dependency issue found by component governance 4.7.0 -> 4.7.1 brought by Microsoft.Build.Tasks.Core package -->
+		<SystemSecurityCryptographyXmlPackageVersion>4.7.1</SystemSecurityCryptographyXmlPackageVersion>
 	</PropertyGroup>
 	<Import Project="Build.props" Condition="Exists('Build.props')" />
 </Project>

builds/Makefile

@@ -55,9 +55,9 @@ print-dotnet-pkg-urls: dotnet-install.sh
 	$(Q) if ! test -f $@-found-it.stamp; then echo "No working urls were found."; exit 1; fi
 	$(Q) rm -f $@-found-it.stamp
 
-# We need https://github.com/dotnet/install-scripts/pull/526, which hasn't been released yet.
+# We need https://github.com/dotnet/install-scripts/pull/555, which hasn't been released yet.
 # Revert to using https://dot.net/v1/dotnet-install.sh whenever https://dot.net/v1/dotnet-install.sh has been updated.
-DOTNET_DOWNLOAD_URL?=https://raw.githubusercontent.com/dotnet/install-scripts/87151ae8d2153247f4566208e0df23169d737e92/src/dotnet-install.sh
+DOTNET_DOWNLOAD_URL?=https://raw.githubusercontent.com/dotnet/install-scripts/d48e000adad7243861d799ec86c0bf7210a8fc7a/src/dotnet-install.sh
 
 DOTNET_DOWNLOAD_URL?=https://dot.net/v1/dotnet-install.sh
 

dotnet/Workloads/SignList.xml

@@ -35,12 +35,13 @@
     <Skip Include="Build\System.Text.Json.dll" />
     <Skip Include="Build\System.Threading.Tasks.Dataflow.dll" />
     <Skip Include="Build\System.Threading.Tasks.Extensions.dll" />
+    <!-- They are now signed by their origin build https://github.com/xamarin/XamarinVS/pull/14522 -->
+    <Skip Include="imobiledevice-*\*.dll" />
+    <Skip Include="imobiledevice-*\*.exe" />
   </ItemGroup>
 
   <ItemGroup>
     <ThirdParty Include="BouncyCastle.Crypto.dll" />
-    <ThirdParty Include="imobiledevice-*\*.dll" />
-    <ThirdParty Include="imobiledevice-*\*.exe" />
     <!-- Build.zip -->
     <ThirdParty Include="Mono.Cecil*.dll" />
 

eng/Versions.props

@@ -12,7 +12,7 @@
     <MicrosoftNETRuntimeMonoTargetsSdkPackageVersion>9.0.0-rc.2.24462.10</MicrosoftNETRuntimeMonoTargetsSdkPackageVersion>
     <MicrosoftTemplateEngineTasksVersion>7.0.100-alpha.1.21601.1</MicrosoftTemplateEngineTasksVersion>
     <MicrosoftDotNetCecilPackageVersion>0.11.5-alpha.24627.1</MicrosoftDotNetCecilPackageVersion>
-    <MicrosoftDotNetXHarnessiOSSharedPackageVersion>10.0.0-prerelease.24610.1</MicrosoftDotNetXHarnessiOSSharedPackageVersion>
+    <MicrosoftDotNetXHarnessiOSSharedPackageVersion>10.0.0-prerelease.25103.1</MicrosoftDotNetXHarnessiOSSharedPackageVersion>
     <MicrosoftDotNetArcadeSdkPackageVersion>9.0.0-beta.25058.5</MicrosoftDotNetArcadeSdkPackageVersion>
     <!-- Manually updated versions -->
     <Emscriptennet7WorkloadVersion>$(MicrosoftNETWorkloadEmscriptenCurrentManifest80100Version)</Emscriptennet7WorkloadVersion>

mk/xamarin.mk

@@ -16,7 +16,7 @@ MACCORE_VERSION   := $(shell cd $(MACCORE_PATH) 2> /dev/null && git rev-parse HE
 MACCORE_BRANCH    := $(shell cd $(MACCORE_PATH) 2> /dev/null && git symbolic-ref --short HEAD 2> /dev/null)
 
 
-NEEDED_ADR_VERSION := d466eac70a9b1bba94c18c63e34e590c66f6d044
+NEEDED_ADR_VERSION := adccd1c48a7c2540b0c37f2bf7add7d2c1398729
 NEEDED_ADR_BRANCH := main
 
 ADR_DIRECTORY := macios-adr

msbuild/Directory.Build.props

@@ -16,7 +16,7 @@
 
 		-->
 		<MessagingVersion Condition="'$(MessagingVersion)' == ''">[3.0.13]</MessagingVersion>
-		<HotRestartVersion>[1.1.7]</HotRestartVersion>
+		<HotRestartVersion>[17.14.133-g001ce2ac7a]</HotRestartVersion>
 	</PropertyGroup>
 	<Import Project="$(MSBuildThisFileDirectory)../Directory.Build.props" />
 </Project>

msbuild/Makefile

@@ -53,7 +53,7 @@ IOS_WINDOWS_TARGETS =                                                      \
 
 TASK_ASSEMBLIES = Xamarin.MacDev.Tasks $(LOCALIZATION_ASSEMBLIES)
 IOS_WINDOWS_TASK_ASSEMBLIES = Xamarin.iOS.Tasks.Windows
-IOS_WINDOWS_DEPENDENCIES = Xamarin.iOS.Windows.Client iSign.Core ws2_32 System.Diagnostics.Tracer System.Formats.Asn1 System.Buffers System.Memory System.Numerics.Vectors System.Runtime.CompilerServices.Unsafe System.Security.Cryptography.ProtectedData System.Security.Cryptography.Pkcs Microsoft.Win32.Registry System.Security.AccessControl System.Security.Principal.Windows
+IOS_WINDOWS_DEPENDENCIES = Xamarin.iOS.Windows.Client iSign.Core ws2_32 System.Diagnostics.Tracer System.Formats.Asn1 System.Buffers System.Memory System.Numerics.Vectors System.Runtime.CompilerServices.Unsafe System.Security.Cryptography.ProtectedData System.Security.Cryptography.Pkcs Microsoft.Win32.Registry
 IOS_WINDOWS_MOBILEDEVICE_TOOLS = iMobileDevice-net bz2 getopt ideviceactivation idevicebackup idevicebackup2 idevicecrashreport idevicedate idevicedebug idevicedebugserverproxy idevicediagnostics ideviceenterrecovery ideviceimagemounter ideviceinfo ideviceinstaller idevicename idevicenotificationproxy idevicepair ideviceprovision idevicerestore idevicescreenshot idevicesyslog idevice_id imobiledevice-net-lighthouse imobiledevice ios_webkit_debug_proxy iproxy irecovery irecovery libcharset libcurl LIBEAY32 libiconv libusb-1.0 libusb0 libxml2 lzma pcre pcreposix plist plistutil plist_cmp plist_test pthreadVC3 readline SSLEAY32 usbmuxd usbmuxd vcruntime140 zip zlib1
 
 PROPS_AND_TARGETS = \

msbuild/Messaging/Xamarin.Messaging.Build/Xamarin.Messaging.Build.csproj

@@ -32,6 +32,10 @@
       <PackageReference Include="Microsoft.Build.Framework" Version="$(MicrosoftBuildFrameworkPackageVersion)" />
       <PackageReference Include="Microsoft.Build.Tasks.Core" Version="$(MicrosoftBuildTasksCorePackageVersion)" />
       <PackageReference Include="Microsoft.Build.Utilities.Core" Version="$(MicrosoftBuildUtilitiesCorePackageVersion)" />
+      <!-- Fix transient dependency issue found by component governance 4.7.0 -> 4.7.2 -->
+      <PackageReference Include="System.Drawing.Common" Version="$(SystemDrawingCommonPackageVersion)" />
+      <!-- Fix transient dependency issue found by component governance 4.7.0 -> 4.7.1 -->
+      <PackageReference Include="System.Security.Cryptography.Xml" Version="$(SystemSecurityCryptographyXmlPackageVersion)" />
       <Compile Include="..\..\Versions.g.cs">
          <Link>Versions.g.cs</Link>
       </Compile>

msbuild/Xamarin.Localization.MSBuild/MSBStrings.resx

@@ -1714,4 +1714,9 @@
             {0}: the path to a file
         </comment>
     </data>
+
+    <data name="W7144" xml:space="preserve">
+        <value>Did not extract {0} because it would write outside the target directory.</value>
+    </data>
+
 </root>

msbuild/Xamarin.MacDev.Tasks/Decompress.cs

@@ -171,6 +171,7 @@ static bool TryDecompressUsingSystemIOCompression (TaskLoggingHelper log, string
 			resource = resource.TrimEnd ('/', '\\');
 			resource = resource.Replace ('\\', zipDirectorySeparator);
 			var resourceAsDir = resource + zipDirectorySeparator;
+			decompressionDir = Path.GetFullPath (decompressionDir);
 
 			using var archive = ZipFile.OpenRead (zip);
 			foreach (var entry in archive.Entries) {
@@ -204,6 +205,17 @@ static bool TryDecompressUsingSystemIOCompression (TaskLoggingHelper log, string
 
 				var isDir = entryPath [entryPath.Length - 1] == zipDirectorySeparator;
 				var targetPath = Path.Combine (decompressionDir, entryPath.Replace (zipDirectorySeparator, Path.DirectorySeparatorChar));
+
+				// canonicalize the path
+				targetPath = Path.GetFullPath (targetPath);
+
+				// validate that the unzipped file is inside the target directory
+				var decompressionDirectoryPath = decompressionDir.Trim (Path.DirectorySeparatorChar) + Path.DirectorySeparatorChar;
+				if (!targetPath.StartsWith (decompressionDirectoryPath)) {
+					log.LogWarning (7144, null, MSBStrings.W7144 /* Did not extract {0} because it would write outside the target directory. */, entryPath);
+					continue;
+				}
+
 				if (isDir) {
 					Directory.CreateDirectory (targetPath);
 				} else {

msbuild/Xamarin.MacDev.Tasks/Tasks/FilterStaticFrameworks.cs

@@ -52,7 +52,7 @@ public override bool Execute ()
 						continue;
 					}
 
-					Log.LogWarning (7091, frameworkExecutablePath, MSBStrings.W7091 /* "The framework {0} is a framework of static libraries, and will not be copied to the app." */, Path.GetDirectoryName (frameworkExecutablePath));
+					Log.LogMessage (MessageImportance.Low, MSBStrings.W7091 /* "The framework {0} is a framework of static libraries, and will not be copied to the app." */, Path.GetDirectoryName (frameworkExecutablePath));
 					list.RemoveAt (i);
 				}
 

msbuild/Xamarin.MacDev.Tasks/Tasks/PackLibraryResources.cs

@@ -49,7 +49,14 @@ public override bool Execute ()
 		{
 			var results = new List<ITaskItem> ();
 
+			var hashOriginalResources = new HashSet<string> (BundleOriginalResourcesWithLogicalNames.Select (v => v.ItemSpec));
+
 			foreach (var item in BundleResourcesWithLogicalNames) {
+				if (hashOriginalResources.Contains (item.ItemSpec)) {
+					Log.LogMessage (MessageImportance.Low, $"Skipping BundleResourcesWithLogicalNames={item.ItemSpec}, because it's also specified in BundleOriginalResourcesWithLogicalNames");
+					continue;
+				}
+
 				var logicalName = item.GetMetadata ("LogicalName");
 
 				if (string.IsNullOrEmpty (logicalName)) {

msbuild/Xamarin.MacDev.Tasks/Xamarin.MacDev.Tasks.csproj

@@ -30,6 +30,10 @@
     <PackageReference Include="Microsoft.NET.ILLink.Tasks" Version="$(MicrosoftNETILLinkTasksPackageVersion)" />
     <PackageReference Include="Microsoft.NET.Runtime.MonoTargets.Sdk" Version="$(MicrosoftNETRuntimeMonoTargetsSdkPackageVersion)" GeneratePathProperty="true"/>
     <PackageReference Include="Xamarin.Messaging.Build.Client" Version="$(MessagingVersion)" />
+    <!-- Fix transient dependency issue found by component governance 4.7.0 -> 4.7.2 -->
+    <PackageReference Include="System.Drawing.Common" Version="$(SystemDrawingCommonPackageVersion)" />
+    <!-- Fix transient dependency issue found by component governance 4.7.0 -> 4.7.1 -->
+    <PackageReference Include="System.Security.Cryptography.Xml" Version="$(SystemSecurityCryptographyXmlPackageVersion)" />
   </ItemGroup>
   <ItemGroup>
     <ProjectReference Include="..\..\external\Xamarin.MacDev\Xamarin.MacDev\Xamarin.MacDev.csproj" />

msbuild/Xamarin.iOS.Tasks.Windows/Xamarin.iOS.Tasks.Windows.csproj

@@ -22,6 +22,7 @@
     <PackageReference Include="Xamarin.Messaging.Build.Client" Version="$(MessagingVersion)" IncludeAssets="contentFiles" />
     <PackageReference Include="Xamarin.Messaging.Core" Version="$(MessagingVersion)" IncludeAssets="build" />
     <PackageReference Include="Xamarin.Messaging.Server" Version="$(MessagingVersion)" IncludeAssets="contentFiles" />
+    <PackageReference Include="iSign.Core" Version="$(HotRestartVersion)" GeneratePathProperty="true" />
     <PackageReference Include="Xamarin.iOS.HotRestart.Client" Version="$(HotRestartVersion)" GeneratePathProperty="true" />
     <PackageReference Include="System.Diagnostics.Tracer" Version="2.1.0-alpha" GeneratePathProperty="true" />
     <!-- GitInfo is pulled in because of Xamarin.Messaging from above, but we don't want it, so just exclude all assets from it -->
@@ -38,7 +39,7 @@
     <None Include="*.targets">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>
-    <None Include="$(PkgXamarin_iOS_HotRestart_Client)\lib\netstandard2.0\iSign.Core.pdb" CopyToOutputDirectory="PreserveNewest" />
+    <None Include="$(PkgiSign_Core)\lib\netstandard2.0\iSign.Core.pdb" CopyToOutputDirectory="PreserveNewest" />
     <None Include="$(PkgXamarin_iOS_HotRestart_Client)\lib\netstandard2.0\Xamarin.iOS.Windows.Client.pdb" CopyToOutputDirectory="PreserveNewest" />
     <Compile Include="../Versions.g.cs" />
     <Compile Include="..\Xamarin.MacDev.Tasks\PublishFolderType.cs">

scripts/rsp-to-csproj/rsp-to-csproj.cs

@@ -175,6 +175,10 @@ void ProcessFile (string file)
 		break;
 	case "noconfig": // this is already passed to csc by default
 		break;
+	case "reportanalyzer":
+		if (!properties.Contains (("ReportAnalyzer", "true")))
+			properties.Add (new ("ReportAnalyzer", "true"));
+		break;
 	default:
 		ReportError ($"Didn't understand argument '{a}'");
 		break;

src/AppKit/Enums.cs

@@ -1439,22 +1439,6 @@ public enum NSPasteboardReadingOptions : ulong {
 		AsKeyedArchive = 4,
 	}
 
-#if !NET && MONOMAC // Use the one in Foundation instead, only keep this in macOS until .NET.
-	[Native]
-	public enum NSUnderlineStyle : long {
-		None                = 0x00,
-		Single              = 0x01,
-		Thick               = 0x02,
-		Double              = 0x09,
-		PatternSolid 		= 0x0000,
-		PatternDot 			= 0x0100,
-		PatternDash 		= 0x0200,
-		PatternDashDot 		= 0x0300,
-		PatternDashDotDot 	= 0x0400,
-		ByWord 				= 0x8000,
-	}
-#endif
-
 	// Convenience enum, untyped in ObjC
 	[NoMacCatalyst]
 	public enum NSUnderlinePattern : int {

src/Foundation/Compat.cs

@@ -1,5 +1,5 @@
 //
-// Compat.cs: Stuff we won't provide in Xamarin.iOS.dll or newer XAMCORE_* profiles
+// Compat.cs: Stuff we won't provide in newer XAMCORE_* profiles
 //
 // Authors:
 //   Sebastien Pouliot  <[email protected]>
@@ -12,30 +12,6 @@
 using ObjCRuntime;
 
 namespace Foundation {
-
-#if !NET
-#if MONOMAC
-	public partial class NSError {
-
-		// removed in Xcode 11 GM
-		[Obsolete ("This API has been removed.")]
-		public static NSError GetFileProviderErrorForOutOfDateItem (FileProvider.INSFileProviderItem updatedVersion)
-		{
-			return null;
-		}
-	}
-#endif
-
-	public partial class NSUserActivity {
-
-		[Obsolete ("Use the constructor that allows you to set an activity type.")]
-		public NSUserActivity ()
-			: this (String.Empty)
-		{
-		}
-	}
-#endif // !NET
-
 #if !XAMCORE_5_0
 #if __IOS__ && !__MACCATALYST__
 	public partial class NSUrlConnection {

src/Foundation/DictionaryContainer.cs

@@ -43,20 +43,14 @@
 using UIKit;
 #endif
 
-#if !NET
-using NativeHandle = System.IntPtr;
-#endif
-
 #nullable enable
 
 namespace Foundation {
 
-#if NET
 	[SupportedOSPlatform ("ios")]
 	[SupportedOSPlatform ("maccatalyst")]
 	[SupportedOSPlatform ("macos")]
 	[SupportedOSPlatform ("tvos")]
-#endif
 	public abstract class DictionaryContainer {
 #if !COREBUILD
 		protected DictionaryContainer ()
@@ -247,11 +241,7 @@ protected DictionaryContainer (NSDictionary? dictionary)
 			return value as NSDictionary<TKey, TValue>;
 		}
 
-#if NET
 		protected T? GetStrongDictionary<[DynamicallyAccessedMembers (DynamicallyAccessedMemberTypes.PublicConstructors | DynamicallyAccessedMemberTypes.NonPublicConstructors)] T> (NSString key)
-#else
-		protected T? GetStrongDictionary<T> (NSString key)
-#endif
 			where T : DictionaryContainer
 		{
 			return GetStrongDictionary (key, dict =>

src/Foundation/Enum.cs

@@ -77,14 +77,9 @@ public enum NSUrlCredentialPersistence : ulong {
 
 #if MONOMAC
 
-#if !NET
-	[Native]
-	public enum NSBundleExecutableArchitecture : long {
-#else
 	/// <summary>The bundle's architecture.</summary>
 	[NoiOS][NoTV][NoMacCatalyst]
 	public enum NSBundleExecutableArchitecture {
-#endif
 		I386 = 0x00000007,
 		PPC = 0x00000012,
 		X86_64 = 0x01000007,
@@ -252,10 +247,6 @@ public enum NSCalendarUnit : ulong {
 	public enum NSDataReadingOptions : ulong {
 		Mapped = 1 << 0,
 		Uncached = 1 << 1,
-#if !NET
-		[Obsolete ("This option is unavailable.")]
-		Coordinated = 1 << 2,
-#endif
 		MappedAlways = 1 << 3,
 	}
 
@@ -592,10 +583,6 @@ public enum NSVolumeEnumerationOptions : ulong {
 	[Flags]
 	[Native]
 	public enum NSDirectoryEnumerationOptions : ulong {
-#if !NET
-		[Obsolete ("Use 'None' instead.")]
-		SkipsNone = 0,
-#endif
 		None = 0,
 		SkipsSubdirectoryDescendants = 1 << 0,
 		SkipsPackageDescendants = 1 << 1,
@@ -808,10 +795,6 @@ public enum NSJsonReadingOptions : ulong {
 		Json5Allowed = 8,
 		[iOS (15, 0), TV (15, 0), MacCatalyst (15, 0)]
 		TopLevelDictionaryAssumed = 16,
-#if !NET
-		[Obsolete ("Use 'FragmentsAllowed. instead.")]
-		AllowFragments = FragmentsAllowed,
-#endif
 	}
 
 	[Flags]
@@ -889,7 +872,6 @@ public enum NSAttributedStringEnumeration : ulong {
 		LongestEffectiveRangeNotRequired = 1 << 20,
 	}
 
-#if NET || !MONOMAC
 	// macOS has defined this in AppKit as well, but starting with .NET we're going
 	// to use this one only.
 	[Native]
@@ -905,7 +887,6 @@ public enum NSUnderlineStyle : long {
 		PatternDashDotDot = 0x0400,
 		ByWord = 0x8000,
 	}
-#endif
 
 	// There's an AppKit.NSWritingDirection, which is deprecated.
 	// There's also an UIKit.UITextWritingDirection, which is deprecated too.
@@ -977,17 +958,6 @@ public enum NSLigatureType : long {
 		All,
 	}
 
-#if !NET
-	[Flags]
-	[Native]
-	public enum NSDateComponentsWrappingBehavior : ulong {
-		None = 0,
-		WrapCalendarComponents = 1 << 0,
-
-		// Did not add the new enums here, we moved them elsewhere, and provided overloads.
-	}
-#endif
-
 	[Flags]
 	[Native]
 	public enum NSCalendarOptions : ulong {