Why not search for ANDROID_HOME first?

Open

#85 opened on Feb 23, 2026 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 2/5
Estimated time: 1-3 hours
Newbie friendliness: 52/100

Issue type: Refactor
Clarity: Mostly clear
Activity status: Stale
Tech stack: android, csharp
Domain: tooling

Research direction

Start in AndroidSdk/SdkLocator.cs at the PreferredPaths method, then compare its environment-variable order with the linked Android Studio Environment variables guidance. Determine the intended precedence for ANDROID_HOME and ANDROID_SDK_ROOT, simplify the method if appropriate, and confirm that the resulting paths preserve the expected SDK lookup behavior.

Written by the indexing model from the issue text.

Description

According to Android Studio's Environment variables page, ANDROID_SDK_ROOT is deprecated, so I am curious why in the PreferredPaths method under SDKLocator.cs, you search for in ANDROID_HOME before ANDROID_SDK_ROOT? Also, seems like that method could be simplified as follows.

public override string[] PreferredPaths()
{
	var items = new List<string>
	{
		"ANDROID_HOME",
		"ANDROID_SDK_ROOT"
	};
	var paths = new List<string>();
	var config = IsWindows
		? MonoDroidSdkLocator.ReadRegistry()
		: MonoDroidSdkLocator.ReadConfigFile();
	
	if (!string.IsNullOrEmpty(config.AndroidSdkPath)) paths.Add(config.AndroidSdkPath);

	paths.AddRange(items.Select(Environment.GetEnvironmentVariable).Where(path => !string.IsNullOrEmpty(path))!);
	return paths.ToArray();
}

Dominant language: C#
Stars: 78
Forks: 18
PR merge metrics: No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

CS0162 "Unreachable code detected" warning from a MSBuildTemp .tmp file in every game project Open

bug

Difficulty 1/5 Under an hour Newbie friendliness 90/100

stride3d/stride#3477 · Opened Sep 23, 2026
docs: update contribution guidelines to account for AI-generated contributions Open

Type: enhancement

Difficulty 2/5 1-3 hours Newbie friendliness 65/100

apache/arrow-adbc#4809 · Opened Sep 23, 2026
[quality-improver] CiRunSummaryAggregateReader.ValidateModule provenance cross-check guards are untested on the throwing path Open

type/automation type/tech-debt

Difficulty 2/5 1-3 hours Newbie friendliness 70/100

microsoft/testfx#11477 · Opened Sep 23, 2026
.NET 11 isolated: feed entries using the new `Azure.Functions.Sdk` MSBuild SDK are misdetected as in-process, so no templates are found Open

Difficulty 2/5 1-3 hours Newbie friendliness 65/100

microsoft/vscode-azurefunctions#5197 · 1 comment · Opened Sep 23, 2026
reactor-build-and-check says mur is 'selfhost only', but mur.exe ships in the same skill-kit zip as that skill Open

Difficulty 1/5 Under an hour Newbie friendliness 85/100

microsoft/microsoft-ui-reactor#1274 · Opened Sep 23, 2026

More C# issues