[release/7.0] Implement proper File System detection for Apple OSes (#…

…74560) * the assert should fail on macOS * Don't use statfs on Apple systems, as for them it returns version-specific file system ids. * move mapping to a separate method and reuse it * minor improvement Co-authored-by: Adam Sitnik <[email protected]> Co-authored-by: Adam Sitnik <[email protected]>
dotnet · Aug 28, 2022 · fe99c44 · fe99c44
1 parent 211930d
commit fe99c44
Show file tree

Hide file tree

Showing 2 changed files with 142 additions and 130 deletions.
diff --git a/src/libraries/Common/src/Interop/Unix/System.Native/Interop.UnixFileSystemTypes.cs b/src/libraries/Common/src/Interop/Unix/System.Native/Interop.UnixFileSystemTypes.cs
@@ -155,7 +155,7 @@ internal static bool TryGetFileSystemType(SafeFileHandle fd, out UnixFileSystemT
         {
             uint fstatfsResult = GetFileSystemType(fd);
             fileSystemType = (UnixFileSystemTypes)fstatfsResult;
-            Debug.Assert(Enum.IsDefined(fileSystemType) || fstatfsResult == 0 || !OperatingSystem.IsLinux(), $"GetFileSystemType returned {fstatfsResult}");
+            Debug.Assert(Enum.IsDefined(fileSystemType) || fstatfsResult == 0, $"GetFileSystemType returned {fstatfsResult}");
             return fstatfsResult != 0;
         }
     }