SHMUP: Restructure Makefile
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Refactoring
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Ambito
- build-system
Direzione di ricerca
Inizia leggendo il Makefile e la pull request collegata #57, quindi confronta le relative variabili, i percorsi e i passaggi di build in-tree o out-of-tree con la checklist di questa issue. Il lavoro è completato quando il Makefile è più semplice, usa la sintassi delle variabili richiesta e il progetto del tutorial continua a compilarsi correttamente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Assembly
- Stelle
- 179
- Fork
- 64
- Merge medio
- 7g 22h
- PR unite (30g)
- 1
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di gbdev/gb-asm-tutorial
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
gbdev/gb-asm-tutorial#194 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
gbdev/gb-asm-tutorial#184 · 1 commento ·
-
good first issue
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
gbdev/gb-asm-tutorial#141 · 2 commenti · 2 reazioni ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 65/100
gbdev/gb-asm-tutorial#109 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
gbdev/gb-asm-tutorial#183 ·
Tutte le issue di gbdev/gb-asm-tutorial
Issue simili
-
Name consistency Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
eellak/triplestore#65 · 1 commento ·
-
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
google-ai-edge/LiteRT-LM#3739 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
area-Bzlmod team-ExternalDeps type: bug untriaged
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
bazelbuild/bazel#31291 · 2 commenti ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
bradcypert/plum#53 ·