Как я могу улучшить этот Makefile?

Question

Я мало знаю о создании Makefiles, но я читал make manual, и я сделал некоторый прогресс. Мой Makefile работает и делает то, что я хочу.

В моей ситуации обычно используется от 1 до 3 различных программ, которые необходимо скомпилировать и отправить в мою TA для маркировки и, например, через веб-форму. Структура каждого приложения - «prog.c», «prog_lib.h» и «prog_lib.c». Раньше я создавал отдельные каталоги для каждой программы и создавал отдельные Make-файлы для каждого каталога для сборки программы, содержащейся внутри. Затем я деактивирую каждую папку и отправляю их отдельно.

Недавно TA запросили, чтобы все исходные файлы находились в одном каталоге и один Makefile с различными целями, которые должны быть построены, чтобы их приложения для маркировки могли работать без какого-либо вмешательства человека.

Мне было интересно, как кто-то более опытный улучшит этот Makefile и как моя ситуация обычно будет решена? Я хотел бы уменьшить объем ввода, который мне нужно сделать, когда я перехожу к следующему заданию и должен обновить несколько мест.

Вот мой Makefile:

ASSIGNMENT = 3 
TARNAME = Assignment$(ASSIGNMENT).tar.bz2 

CC = gcc 
CFLAGS = -O2 -Wall -ansi -pedantic -W # I like warnings 
LDFLAGS = -lm 
DEBUG = -g # to resolve symbols in GDB and valgrind 

FREQ_OUT = frequency_table 
FREQ_SOURCES = frequency_table.c frequency_table_lib.c 
FREQ_OBJECTS = frequency_table.o frequency_table_lib.o 

DECODE_OUT = decode 
DECODE_SOURCES = decode.c decode_lib.c 
DECODE_OBJECTS = decode.o decode_lib.o 

SOURCES = $(FREQ_SOURCES) $(DECODE_SOURCES) 
OBJECTS = $(FREQ_OBJECTS) $(DECODE_OBJECTS) 
OUT = $(FREQ_OUT) $(DECODE_OUT) 

.PHONY: info 
info: 
    @echo -e "make info\n" \ 
      "\tmake all \t\t\tMake all targets\n" \ 
      "\tmake frequency_table \t\tMakes frequency table\n" \ 
      "\tmake decode \t\t\tMakes decode\n" \ 
      "\tmake dist \t\t\tMakes tar archive of sources and Makefile\n" \ 
      "\tmake clean \t\t\tRemoves all the object files and executables\n" \ 
      "\tmake distclean \t\t\tPerforms clean and removes tar archive" 

.PHONY: all 
all: $(OUT) 

$(FREQ_OUT): $(FREQ_OBJECTS) 
    $(CC) $(CFLAGS) $(DEBUG) $(LDFLAGS) $(FREQ_OBJECTS) -o $@ 

$(DECODE_OUT): $(DECODE_OBJECTS) 
    $(CC) $(CFLAGS) $(DEBUG) $(LDFLAGS) $(DECODE_OBJECTS) -o $@ 

.o: 
    $(CC) -c $(CFLAGS) -o $@ $< 

.PHONY: dist 
dist: $(SOURCES) 
    @echo "Creating tar archive. See $(TARNAME)" 
    tar cvjf $(TARNAME) $(SOURCES) $(wildcard *_lib.h) Makefile 

.PHONY: clean 
clean: 
    rm -f $(OUT) $(OBJECTS) 

.PHONY: distclean 
distclean: clean 
    rm -f $(TARNAME) 

Answer 1

Вам действительно не нужны линии $(CC) $(CFLAGS) $(DEBUG) $(LDFLAGS) $(FREQ_OBJECTS) -o $@. make уже знает, как создавать двоичные файлы.

Если ваши имена файлов являются постоянными для различных бинарных файлов (binary.c и binary_lib.c), вы также можете создать общее правило, что:

FOO := $(shell ls *_lib.c) 
BIN = $(FOO:%_lib.c=%) 

$(BIN) : % : %.o %_lib.o 

EDIT: Вот как это работает:

1. FOO список всех файлов, заканчивающихся _lib.c
2. BIN в тот же список, с «_lib.c» суффиксы удалены, так что список ваших бинарных файлов
3. Последняя строка - ваше правило make. Правило гласит, каждый Foo в $ (BIN) зависит от foo.o и foo_lib.o

Answer 2

По моему опыту, вы не должны нуждаться в .o цели там, это подразумевается.

Кроме того, неявная версия обычно включает в себя стоимость $(CPPFLAGS), в котором вы должны поставить любые -Iпути или -Dмакросов параметров, которые могут потребоваться.

Я бы также включил $(DEBUG) в $(CFLAGS), а не перечислил его явно в любой из целей сборки.

Answer 3

Пример:

# CC, CFLAGS, etc. go here 

# Object variables go here 
MYPROG_OBJECTS = main.o someother.o 

# all, info, etc. targets go here 

# first actual target: 
myprog: $(MYPROG_OBJECTS) 
    <Do the linking stuff> 

# This will take care of the objects, sources are placed in src directory: 
%.o: src/%.c 
    <The usual compilation commands> 

# Automatic dependency magic: 
%.d: src/%.c 
    $(CC) -MM -o$@ $< 

-include (MYPROG_OBJECTS:%.o=%.d) 

Answer 4

Сделать «все» предназначаться первый, если вы действительно, что ваш пользователи должны будут ввести что-то после «make», чтобы получить проект для сборки. Цель «info» хороша, но обычна.

(У меня есть один файл makefile, где цель по умолчанию - не все - в каталоге с исходным кодом для более 100 команд. Я не хочу, чтобы все было установлено «по умолчанию», я рассчитываю построить только один или два, которые я хочу построить.Существует «все». Но это очень необычно. Обычно значение по умолчанию должно быть «все».)

Кроме того, ни «$ (FREQ_OUT)», ни «$ (DECODE_OUT)» не является целью PHONY; они настоящие программы, не так ли? Цели «все», «информация», «dist», «чистый», «realclean» и т. Д. - это фальшивые. Но программ, которые вы создаете, нет.