Syntax error: end of file unexpected

Question

Имею следующий Makefile:

./object_files/server.o: ./source_files/server.c ./header_files/server.h
    if which gcc 
    then
    gcc -c ./source_files/server.c
    mv ./server.o ./object_files/server.o
    elif which clang
    then
    clang -c ./source_files/server.c
    mv ./server.o ./object_files/server.o
    else    
    echo -e "\033[31m\033[5mPlease, install a compilar for C language such as gcc or clang!!!\033[0m\n"
    fi
./executable_files/server: ./object_files/server.o
    if which gcc
    then
    gcc -o ./executable_files/server  ./object_files/server.o
    elif which clang
    then
    clang -o ./executable_files/server ./object_files/server.o
    else
    echo -e "\033[31m\033[5mPlease, install a compilar for C language such as gcc or clang!!!\033[0m\n"
    fi

Вывод консоли:

if which gcc 
/bin/sh: 1: Syntax error: end of file unexpected (expecting "then")
make: *** [Makefile:2: object_files/server.o] Error 2

а почему один раз не определить переменную в виде CC=gcc или СС=clang и потом просто подставлять $(CC) — KoVadim, Jul 22 '20 at 19:44
@KoVadim, если использовать «каноническое» имя переменной CC, то его и подставлять не надо (см. мой ответ). впрочем, не все программисты приемлют принцип «kiss». увы… — aleksandr barakin, Jul 22 '20 at 21:35
зависит от задачи, может человеку нужно два компилятора одновременно, тогда да, плохая идея. А может ему любой подходящий, тогда эту переменную можно по необходимости переопределить извне. Негоже makefile заниматься таким. — KoVadim, Jul 23 '20 at 07:03

score 1 · Answer 1 · answered Jul 22 '20 at 19:44

Каждая строчка в мейкфайле выполняется как отдельная команда, за исключением случаев, когда конец предыдущей экранирован. Так что:

./object_files/server.o: ./source_files/server.c ./header_files/server.h
  if which gcc;\
  then\
    gcc -c ./source_files/server.c;\
    mv ./server.o ./object_files/server.o;\
  elif which clang;\
  then\
    clang -c ./source_files/server.c;\
    mv ./server.o ./object_files/server.o;\
  else\   
    echo -e "\033[31m\033[5mPlease, install a compilar for C language such as gcc or clang!!!\033[0m\n";\
  fi

Я лично всегда подобные вещи просто кладу в отдельный скрипт.

Все зделан, но все равно есть эта ошибка. — Danny, Jul 22 '20 at 20:12

aleksandr barakin · Answer 2 · 2020-07-22T21:38:15.300

в программировании считается хорошим тоном переиспользовать существующий код
к чему все эти манипуляции с подкаталогами? kiss вроде никто пока не отменял. у программы gnu/make по этому поводу есть удобная «магия».

если избавиться от дублирования кода и поместить файлы server.c и server.h в текущий каталог, то ровно той же функциональности можно добиться гораздо более лаконичным (а потому проще поддерживаемым) кодом:

CC = $(shell command -v gcc || command -v clang)
ifeq ($(CC),)
$(error Please, install a compilar for C language such as gcc or clang!!!)
endif
server:
server.o: server.c server.h

p.s. подробнее о применённой «магии» (или в первоисточнике: implicit rules).

Стоит добавить, что это GNU Make. POSIX Make не умеет ни $(shell …), ни $(error …), ни ifeq. — Ainar-G, Jul 22 '20 at 21:38
хм. действительно, в процессе редактирования потерялось упоминание про gnu/make. надо добавить. — aleksandr barakin, Jul 22 '20 at 21:39

Syntax error: end of file unexpected

2 Answers2