On Mon, Jan 22, 2024 at 03:11:35PM -0600, Ryan Eatmon wrote:
On 1/22/2024 6:59 AM, Max Krummenacher wrote:
Hi
On Fri, Jan 05, 2024 at 05:09:17PM +0530, Neha Malcom Francis wrote:
Use yamllint for checking whether YAML configuration files are adhering to default yamllint rules.
If I understand this correctly this patch now runs checks to verify that yaml files which are part of the U-Boot source tree are correct.
Shouldn't this be done when one commits a yaml file, i.e. in U-Boot CI rather than repeating the process on every build and thus having an additional dependency to build U-Boot and additional build time?
Note that in openembedded there is currently no python yamllint recipe providing the used python module in the regularly used layers. How do you plan to integrate the change in the OE U-Boot recipe? At least our OE CI of latest master now fails (for a TI AM62 based SoM) as the python module is missing.
I have submitted a patch to oe-core for this missing recipe.
https://lists.openembedded.org/g/openembedded-core/message/194192
Hopefully they accept it. Otherwise, we are going to "temporarily" carrying the same recipe in meta-ti-extras until they accept it in oe-core.
Hi Ryan
Thanks for looking into it. Max
Regards Max
-- Ryan Eatmon reatmon@ti.com
Texas Instruments, Inc. - LCPD - MGTS