@mcb30 thank you for your explanation. I found the sentiment you referenced:
> When make runs a recipe, variables defined in the makefile are placed into
> the environment of each shell.
> ...
> Other use of variables from the environment is not recommended. It is not
> wise for makefiles to
Closed #115.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/ipxe/ipxe/pull/115#event-3503395947___
ipxe-devel mailing list
ipxe-devel@lists.ipxe.org
@mcb30 I can't tell if you're biding your time in thoughtful contemplation or
giving me the "Slow No". I want to believe it's the former, but --- since this
patch is a literal single character --- I doubt that optimism is justified. If
you're _not_ going to merge it, then could you please close
Found it:
> Any target in the makefile may be specified as a goal (unless it starts with
> ‘-’ or contains an ‘=’, in which case it will be parsed as a switch or
> variable definition, respectively).
-- https://www.gnu.org/software/make/manual/make.html#Goals
I make my living on exactly this
> What problem is this actually solving?
Allow me to demonstrate:
```
$ echo && cat Makefile.bad && echo && EMBED=foobar make -f Makefile.bad && echo
&& cat Makefile.good && EMBED=foobar make -f Makefile.good
EMBED := $(EMBEDDED_IMAGE)
.PHONY: all
all:
echo "EMBEDDED_IMAGE:
What problem is this actually solving? Any variables specified in the `make`
command line will override assignments within the Makefile. Building with
`make EMBED=...` works with no problems already.
--
You are receiving this because you are subscribed to this thread.
Reply to this email
@danderson approved this pull request.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/ipxe/ipxe/pull/115#pullrequestreview-432819509___
ipxe-devel mailing
You can view, comment on, or merge this pull request online at:
https://github.com/ipxe/ipxe/pull/115
-- Commit Summary --
* fix EMBED: dont eagerly assign over non-empty value
-- File Changes --
M src/Makefile.housekeeping (2)
-- Patch Links --
8 matches
Mail list logo