правила makefile, применяемые, когда они не должны

Question

У меня есть make-файл (вставленный ниже) для некоторого кода C ++, который вызывает досадную проблему: когда я выполняю make clean, файлы зависимостей компилируются (перед повторным удалением), что делает make clean слишком медленным. Правила зависимости:

$(DEPENDDIR)%.d: %.cpp
    @mkdir -p $(DEPENDDIR)
    $(CXX) -M -MG -MT $(OBJECTDIR)$*.o $(CXXFLAGS) $< > $@

Кто-нибудь может увидеть в чем проблема?

Я попытался переместить зависимости в ту часть, где объекты компилируются, например:

$(OBJECTDIR)%.o: %.cpp
    @mkdir -p $(OBJECTDIR) # $(dir $@)
    @echo " "
    $(CXX) -M -MG -MT $@ $(CXXFLAGS) $< \
        -MF $(patsubst $(OBJECTDIR)%.o, $(DEPENDDIR)%.d, $@)

но тогда файл заголовка version.hpp не был создан (и компиляция не удалась)

KERNEL := $(shell uname -s)

PROGNAME=nextsim.exec

CXX = g++

# setting the C++ standard according to the gcc compiler version (from gcc-5.2, the default is C++14)
ifeq ($(shell echo `$(CXX) -dumpversion | cut -f1-2 -d.` \>= 5.2 | sed -e 's/\.//g' | bc),1)
    CXXFLAGS += -std=c++14
else
    CXXFLAGS += -std=c++11
endif

# add g++ option flags
CXXFLAGS += -ftemplate-depth-256 -Wno-inline \
        -fPIC -fopenmp \
        -DHAVE_CONFIG_H -D_MULTITHREADING_
ifdef NEXTSIM_COMPILE_VERBOSE
    CXXFLAGS += -v
endif

ifdef USE_OASIS
    CXXFLAGS += -DOASIS
    CXXFLAGS += -I $(NEXTSIMDIR)/modules/oasis/include
    LDFLAGS += -lgfortran
    LDFLAGS += -L $(NEXTSIMDIR)/lib -loasis
    CHAN = MPI1
    #LIBPSMILE = $(OASIS_DIR)/lib/libpsmile.${CHAN}.a $(OASIS_DIR)/lib/libmct.a $(OASIS_DIR)/lib/libmpeu.a $(OASIS_DIR)/lib/libscrip.a
endif

ifneq (,$(strip $(filter DEBUG Debug debug PROFILE Profile profile,$(NEXTSIM_BUILD_TYPE))))
    #ifeq ($(NEXTSIM_BUILD_TYPE),$(filter $(NEXTSIM_BUILD_TYPE),Debug debug))
    CXXFLAGS := $(filter-out -O3 -pthread,$(CXXFLAGS))
    CXXFLAGS += -g -O0 -DNDEBUG
ifneq (,$(strip $(filter PROFILE Profile profile,$(NEXTSIM_BUILD_TYPE))))
    CXXFLAGS += -DWITHGPERFTOOLS
endif
ifneq ($(KERNEL),Linux)
    CXXFLAGS += -Wl,-no_pie
endif
else
    CXXFLAGS += -O3 -pthread
endif

# add include paths
CXXFLAGS += -I $(NEXTSIMDIR)/core/include
CXXFLAGS += -isystem $(NEXTSIMDIR)/contrib/bamg/include # suppress annoying compilation warnings from -I
CXXFLAGS += -isystem $(NEXTSIMDIR)/contrib/mapx/include # suppress annoying compilation warnings from -I
# CXXFLAGS += -I $(NEXTSIMDIR)/contrib/interp/include

ifdef USE_ENSEMBLE
    CXXFLAGS += -DENSEMBLE
    CXXFLAGS += -I $(NEXTSIMDIR)/modules/enkf/perturbation/include
endif

ifdef USE_AEROBULK
        CXXFLAGS += -I $(AEROBULK_DIR)/include
        CXXFLAGS += -DAEROBULK
endif

# openmpi
CXXFLAGS += -I $(OPENMPI_INCLUDE_DIR)/

# petsc
CXXFLAGS += -I $(PETSC_DIR)/include

# boost
CXXFLAGS += -I $(BOOST_INCDIR)/ -I .

# netcdf
CXXFLAGS += -I $(NETCDF_DIR)/include

# gmsh
CXXFLAGS += -I $(GMSH_DIR)/include/gmsh

CXXFLAGS += -I /opt/local/include

ifeq ($(KERNEL),Linux)
    #CXXFLAGS += -std=c++0x -std=c++11 -pedantic -ftemplate-depth-256 -Wno-inline -fPIC -g -lm -pthread -v #-MMD -MP -lm -pthread -v

else

    ifeq ($(CXX),clang)
    CXXFLAGS += -stdlib=libc++
    endif

    CXXFLAGS += -I /usr/local/include #-I /opt/local/include/openmpi-mp

    #LDFLAGS += -Wl,-rpath,/usr/local/lib #-Wl,-rpath,/opt/local/lib/openmpi-mp
    #LDFLAGS += -L /usr/local/lib #-L /opt/local/lib/openmpi-mp -lmpi_cxx -lmpi -ldl -lstdc++ -lpthread

    ifeq ($(CXX),clang)
    LDFLAGS += -stdlib=libc++
    endif

endif

LDFLAGS += -L /usr/local/lib

LDFLAGS += -Wl,-rpath,$(OPENMPI_LIB_DIR)/
ifndef MACHINE_HEXAGON
    LDFLAGS += -L $(OPENMPI_LIB_DIR)/ -lmpi_cxx -lmpi -ldl -lstdc++ #-lpthread
else
    LDFLAGS += -L $(OPENMPI_LIB_DIR)/ -lmpichcxx -lmpich -ldl -lstdc++ #-lpthread #-lssl -luuid -lpthread -lrt
        LDFLAGS += -Wl,-rpath,$(BLAS_LAPACK_DIR)/lib
        LDFLAGS += -L $(BLAS_LAPACK_DIR)/lib -lsci_gnu_mp
endif

LDFLAGS += -Wl,-rpath,$(NETCDF_DIR)/lib -L $(NETCDF_DIR)/lib -lnetcdf_c++4

LDFLAGS += -Wl,-rpath,$(BOOST_LIBDIR)
LDFLAGS += -L $(BOOST_LIBDIR) -lboost_program_options -lboost_filesystem -lboost_system -lboost_serialization -lboost_mpi -lboost_date_time

LDFLAGS += -Wl,-rpath,$(PETSC_DIR)/lib
LDFLAGS += -L $(PETSC_DIR)/lib -lpetsc

LDFLAGS += -Wl,-rpath,$(NEXTSIMDIR)/lib
LDFLAGS += -L $(NEXTSIMDIR)/lib -lbamg
#LDFLAGS += -L $(NEXTSIMDIR)/lib -linterp
LDFLAGS += -L $(NEXTSIMDIR)/lib -lmapx
#LDFLAGS += -L $(NEXTSIMDIR)/lib -loasis

ifdef USE_ENSEMBLE
    LDFLAGS += -L $(NEXTSIMDIR)/lib -lpseudo2D
    LDFLAGS += -lgfortran
endif

ifdef USE_AEROBULK
        LDFLAGS += -L $(AEROBULK_DIR)/lib -laerobulk_cxx -laerobulk
        LDFLAGS += -lgfortran
endif

ifneq (,$(strip $(filter DEBUG Debug debug PROFILE Profile profile,$(NEXTSIM_BUILD_TYPE))))
#ifeq ($(NEXTSIM_BUILD_TYPE),$(filter $(NEXTSIM_BUILD_TYPE),Debug debug))
    LDFLAGS += -Wl,-rpath,/opt/local/lib
ifneq (,$(strip $(filter PROFILE Profile profile,$(NEXTSIM_BUILD_TYPE))))
    LDFLAGS += -L /opt/local/lib -lprofiler
endif
endif

LDFLAGS += -L $(NEXTSIMDIR)/lib -lnextsim

OBJECTDIR=$(NEXTSIMDIR)/objs/
DEPENDDIR=$(NEXTSIMDIR)/.deps/
BINARYDIR=bin/

# C++ files
CXXSRCDIR=.
CXXHDRDIR=.
CXXSRC=$(wildcard $(CXXSRCDIR)/*.cpp)
# We must exclude the version.hpp file from the list of header files because otherwise we get a circular dependency
CXXHDR=$(filter-out $(CXXHDRDIR)/version.hpp, $(wildcard $(CXXHDRDIR)/*.hpp))

OBJS=$(CXXSRC:%.cpp=$(OBJECTDIR)%.o)
DEPS=$(CXXSRC:%.cpp=$(DEPENDDIR)%.d)

# Rules to always execute.
.PHONY: exec clean mrproper all cleanall mrproperall

# Default action.
exec: $(PROGNAME)

# Create a header file with the git version
version.hpp: version.sh $(CXXSRC) $(CXXHDR)
    $(SHELL) -x $<

# Delete the object files.
clean:
    @echo " "
    $(RM) $(OBJS) $(DEPS)
    @echo " "

mrproper: clean
    $(RM) $(BINARYDIR)$(PROGNAME)
    @echo " "

# Rule for making the actual target
lines="=========="
Lines=$(lines)$(lines)$(lines)$(lines)$(lines)$(lines)$(lines)$(lines)
$(PROGNAME): $(OBJS) #$(CCOBJS)
    @mkdir -p $(BINARYDIR)
    @echo " "
    @echo $(Lines)$(Lines)
    @echo "Creating executable: $(BINARYDIR)$(PROGNAME)"
    @echo $(Lines)$(Lines)
    @echo " "
    $(CXX) $(CXXFLAGS) -o $(BINARYDIR)$@ $^ $(LDFLAGS)
    @echo " "
    @echo $(Lines)$(Lines)
    @echo "Created executable: $(BINARYDIR)$(PROGNAME)"
    @echo $(Lines)$(Lines)
    @echo " "

# Rules for object files from cpp files
$(OBJECTDIR)%.o: %.cpp
    @mkdir -p $(OBJECTDIR) # $(dir $@)
    @echo " "
    $(CXX) -o $@ -c $< $(CXXFLAGS)

# Make dependancy rules
$(DEPENDDIR)%.d: %.cpp
    @mkdir -p $(DEPENDDIR)
    $(CXX) -M -MG -MT $(OBJECTDIR)$*.o $(CXXFLAGS) $< > $@

# The compilation depends on this Makefile.
$(OBJS): Makefile

# Make everything
all:
    cd ..; $(MAKE) all


# Clean everything
cleanall:
    cd ..; $(MAKE) clean

# Properly clean everything
mrproperall:
    cd ..; $(MAKE) mrproper

# Properly clean & recompile
fresh:
    cd ..; $(MAKE) fresh

-include $(DEPS)

Answer 1

Он строится, потому что вы используете -include $(DEPS).

См. Включая другие файлы Makefile

Если включенный make-файл не может быть найден ни в одном из этих каталогов, предупреждающее сообщение генерируется, но не сразу ошибка; обработка make-файла, содержащего include, продолжается. Как только он закончит чтение make-файлов, make попытается переделать любой которые устарели или не существуют. Посмотрите, как переделаны Makefiles. Только после того, как он попытался найти способ переделать make-файл и потерпел неудачу, сделает диагностику отсутствующего make-файла фатальной ошибкой.

Затем он продолжает:

Если вы хотите, чтобы make просто игнорировал make-файл, который не существует, или не может быть переделан, без сообщения об ошибке, используйте директиву -include вместо включения, вот так: -include filenames…

Это действует как include во всех отношениях, за исключением того, что нет ошибки (даже предупреждения), если любое из имен файлов (или любое другое) предпосылки любого из имен файлов) не существуют или не могут быть переделан.

Правда, это, вероятно, можно интерпретировать по-разному. Я только что проверил, что он действительно пытается переделать файлы в строке -include, у которых есть явное правило, которое нужно переделать:

all:
        @echo building $@

foo.d :
        @echo building $@
        @touch $@

-include foo.d bar.d

Что дает:

 tmp> make all
 building foo.d
 building all

Итак, это оставляет интересный случай, как решить вашу проблему. Наличие отдельного правила для файлов зависимостей увеличивает время компиляции - теперь вы анализируете каждый исходный файл дважды - один раз для генерации .d и один раз для компиляции. Это не хорошо. Я считаю, что объединение правила зависимости и .o в ваших интересах. Затем сделайте подмножество $(OBJS), которое полагается на version.hpp, явно зависимым от него, и у вас все получится.

Answer 2

Вы должны сделать строку -include $(DEPS) условной - исключите ее из make-файла, когда MAKECMDGOALS содержит только «чистые» цели:

ifneq(,$(filter-out clean distclean clobber,$(MAKECMDGOALS)))
-include $(DEPS)
endif

Кроме того, остерегайтесь предположения, что cd будет успешным - используйте && после него - или, если вызываете Make из другого каталога, просто используйте его опцию -C.