Дублирование кода в целях makefile, как избежать?

Question

[Влад Зайцев]

ubrige: 
 make -j5 TARGET=unwired BOARD=$(BOARD)/$(CPU) -f makefiles/Makefile.$@
 ../bootloader/generate-metadata/generate-metadata $@.bin 0x0 0x01A00001 1
 srec_cat firmware-metadata.bin -binary $@.bin -binary -offset 0x100 -o $@-ota-image.bin -binary
 rm $@.hex

ubrige-fb: ubrige-CLEAN
  ../../tools/backdoor-bootloader.py -e -w -v ubrige-firmware.hex

ubrige-f: ubrige-CLEAN
 /Applications/ti/Uniflash/flash_cc1310.sh ubrige-firmware.hex 

ubrige-ota: ubrige
 scp ubrige-ota-image.bin root@192.168.20.187:/root/unwired_smarthome

Нетрудно заметить, что там часто повторяется имя цели(в данном случае ubrige). Это имя проекта. Если я создаю новый проект, то мне приходится копировать эти четыре цели, а потом заменять ubrige на новое имя проекта. Шут бы с заменой, но мне не нравится, что в файле куча одинакового кода.

Для первой цели я нашел $@, перестало требоваться заменять все названия проекта в командах.
Но она не уменьшает количество кода — все равно для каждого проекта надо скопировать несколько целей. В примере выше содержимое ubrige, ubrige-fb, ubrige-f, ubrige-ota будет одно и то же для всех проектов, отличатся только названием ubrige.

Как мне избавится от дублирования кода, который одинаковый для нескольких целей, однако, отличается именами файлов(пусть имена файлов совпадают с названием целей)?

Answer 1

[jcmvbkbc]

Как мне избавится от дублирования кода, который одинаковый для нескольких целей, однако, отличается именами файлов(пусть имена файлов совпадают с названием целей)?

Можно написать макрос-генератор целей. Изменяющиеся параметры можно передать как параметры макроса, внутри на них можно ссылаться по номеру. Например так:

define generate-targets
$(1): 
 make -j5 TARGET=unwired BOARD=$$(BOARD)/$$(CPU) -f makefiles/Makefile.$$@
 ../bootloader/generate-metadata/generate-metadata $$@.bin 0x0 0x01A00001 1
 srec_cat firmware-metadata.bin -binary $$@.bin -binary -offset 0x100 -o $$@-ota-image.bin -binary
 rm $$@.hex

$(1)-fb: $(1)-CLEAN
  ../../tools/backdoor-bootloader.py -e -w -v $(1)-firmware.hex

$(1)-f: $(1)-CLEAN
 /Applications/ti/Uniflash/flash_cc1310.sh $(1)-firmware.hex 

$(1)-ota: $(1)
 scp $(1)-ota-image.bin root@192.168.20.187:/root/unwired_smarthome

endef

Внутри макроса нужно экранировать $ заменяя его на $$.
Этот макрос можно вызывать в нужном месте следующим образом:

$(eval $(call generate-targets,ubrige))

Сгенерированный в этом месте код будет эквивалентен коду из вопроса.

Answer 2

[res2001]

Используйте переменные:

PROJECT=ubridge

$(PROJECT)-fb: $(PROJECT)e-CLEAN
  ../../tools/backdoor-bootloader.py -e -w -v $(PROJECT)-firmware.hex

$(PROJECT)-f: $(PROJECT)-CLEAN
 /Applications/ti/Uniflash/flash_cc1310.sh $(PROJECT)-firmware.hex

Comments

[res2001]

Для одинакового кода в целях можно использовать "процедуры" make.
Посмотрите пример.

[rustler2000]

res2001, проще как вы указали через переменную плюс include .mk с общими правилами.

Процедуры обычно очень хлопотно дебажить.

[rustler2000]

res2001, а еще переменные прекрасно переопределяются без вторжения в makefile\.mk, что оч удобно для всякого CI

[Влад Зайцев]

res2001, я все равно не понял, а как оно уменьшит количество кода? Т.е. мне же все равно надо будет повторять эти строки для каждого проекта?

PROJECT=ubridge

$(PROJECT)-fb: $(PROJECT)e-CLEAN
  ../../tools/backdoor-bootloader.py -e -w -v $(PROJECT)-firmware.hex

PROJECT=root

$(PROJECT)-fb: $(PROJECT)e-CLEAN
  ../../tools/backdoor-bootloader.py -e -w -v $(PROJECT)-firmware.hex

Или можно вынести их в отдельный файл и после каждого переопределения переменной делать include?

[res2001]

Влад Зайцев, В новом проекте вы просто можете скопировать makefile и в одном месте исправить название проекта.
Или как предложил rustler2000 строки
$(PROJECT)-fb: $(PROJECT)e-CLEAN
../../tools/backdoor-bootloader.py -e -w -v $(PROJECT)-firmware.hex
и т.п. вынести в отдельный makefile и подключать его с помощью include в makefile проекта, который содержит инициализацию переменной PROJECT.
Либо даже можно обойтись без makefile проекта и инициализировать PROJECT при запуске make (об этом то же писал rustler2000 ):
make /f makefile PROJECT=ubridge

[Влад Зайцев]

res2001, нет, проблема в том, что у меня всегда в одном makefile несколько проектов, которые отличаются только названием проекта в файлах.

[res2001]

Влад Зайцев, Зачем?
Сделайте 1 makefile с использованием PROJECT
сделайте командный файл с запуском make с инициализацией PROJECT, как я указал выше.
И у вас останется только один короткий makefile.
На каждый новый проект просто будете добавлять в файл командный файл еще одну строку с измененным именем проекта.

[res2001]

Итого у вас будет 2 небольших простых файла

[rustler2000]

res2001, это очень хорошо для ci build matrix