SHMUP: Restructure Makefile
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Área
- build-system
Línea de trabajo
Empieza leyendo el Makefile y el pull request enlazado #57; después, compara sus variables, rutas y pasos de compilación in-tree o out-of-tree con la lista de comprobación de este issue. Se considera terminado cuando el Makefile es más sencillo, usa la sintaxis de variables solicitada y el proyecto del tutorial se sigue compilando correctamente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Continuing the discussion in https://github.com/gbdev/gb-asm-tutorial/pull/57:
@ISSOtm said:
Regarding the Makefile, I have a few primary suggestions right now. Most of them revolve around making the Makefile drastically simpler, following the theme of introducing new concepts incrementally.
- Remove the
SRCDIRvariable. The point of variables likeOBJDIRis to allow building out-of-tree in a flexible location; however, it makes imo no sense to allow pointing the Makefile at a different source tree, since it's inherently part of it. - Use
${}for variables instead of$(). This helps differentiate them from function calls (which only accept the latter syntax); this is especially useful when both kinds are nested. - Remove all of the
*DIRandRGB*variables. For a "directed" project like this, it should be fine to hardcode all paths; this will notably remove one layer of indirection, and allow the reader to familiarise themselves with what that layer of indirection will "resolve" to. (There may be a case for keeping all of theRGB*variables, in case someone doesn't have RGBDS in their path, or under different names? Idk.) - Build in-tree. Building out-of-tree is better for a lot of reasons, but adds significant complexity (creating output directories, doing more path substitution instead of just extension rewriting, etc.). Again, since this is a "controlled environment", it should be fine to do things slightly dirty if it helps introducing concepts gradually; additionally, I believe experiencing the downsides of in-tree builds should help justify the extra complexity of out-of-tree builds when they're introduced later.
- Lenguaje dominante
- Assembly
- Estrellas
- 179
- Forks
- 64
- Merge medio
- 7 d 22 h
- PR fusionados (30 d)
- 1
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.
Más de gbdev/gb-asm-tutorial
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
gbdev/gb-asm-tutorial#194 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
gbdev/gb-asm-tutorial#184 · 1 comentario ·
-
good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
gbdev/gb-asm-tutorial#141 · 2 comentarios · 2 reacciones ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
gbdev/gb-asm-tutorial#109 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
gbdev/gb-asm-tutorial#183 ·
Todos los issues de gbdev/gb-asm-tutorial
Issues similares
-
core dependencies
Dificultad 1/5 Menos de una hora Aptitud para principiantes 80/100
-
bug github_actions
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
registrystack/registry-stack#1393 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Name consistency Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
eellak/triplestore#65 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100