Make "Personal" on Android also point to a Documents folder #76250
Conversation
dotnet#68610 moved `Personal`/`MyDocuments` on Unix systems from HOME to the documents folder. However, it didn't do so for Android systems, which was only noticed after tests later failed. This fixes it.
|
I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label. |
ghost
added
the
community-contribution
|
Tagging subscribers to this area: @dotnet/area-system-runtime Issue Details#68610 moved If this gets merged, dotnet/docs#31423 should get adjusted to include Android as well.
|
|
/azp run runtime-extra-platforms |
|
Azure Pipelines successfully started running 1 pipeline(s). |
steveisok
left a comment
steveisok
left a comment
There was a problem hiding this comment.
Android tests now pass. Thanks for the PR!
eerhardt
left a comment
eerhardt
left a comment
There was a problem hiding this comment.
LGTM. I'll update the breaking change doc once this is merged.
While looking at #76250 I noticed that we were querying the HOME env variable for SpecialFolder.UserProfile but we're already doing that in PersistedFiles.GetHomeDirectory()
While looking at #76250 I noticed that we were querying the HOME env variable for SpecialFolder.UserProfile but we're already doing that in PersistedFiles.GetHomeDirectory()
|
Updated dotnet/docs#31423 to include the Android behavior change. |
ghost
locked as resolved and limited conversation to collaborators
5 participants
#68610 moved
Personal/MyDocumentson Unix systems from HOME to the documents folder. However, it didn't do so for Android systems, which was only noticed after tests later failed. This PR fixes it.If this gets merged, dotnet/docs#31423 should get adjusted to include Android as well.