Makefile_added

[madza93]

No description provided.

[ziobron]

Skomentowane bezpośrednio w pliku Makefile. W razie niejasności pytaj :)

[ziobron]

-std=c++17 przerzuć do CFLAGS.
Zwyczajowo też zmienna o nazwie CFLAGS odnosi się do języka C. W C++ nazywa się ją CXXFLAGS.
CC podobnie stosuje się w C, a w C++ nazywa się ją CXX, więc możesz zmienić te nazwy :)

Pod zmienną OBJECTS umieszcza się pliki obiektowe, czyli te które są po etapie kompilacji, z rozszerzeniem* .o.
W zmiennej SOURCES umieszcza się pliki *.cpp

[ziobron]

Użyj też proszę zmiennych $@ i $< zarówno w celu debug jak i w release
https://stackoverflow.com/questions/3220277/what-do-the-makefile-symbols-and-mean

[ziobron]

Już jest coraz bliżej prawidłowego rozwiązania :-) jeszcze tylko drobne poprawki.

[ziobron]

Zmiennej $< nie musisz definiować. Ta zmienna jest automatycznie dostępna w każdym celu i oznacza pierwszy element z listy zależności, czyli zarówno dla debug jak i release będzie ona miała wartość *.cpp, a więc może zastąpić zmienną $(SOURCES), która tak naprawdę może trafić na listę zależności właśnie :-)
Zmienną $@ używasz poprawnie :)

[ziobron]

Aby było w pełni poprawnie to brakuje Ci kropki - *.cpp. Bez tego do zmiennej SOURCES załapałby się np taki plik jak hellocpp

[ziobron]

Hej, zobacz zmiany Martyny: https://github.com/LordLukin/modern_cpp/pull/2/files
O coś takiego chodzi ;)

[ziobron]

Już cele debug i release powinny przy takim kodzie działać :)
Jeszcze all i clean wymagają poprawy, bo używasz tam $<, które tak naprawdę można użyć w debug i release, ale o nie nie będę się już czepiał.