runtime icon indicating copy to clipboard operation
runtime copied to clipboard

74642 change isostorage path

Open mkhamoyan opened this issue 3 years ago • 11 comments

The path of a file created by IsolatedStorageFile.GetUserStoreForApplication() changes between Xamarin.iOS and .NET 6, and also changes between different builds of the application under .NET 6.

Switched back to the .isolatedstorage approach we had in legacy mono https://github.com/mono/mono/blob/main/mcs/class/corlib/System.IO.IsolatedStorage/IsolatedStorageFile.cs#L304-L327

Fix #74642

mkhamoyan avatar Sep 13 '22 15:09 mkhamoyan

Tagging subscribers to this area: @dotnet/area-system-io See info in area-owners.md if you want to be subscribed.

Issue Details

null

Author: mkhamoyan
Assignees: -
Labels:

area-System.IO

Milestone: -

ghost avatar Sep 13 '22 15:09 ghost

/azp run runtime-extra-platforms

mkhamoyan avatar Sep 13 '22 15:09 mkhamoyan

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Sep 13 '22 15:09 azure-pipelines[bot]

/azp run runtime-extra-platforms

mkhamoyan avatar Sep 16 '22 09:09 mkhamoyan

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Sep 16 '22 09:09 azure-pipelines[bot]

/azp run runtime-extra-platforms

mkhamoyan avatar Sep 16 '22 09:09 mkhamoyan

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Sep 16 '22 09:09 azure-pipelines[bot]

/azp run runtime-extra-platforms

mkhamoyan avatar Sep 16 '22 14:09 mkhamoyan

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Sep 16 '22 14:09 azure-pipelines[bot]

/azp run runtime-extra-platforms

mkhamoyan avatar Sep 20 '22 08:09 mkhamoyan

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Sep 20 '22 09:09 azure-pipelines[bot]

/azp run runtime-extra-platforms

mkhamoyan avatar Sep 26 '22 08:09 mkhamoyan

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Sep 26 '22 08:09 azure-pipelines[bot]

/azp run runtime-extra-platforms

mkhamoyan avatar Sep 26 '22 11:09 mkhamoyan

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Sep 26 '22 11:09 azure-pipelines[bot]

I'm imagining something like two different applications that just so happen to store their own version of a file file.txt and that the IdentityHash helps distinguish the files so that they wouldn't be unwantedly modified by a different app. If this is the case, then it seems like a bug for iOS that should be fixed (since it was stated that the hash stays the same on Android).

On disk, each app has it's own directory, so I'm not concerned about collisions between different apps.

steveisok avatar Sep 27 '22 13:09 steveisok

/backport to release/7.0

steveisok avatar Sep 27 '22 15:09 steveisok

Started backporting to release/7.0: https://github.com/dotnet/runtime/actions/runs/3137018195

github-actions[bot] avatar Sep 27 '22 15:09 github-actions[bot]

@mkhamoyan, I am looking at runtime outerloop failures. Have you checked the test failures in

Runtime-extra-platforms tests
-	System.Collections.Immutable.Tests.WorkItemExecution
-	System.Formats.Tar.Tests.WorkItemExecution
-	System.Net.Http.Functional.Tests.WorkItemExecution

JulieLeeMSFT avatar Sep 28 '22 20:09 JulieLeeMSFT

@mkhamoyan, I am looking at runtime outerloop failures. Have you checked the test failures in

This PR doesn't appear to have anything to do with the suites you listed. Are they tvOS failures? If so, can you provide a link to where you are looking? I believe they may be known intermittent issues with devices and not problems with tests.

steveisok avatar Sep 28 '22 20:09 steveisok