[Buildroot] [PATCH] docs/manual: Document ERLANG_FOOBAR_KEEP_DEPENDENCIES in rebar infra
thomas.petazzoni at bootlin.com
Mon Dec 3 21:14:18 UTC 2018
On Mon, 3 Dec 2018 12:19:47 +0100, Johan Oudinet wrote:
> +* +ERLANG_FOOBAR_KEEP_DEPENDENCIES+, to keep the dependencies
> + described in the rebar.config file. By default, the _rebar_
> + infrastructure removes such dependencies as a post-patch hook to
as a post-patch hook -> *in* a post-patch hook
> + ensure rebar does not download or compile them. Only set this
> + variable to a non-empty value if the _rebar_ infrastructure must not
> + modify the rebar.config file included in the package.
Now that you write down the semantic of the variable, I don't like it
anymore: setting to a non-empty value is clearly not what Buildroot
normally does for boolean variables. We use a YES/NO value.
So instead of this, could we change pkg-rebar.mk to do this:
$(2)_KEEP_DEPENDENCIES ?= NO
$(2)_POST_PATCH_HOOKS += remove-rebar-config-dependencies
And then the document should say that the accepted values are "YES" and
"NO" and that the infrastructure defaults to "NO".
Could you have a look at doing this ?
Thanks a lot!
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
More information about the buildroot