From 220a34552ecf9184ff6572e7bc472185bee26960 Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=B4me=20Benoit?= Date: Wed, 1 Mar 2017 14:08:23 +0100 Subject: [PATCH] Update the exercice skeleton Makefile to the one given during the course. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Signed-off-by: Jérôme Benoit --- TP_9/exo2/clist.c | 2 +- exo_skel/Makefile | 86 ++++++++++++++++++++++++++++++++++++----------- 2 files changed, 68 insertions(+), 20 deletions(-) diff --git a/TP_9/exo2/clist.c b/TP_9/exo2/clist.c index 2857042..0ee532f 100644 --- a/TP_9/exo2/clist.c +++ b/TP_9/exo2/clist.c @@ -45,7 +45,7 @@ unsigned list_count(link_t* head) { void list_set(link_t* head, unsigned index, int value) { - // FIXME: check for the index value validity + //FIXME: check for the index value validity for (unsigned count = 0; count < index; count++) { head = head->next; } diff --git a/exo_skel/Makefile b/exo_skel/Makefile index 1822902..51c401a 100644 --- a/exo_skel/Makefile +++ b/exo_skel/Makefile @@ -1,31 +1,79 @@ -TARGET = exo_skel -LIBS = -CC = gcc -# Enforce C11 ISO standard for now -CFLAGS = -std=c11 -g -Wall -Wextra -LDFLAGS = -g -Wall -Wextra +# Sample Makefile to build simple project. +# +# This Makefile expect all source files (.c) to be at the same level, in the +# current working directory. +# +# It will automatically generate dependencies, compile all files, and produce a +# binary using the provided name. +# +# Set BINARY_NAME to the name of the binary file to build. +# Set BUILD_TYPE to either debug or release +# +# Automatic dependencies code from: +# http://make.mad-scientist.net/papers/advanced-auto-dependency-generation/#tldr +BINARY_NAME=exo_skel +BUILD_TYPE=debug -.PHONY: default all clean +# ==================================== +# DO NOT CHANGE STUFF BEYOND THIS LINE +# ==================================== -default: $(TARGET) -all: default +all: $(BINARY_NAME) -OBJECTS = $(patsubst %.c, %.o, $(wildcard *.c)) -HEADERS = $(wildcard *.h) +CC=gcc +LD=gcc -%.o: %.c $(HEADERS) - $(CC) $(CFLAGS) -c $< -o $@ +WARN_FLAGS = -Wall -Wextra +STD_FLAG = -std=c99 -.PRECIOUS: $(TARGET) $(OBJECTS) +ifeq ($(BUILD_TYPE),debug) +BUILDDIR := .build/debug +DEBUG_FLAG = -g +STRIP_FLAG = +OPTI_FLAG = -O0 +else +BUILDDIR := .build/release +DEBUG_FLAG = +STRIP_FLAG = -s +OPTIFLAG = -O3 +endif -$(TARGET): $(OBJECTS) - $(CC) $(OBJECTS) $(LDFLAGS) $(LIBS) -o $@ +CFLAGS := $(CFLAGS) $(WARN_FLAGS) $(STD_FLAG) $(OPTI_FLAG) $(DEBUG_FLAG) +LDFLAGS := $(LDFLAGS) $(STRIP_FLAG) + +OBJDIR := $(BUILDDIR)/objs +$(shell mkdir -p $(OBJDIR)) + +SRCS=$(wildcard *.c) +OBJS=$(patsubst %.c,$(OBJDIR)/%.o,$(SRCS)) + +DEPDIR := $(BUILDDIR)/deps +$(shell mkdir -p $(DEPDIR)) +DEPFLAGS = -MT $@ -MMD -MP -MF $(DEPDIR)/$*.Td +POSTCOMPILE = mv -f $(DEPDIR)/$*.Td $(DEPDIR)/$*.d + +$(BINARY_NAME): $(OBJS) + @echo "[LD ] $@" + @$(LD) $(CFLAGS) $(LDFLAGS) $^ $(LDLIBS) -o $@ + +$(OBJDIR)/%.o: %.c $(DEPDIR)/%.d + @echo "[C ] $*" + @$(CC) $(DEPFLAGS) $(CFLAGS) -c $< -o $@ + @$(POSTCOMPILE) + +$(DEPDIR)/%.d: ; + +.PRECIOUS: $(DEPDIR)/%.d + +include $(wildcard $(patsubst %,$(DEPDIR)/%.d,$(basename $(SRCS)))) clean: - -rm -f $(TARGET) $(OBJECTS) + @echo "[CLN]" + -@rm -r $(BUILDDIR) + -@rm $(BINARY_NAME) -disassemble: $(TARGET) +disassemble: $(BINARY_NAME) objdump -d $< | less -symbols: $(TARGET) +symbols: $(BINARY_NAME) objdump -t $< | sort | less -- 2.34.1