Re: [PATCH v2 2/2] tools: binman: ti_board_cfg: Check for linting problems

[Max Krummenacher <max.oss.09@gmail.com>]

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