Technically a recipe can still set it just fine, they just have to set it
after the inherit, not before. This is likely still a good change to apply,
but wanted to clarify :)

On Tue, Oct 23, 2018 at 4:49 AM Jef Driesen <jef.drie...@niko.eu> wrote:

> Commit b1f10b18eaf6c8009e86863ca4a26f429de97082 added the NPM_INSTALL_DEV
> variable. But due to the use of simple assignment, a recipe can't override
> its
> value.
>
> Signed-off-by: Jef Driesen <jef.drie...@niko.eu>
> ---
>  meta/classes/npm.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/classes/npm.bbclass b/meta/classes/npm.bbclass
> index c351ff0..a5b155a 100644
> --- a/meta/classes/npm.bbclass
> +++ b/meta/classes/npm.bbclass
> @@ -22,7 +22,7 @@ def npm_oe_arch_map(target_arch, d):
>      return target_arch
>
>  NPM_ARCH ?= "${@npm_oe_arch_map(d.getVar('TARGET_ARCH'), d)}"
> -NPM_INSTALL_DEV = "0"
> +NPM_INSTALL_DEV ?= "0"
>
>  npm_do_compile() {
>         # Copy in any additionally fetched modules
> --
> 2.7.4
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>


-- 
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Senior Software Engineer, Mentor Graphics
-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to