Re: [PATCH 2/2] CI: Make use of buildman requirements.txt

[Neha Malcom Francis <n-francis@ti.com>]

Hi Tom

On 04/05/23 18:32, Tom Rini wrote:
> On Thu, May 04, 2023 at 09:42:54AM +0530, Neha Malcom Francis wrote:
>> Hi Tom,
>>
>> On 03/05/23 18:34, Tom Rini wrote:
>>> On Wed, May 03, 2023 at 11:27:20AM +0530, Neha Malcom Francis wrote:
>>>> Hi Tom
>>>>
>>>> Thanks for these patches!
>>>>
>>>> On 27/04/23 01:14, Tom Rini wrote:
>>>>> Now that buildman has a requirements.txt file we need to make use of it.
>>>>>
>>>>> Signed-off-by: Tom Rini <trini@konsulko.com>
>>>>> ---
>>>>>     .azure-pipelines.yml | 3 +++
>>>>>     .gitlab-ci.yml       | 4 ++++
>>>>>     2 files changed, 7 insertions(+)
>>>>>
>>>>
>>>> However, while trying to ensure CI/CD coverage, I'm running into this "
>>>> error 'No module named 'jsonschema'" for am62ax [1], any idea why after
>>>> building successfully for other devices?
>>>>
>>>>
>>>> [1] https://dev.azure.com/u-boot/u-boot/_build/results?buildId=6236&view=logs&jobId=6fe7c803-7a3b-5b46-f057-c1c62fd89ba1&j=22dc4ac5-ae35-5978-08ac-5f386151834e&t=fae48c67-4bb5-5f06-119f-00d23f780e3c
>>> o
>>>
>>> We need to have the requirements.txt file installed in any job that's
>>> using this part of binman now and I guess my patch above wasn't
>>> complete? I didn't fully check what happened on Azure due to the other
>>> problems (ie iot2050 boards not building).
>>>
>>
>> Probably, I'm not sure about how to rectify this. Could you have a look if
>> possible? Regarding iot2050, I have started working on it.
> 
> I see it now.  The "script" section at the bottom of
> .azure-pipelines.yml needs the pip install as well, I had missed that.
> Go ahead and take that up with your reposting of the series :)
> 

Thanks for catching that! Will add it in

-- 
Thanking You
Neha Malcom Francis