Considering that every command is run in its own shell, what is the best way to run a multi-line bash command in a makefile? For example, like this:
for i in `find`
do
all="$all $i"
done
gcc $all
You can use backslash for line continuation. However note that the shell receives the whole command concatenated into a single line, so you also need to terminate some of the lines with a semicolon:
foo:
for i in `find`; \
do \
all="$$all $$i"; \
done; \
gcc $$all
But if you just want to take the whole list returned by the find
invocation and pass it to gcc
, you actually don't necessarily need a multiline command:
foo:
gcc `find`
Or, using a more shell-conventional $(command)
approach (notice the $
escaping though):
foo:
gcc $$(find)
As indicated in the question, every sub-command is run in its own shell. This makes writing non-trivial shell scripts a little bit messy -- but it is possible! The solution is to consolidate your script into what make will consider a single sub-command (a single line).
$
by replacing with $$
;
between commands\
set -e
to match make's provision to abort on sub-command failure. You can also use set -e -o pipefail
to make sure errors in pipe commands cause the script to abort (note: this is a bashism, so requires SHELL := /bin/bash
or similar)()
or {}
to emphasize the cohesiveness of a multiple line sequence -- that this is not a typical makefile command sequenceHere's an example inspired by the OP:
mytarget:
{ \
set -e ;\
msg="header:" ;\
for i in $$(seq 1 3) ; do msg="$$msg pre_$${i}_post" ; done ;\
msg="$$msg :footer" ;\
echo msg=$$msg ;\
}
SHELL := /bin/bash
in your makefile to enable BASH-specific features such as process substitution.
Commented
May 28, 2015 at 4:29
{
is crucial to prevent interpretation of {set
as an unknown command.
Commented
Sep 7, 2015 at 3:42
{}
and ()
makes a big difference if you sometimes want to copy the script and run it directly from a shell prompt. You can wreak havoc on your shell instance by declaring variables, and especially modifying state with set
, inside of {}
. ()
prevents the script from modifying your environment, which is probably preferred. Example (this will end your shell session): { set -e ; } ; false
.
Commented
Oct 20, 2015 at 19:37
command ; ## my comment \` (the comment is between
;` and `\`). This seems to work fine except that if you run the command manually (by copy-and-paste), the command history will include the comment in a way that breaks the command (if you try to reuse it). [Note: The syntax highlighting is broken for this comment due to the use of backslash inside backtick.]
Commented
Nov 11, 2015 at 5:29
The ONESHELL directive allows to write multiple line recipes to be executed in the same shell invocation.
all: foo
SOURCE_FILES = $(shell find . -name '*.c')
.ONESHELL:
foo: ${SOURCE_FILES}
FILES=()
for F in $^; do
FILES+=($${F})
done
gcc "$${FILES[@]}" -o $@
There is a drawback though : special prefix characters (‘@’, ‘-’, and ‘+’) are interpreted differently.
https://www.gnu.org/software/make/manual/html_node/One-Shell.html
Of course, the proper way to write a Makefile is to actually document which targets depend on which sources. In the trivial case, the proposed solution will make foo
depend on itself, but of course, make
is smart enough to drop a circular dependency. But if you add a temporary file to your directory, it will "magically" become part of the dependency chain. Better to create an explicit list of dependencies once and for all, perhaps via a script.
GNU make knows how to run gcc
to produce an executable out of a set of .c
and .h
files, so maybe all you really need amounts to
foo: $(wildcard *.h) $(wildcard *.c)
What's wrong with just invoking the commands?
foo:
echo line1
echo line2
....
And for your second question, you need to escape the $
by using $$
instead, i.e. bash -c '... echo $$a ...'
.
EDIT: Your example could be rewritten to a single line script like this:
gcc $(for i in `find`; do echo $i; done)
Found this post while looking up how to detect if docker is running from a Makefile...didnt find an example so I wrote one. Here is the GNUmakefile in context...
https://github.com/RandyMcMillan/make-docker-start
OS :=$(shell uname -s)
export OS
.ONESHELL:
docker-start:## detect whether docker is running...
@( \
while ! docker system info > /dev/null 2>&1; do\
echo 'Waiting for docker to start...';\
if [[ '$(OS)' == 'Linux' ]]; then\
systemctl restart docker.service;\
fi;\
if [[ '$(OS)' == 'Darwin' ]]; then\
open --background -a /./Applications/Docker.app/Contents/MacOS/Docker;\
fi;\
sleep 1;\
done\
)
docker-pull:docker-start## pull alpine image
docker pull alpine