Why not search for ANDROID_HOME first?

Abierto
#85 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
52/100
Tipo de issue
Refactorización
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
android, csharp
Área
tooling

Línea de trabajo

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.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Image 
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();
}
Lenguaje dominante
C#
Estrellas
78
Forks
18
Sin PR fusionados en 30 d

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de Redth/AndroidSdk.Tools

Todos los issues de Redth/AndroidSdk.Tools

Issues similares

Más issues de C#

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.