Resolve "add makefile with structure"
Closes #3 (closed)
Merge request reports
Activity
assigned to @tom.ryser
requested review from @orestis.malaspin
removed review request for @orestis.malaspin
requested review from @orestis.malaspin
requested review from @paul.albuquer and removed review request for @orestis.malaspin
requested review from @orestis.malaspin and removed review request for @paul.albuquer
added DONE label
- Resolved by tom.ryser
While I understand the reason you want split things between include and source I don't think it makes much sense here.
Also keep the Makefile simple for the time being. Overcomplicating things from day one may cause sad problems in the long term.
What I suggest at the moment is have a simpler structure:
stack.h stack.c main.c
at the root.
Also adding so many defaults to the .gitignore is probably too much for a collaborative project. Keep things simple and avoid huge defaults.
- Resolved by tom.ryser
- Resolved by tom.ryser
- Resolved by tom.ryser
added 4 commits
-
aea23828...1ccc0ef1 - 3 commits from branch
main
- 071e3098 - Merge branch 'main' into '3-add-makefile-with-structure'
-
aea23828...1ccc0ef1 - 3 commits from branch
- Resolved by tom.ryser
- Resolved by tom.ryser
removed DONE label
- main.c 0 → 100644
1 #include <stdio.h> 2 #include "../include/stack.h" changed this line in version 9 of the diff
- Makefile 0 → 100644
1 CC:=gcc 2 CFLAGS:= -g -Wextra -pedantic -fsanitize=address 3 NAME:=stack 4 5 $(NAME): main.o $(NAME).o 6 $(CC) $(CFLAGS) -o $@ $^ -lSDL2 changed this line in version 9 of the diff
- Makefile 0 → 100644
1 CC:=gcc 2 CFLAGS:= -g -Wextra -pedantic -fsanitize=address changed this line in version 9 of the diff
@tom.ryser une dernière souffrance?
added 25 commits
-
bc2e9a5f...c4e5b4ad - 24 commits from branch
main
- 4ee8a912 - Merge branch 'main' into '3-add-makefile-with-structure'
-
bc2e9a5f...c4e5b4ad - 24 commits from branch
added 1 commit
- 18d33345 - fixed makefile, main compilation and test compilation
mentioned in commit 42523a25