Give tvOS a special value for Personal/MyDocuments

src/libraries/Common/src/Interop/OSX/System.Native/Interop.SearchPath.iOS.cs

@@ -0,0 +1,14 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+using System;
+using System.Runtime.InteropServices;
+
+internal static partial class Interop
+{
+    internal static partial class Sys
+    {
+        [DllImport(Libraries.SystemNative, EntryPoint = "SystemNative_SearchPath_TempDirectory")]
+        internal static extern string? SearchPathTempDirectory();
+    }
+}

src/libraries/Common/tests/TestUtilities/System/IO/FileCleanupTestBase.cs

@@ -2,6 +2,7 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 
 using Xunit;
+using System;
 using System.Diagnostics;
 using System.Runtime.CompilerServices;
 using System.Threading;
@@ -20,6 +21,7 @@ public abstract class FileCleanupTestBase : IDisposable
         /// <summary>Initialize the test class base.  This creates the associated test directory.</summary>
         protected FileCleanupTestBase(string tempDirectory = null)
         {
+            //tempDirectory ??= PlatformDetection.IsNotAppleMobile ? Path.GetTempPath() : Environment.GetFolderPath(Environment.SpecialFolder.InternetCache);
             tempDirectory ??= Path.GetTempPath();
 
             // Use a unique test directory per test class.  The test directory lives in the user's temp directory,

src/libraries/Common/tests/TestUtilities/System/PlatformDetection.cs

@@ -40,6 +40,8 @@ public static partial class PlatformDetection
         public static bool IsNotBrowser => !IsBrowser;
         public static bool IsMobile => IsBrowser || IsMacCatalyst || IsiOS || IstvOS || IsAndroid;
         public static bool IsNotMobile => !IsMobile;
+        public static bool IsAppleMobile => IsMacCatalyst || IsiOS || IstvOS;
+        public static bool IsNotAppleMobile => !IsAppleMobile;
         public static bool IsNotNetFramework => !IsNetFramework;
 
         public static bool IsArmProcess => RuntimeInformation.ProcessArchitecture == Architecture.Arm;

src/libraries/Native/Unix/System.Native/entrypoints.c

@@ -218,6 +218,7 @@ static const Entry s_sysNative[] =
     DllImportEntry(SystemNative_GetOSArchitecture)
     DllImportEntry(SystemNative_GetProcessArchitecture)
     DllImportEntry(SystemNative_SearchPath)
+    DllImportEntry(SystemNative_SearchPath_TempDirectory)
     DllImportEntry(SystemNative_RegisterForSigChld)
     DllImportEntry(SystemNative_SetDelayedSigChildConsoleConfigurationHandler)
     DllImportEntry(SystemNative_SetTerminalInvalidationHandler)

src/libraries/Native/Unix/System.Native/pal_searchpath.c

@@ -10,3 +10,9 @@ const char* SystemNative_SearchPath(int32_t folderId)
     __builtin_unreachable();
     return NULL;
 }
+
+const char* SystemNative_SearchPath_TempDirectory()
+{
+    __builtin_unreachable();
+    return NULL;
+}

src/libraries/Native/Unix/System.Native/pal_searchpath.h

@@ -7,3 +7,5 @@
 #include "pal_types.h"
 
 PALEXPORT const char* SystemNative_SearchPath(int32_t folderId);
+
+PALEXPORT const char* SystemNative_SearchPath_TempDirectory(void);

src/libraries/Native/Unix/System.Native/pal_searchpath.m

@@ -11,3 +11,10 @@
     const char* path = [[url path] UTF8String];
     return path == NULL ? NULL : strdup (path);
 }
+
+const char* SystemNative_SearchPath_TempDirectory()
+{
+    NSString* iosPath = NSTemporaryDirectory();
+    const char *path = [iosPath UTF8String];
+    return path == NULL ? NULL : strdup (path);
+}

src/libraries/System.Private.CoreLib/src/System.Private.CoreLib.Shared.projitems

@@ -2112,6 +2112,8 @@
     <Compile Include="$(MSBuildThisFileDirectory)System\IO\FileSystem.Exists.Unix.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)System\IO\FileSystemInfo.Unix.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)System\IO\Path.Unix.cs" />
+    <Compile Condition="'$(IsiOSLike)' == 'true'" Include="$(MSBuildThisFileDirectory)System\IO\Path.Unix.iOS.cs" />
+    <Compile Condition="'$(IsiOSLike)' != 'true'" Include="$(MSBuildThisFileDirectory)System\IO\Path.Unix.NoniOS.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)System\IO\PathInternal.Unix.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)System\IO\PersistedFiles.Names.Unix.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)System\IO\RandomAccess.Unix.cs" />

src/libraries/System.Private.CoreLib/src/System/IO/Path.Unix.NoniOS.cs

@@ -0,0 +1,10 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+namespace System.IO
+{
+    public static partial class Path
+    {
+        private static string? DefaultTempPath => "/tmp/";
+    }
+}

src/libraries/System.Private.CoreLib/src/System/IO/Path.Unix.cs

@@ -80,14 +80,13 @@ private static string RemoveLongPathPrefix(string path)
         public static string GetTempPath()
         {
             const string TempEnvVar = "TMPDIR";
-            const string DefaultTempPath = "/tmp/";
 
             // Get the temp path from the TMPDIR environment variable.
             // If it's not set, just return the default path.
             // If it is, return it, ensuring it ends with a slash.
             string? path = Environment.GetEnvironmentVariable(TempEnvVar);
             return
-                string.IsNullOrEmpty(path) ? DefaultTempPath :
+                string.IsNullOrEmpty(path) ? DefaultTempPath! :
                 PathInternal.IsDirectorySeparator(path[path.Length - 1]) ? path :
                 path + PathInternal.DirectorySeparatorChar;
         }

src/libraries/System.Private.CoreLib/src/System/IO/Path.Unix.iOS.cs

@@ -0,0 +1,21 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+using System;
+
+namespace System.IO
+{
+    public static partial class Path
+    {
+        private static string? s_defaultTempPath;
+
+        private static string? DefaultTempPath
+        {
+            get
+            {
+                s_defaultTempPath ??= Interop.Sys.SearchPathTempDirectory()!;
+                return s_defaultTempPath!;
+            }
+        }
+    }
+}

src/libraries/System.Runtime.Extensions/tests/System/AppDomainTests.cs

@@ -763,14 +763,16 @@ public void SetThreadPrincipal()
 
         private void CopyTestAssemblies()
         {
-            string destTestAssemblyPath = Path.Combine(Environment.CurrentDirectory, "AssemblyResolveTestApp", "AssemblyResolveTestApp.dll");
+            string curDirectory = PlatformDetection.IsNotAppleMobile ? Environment.CurrentDirectory : Environment.GetFolderPath(Environment.SpecialFolder.InternetCache); 
+
+            string destTestAssemblyPath = Path.Combine(curDirectory, "AssemblyResolveTestApp", "AssemblyResolveTestApp.dll");
             if (!File.Exists(destTestAssemblyPath) && File.Exists("AssemblyResolveTestApp.dll"))
             {
                 Directory.CreateDirectory(Path.GetDirectoryName(destTestAssemblyPath));
                 File.Copy("AssemblyResolveTestApp.dll", destTestAssemblyPath, false);
             }
 
-            destTestAssemblyPath = Path.Combine(Environment.CurrentDirectory, "TestAppOutsideOfTPA", "TestAppOutsideOfTPA.exe");
+            destTestAssemblyPath = Path.Combine(curDirectory, "TestAppOutsideOfTPA", "TestAppOutsideOfTPA.exe");
             if (!File.Exists(destTestAssemblyPath) && File.Exists("TestAppOutsideOfTPA.exe"))
             {
                 Directory.CreateDirectory(Path.GetDirectoryName(destTestAssemblyPath));

src/libraries/System.Runtime.Extensions/tests/System/EnvironmentTests.cs

@@ -330,6 +330,13 @@ public void FailFast_ExceptionStackTrace_InnerException()
             }
         }
 
+        [Fact]
+        [PlatformSpecific(TestPlatforms.AnyUnix | TestPlatforms.Browser)]
+        public void GetFolderPath_Unix_PersonalExists()
+        {
+            Assert.True(Directory.Exists(Environment.GetFolderPath(Environment.SpecialFolder.Personal)));
+        }
+
         [Fact]
         [PlatformSpecific(TestPlatforms.AnyUnix | TestPlatforms.Browser)]  // Tests OS-specific environment
         public void GetFolderPath_Unix_PersonalIsHomeAndUserProfile()
@@ -339,7 +346,11 @@ public void GetFolderPath_Unix_PersonalIsHomeAndUserProfile()
                 Assert.Equal(Environment.GetEnvironmentVariable("HOME"), Environment.GetFolderPath(Environment.SpecialFolder.Personal));
                 Assert.Equal(Environment.GetEnvironmentVariable("HOME"), Environment.GetFolderPath(Environment.SpecialFolder.MyDocuments));
             }
-            Assert.Equal(Environment.GetEnvironmentVariable("HOME"), Environment.GetFolderPath(Environment.SpecialFolder.UserProfile));
+            // tvOS effectively doesn't have a HOME
+            if (!PlatformDetection.IstvOS)
+            {
+                Assert.Equal(Environment.GetEnvironmentVariable("HOME"), Environment.GetFolderPath(Environment.SpecialFolder.UserProfile));
+            }
         }
 
         [Theory]

src/libraries/sendtohelixhelp.proj

@@ -294,7 +294,7 @@
         <TestTarget>$(AppleTestTarget)</TestTarget>
       </XHarnessAppBundleToTest>
       <!-- Create work items for run-only apps -->
-      <XHarnessAppBundleToTest Condition="Exists('$(TestArchiveRoot)runonly')" Include="$([System.IO.Directory]::GetDirectories('$(TestArchiveRoot)runonly', '*.app', System.IO.SearchOption.AllDirectories))" >
+      <XHarnessAppBundleToTest Condition="Exists('$(TestArchiveRoot)runonly') and '$(TargetOS)' != 'tvOS'" Include="$([System.IO.Directory]::GetDirectories('$(TestArchiveRoot)runonly', '*.app', System.IO.SearchOption.AllDirectories))" >
         <!-- The sample app doesn't need test runner -->
         <IncludesTestRunner>false</IncludesTestRunner>
         <!-- The sample's C# Main method returns 42 so it should be considered by xharness as a success -->

src/mono/System.Private.CoreLib/System.Private.CoreLib.csproj

@@ -271,6 +271,9 @@
       <Compile Include="$(CommonPath)Interop\OSX\System.Native\Interop.SearchPath.cs">
         <Link>Common\Interop\OSX\Interop.SearchPath.cs</Link>
       </Compile>
+      <Compile Include="$(CommonPath)Interop\OSX\System.Native\Interop.SearchPath.iOS.cs">
+        <Link>Common\Interop\OSX\Interop.SearchPath.iOS.cs</Link>
+      </Compile>
   </ItemGroup>
   <ItemGroup Condition="'$(TargetsBrowser)' == 'true'">
       <Compile Include="$(BclSourcesRoot)\System\Threading\TimerQueue.Browser.Mono.cs" />

src/mono/System.Private.CoreLib/src/System/Environment.iOS.cs

@@ -34,20 +34,22 @@ private static string GetFolderPathCore(SpecialFolder folder, SpecialFolderOptio
 
         private static string? GetSpecialFolder(SpecialFolder folder)
         {
+#if TARGET_TVOS
+        string? DocumentsSearchPath = CombineAndCreateSearchPath(NSSearchPathDirectory.NSLibraryDirectory, Path.Combine("Caches", "Documents"));
+#else
+        string? DocumentsSearchPath = Interop.Sys.SearchPath(NSSearchPathDirectory.NSDocumentDirectory);
+#endif
+
             switch (folder)
             {
-                // TODO: fix for tvOS (https://github.com/dotnet/runtime/issues/34007)
-                // The "normal" NSDocumentDirectory is a read-only directory on tvOS
-                // and that breaks a lot of assumptions in the runtime and the BCL
-
                 case SpecialFolder.Personal:
                 case SpecialFolder.LocalApplicationData:
-                    return Interop.Sys.SearchPath(NSSearchPathDirectory.NSDocumentDirectory);
+                    return DocumentsSearchPath;
 
                 case SpecialFolder.ApplicationData:
                     // note: at first glance that looked like a good place to return NSLibraryDirectory
                     // but it would break isolated storage for existing applications
-                    return CombineSearchPath(NSSearchPathDirectory.NSDocumentDirectory, ".config");
+                    return DocumentsSearchPath != null ? Path.Combine(DocumentsSearchPath, ".config") : string.Empty;
 
                 case SpecialFolder.Resources:
                     return Interop.Sys.SearchPath(NSSearchPathDirectory.NSLibraryDirectory); // older (8.2 and previous) would return String.Empty
@@ -63,7 +65,7 @@ private static string GetFolderPathCore(SpecialFolder folder, SpecialFolderOptio
                     return Path.Combine(GetFolderPathCore(SpecialFolder.Personal, SpecialFolderOption.None), "Pictures");
 
                 case SpecialFolder.Templates:
-                    return CombineSearchPath(NSSearchPathDirectory.NSDocumentDirectory, "Templates");
+                    return DocumentsSearchPath != null ? Path.Combine(DocumentsSearchPath, "Templates") : string.Empty;
 
                 case SpecialFolder.MyVideos:
                     return Path.Combine(GetFolderPathCore(SpecialFolder.Personal, SpecialFolderOption.None), "Videos");
@@ -72,7 +74,7 @@ private static string GetFolderPathCore(SpecialFolder folder, SpecialFolderOptio
                     return "/usr/share/templates";
 
                 case SpecialFolder.Fonts:
-                    return CombineSearchPath(NSSearchPathDirectory.NSDocumentDirectory, ".fonts");
+                    return DocumentsSearchPath != null ? Path.Combine(DocumentsSearchPath, ".fonts") : string.Empty;
 
                 case SpecialFolder.Favorites:
                     return CombineSearchPath(NSSearchPathDirectory.NSLibraryDirectory, "Favorites");
@@ -100,6 +102,21 @@ static string CombineSearchPath(NSSearchPathDirectory searchPath, string subdire
                     Path.Combine(path, subdirectory) :
                     string.Empty;
             }
+
+#if TARGET_TVOS
+            // Special version of CombineSearchPath which creates the path if needed.
+            // This isn't needed for "real" search paths which always exist, but on tvOS
+            // the base path is really a subdirectory we define rather than an OS directory.
+            // In order to not treat Directory.Exists(SpecialFolder.ApplicationData) differently
+            // on tvOS, guarantee that it exists by creating it here
+            static string CombineAndCreateSearchPath(NSSearchPathDirectory searchPath, string subdirectory)
+            {
+                string path = CombineSearchPath(searchPath, subdirectory);
+                if (!Directory.Exists (path))
+                    Directory.CreateDirectory (path);
+                return path;
+            }
+#endif
         }
     }
 }