NativeScript / NativeScript/nativescript-cli
CLI does not handle deleted files from App_Resources/Android/src
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 1.1k
- Forks
- 204
- Merge medio
- 1 d 9 h
- PR fusionados (30 d)
- 8
Descripción
If a user adds some Java/Kotlin files in the App_Resources/Android/src folder and runs a build, the files are copied into the platforms/android/app/src folder.
If the user, however, deletes some file in App_Resources/Android/src and triggers a second build, this does not change the state of the platforms/android/app/src directory.
Issue seems to be due to this logic: https://github.com/NativeScript/nativescript-cli/blob/master/lib/services/android-project-service.ts#L294
The whole implementation of the prepareAppResources method could probably be improved with setting an additional value provided to srcDir in the build.gradle
sourceSets {
main {
java {
srcDir 'src/src'
}
}
}
As for the libs folder, it's probably a good idea to attach it as an implementation filetree(<libs_path>) in the dependencies block in the build.gradle
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza en lib/services/android-project-service.ts, en prepareAppResources, e inspecciona el manejo de los sourceSets y dependencies relacionados de build.gradle. Reproduce el problema compilando, eliminando un archivo bajo App_Resources/Android/src y compilando de nuevo; estará terminado cuando el estado generado de platforms/android/app/src refleje tanto las eliminaciones como las adiciones.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- android, java, javascript, kotlin
- Área
- build-system, cli, mobile
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100