diff options
author | Tom Hochstein <tom.hochstein@nxp.com> | 2022-07-28 13:44:36 -0500 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2022-08-02 11:30:00 +0100 |
commit | 71d0913599930aa19b493164d09f9019d1c28fec (patch) | |
tree | 3b06562df478b61d7e1dcd61c644959aac929fd9 | |
parent | 06b27bb7981fccba2b5ae3a624c5d19bd018d518 (diff) | |
download | poky-71d0913599930aa19b493164d09f9019d1c28fec.tar.gz |
uboot-config.bbclass: Raise error for bad key
If an invalid key is used, the class ignores the error, with an
indeterminate result. In my case, the problem surfaced in do_deploy:
```
| cp: cannot stat '/.../build/tmp/work/imx6qdlsabresd-fsl-linux-gnueabi/u-boot-imx-mfgtool/2022.04-r0/deploy-u-boot-imx-mfgtool/u-boot.imx': No such file or
| directory
```
The root cause of this was that the uboot config key did not match
a valid option. With the fix, the error is caught by the class:
```
ERROR: Nothing PROVIDES 'u-boot-imx-mfgtool'
u-boot-imx-mfgtool was skipped: The selected UBOOT_CONFIG key ['mfgtool'] has no match in dict_keys(['sd-fslc', 'sd-imx', 'sd-optee-imx', 'sata-imx', 'mfgtool-imx']).
```
(From OE-Core rev: 801a27d73b10017cac3c0caa05d0a2af3502a7ba)
Signed-off-by: Tom Hochstein <tom.hochstein@nxp.com>
Acked-by: Otavio Salvador <otavio@ossystems.com.br>
Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/classes/uboot-config.bbclass | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/meta/classes/uboot-config.bbclass b/meta/classes/uboot-config.bbclass index b9ad35821a..e8da8c7452 100644 --- a/meta/classes/uboot-config.bbclass +++ b/meta/classes/uboot-config.bbclass | |||
@@ -125,5 +125,6 @@ python () { | |||
125 | else: | 125 | else: |
126 | bb.debug(1, "Appending '%s' to UBOOT_BINARIES." % ubootbinary) | 126 | bb.debug(1, "Appending '%s' to UBOOT_BINARIES." % ubootbinary) |
127 | d.appendVar('UBOOT_BINARIES', ' ' + ubootbinary) | 127 | d.appendVar('UBOOT_BINARIES', ' ' + ubootbinary) |
128 | break | 128 | return |
129 | raise bb.parse.SkipRecipe("The selected UBOOT_CONFIG key %s has no match in %s." % (ubootconfig, ubootconfigflags.keys())) | ||
129 | } | 130 | } |