[PATCH v2] Nokia RX-51: Add automated test for running RX-51 build in qemu

[Tom Rini]

On Fri, May 15, 2020 at 09:33:47AM +0200, Pali Rohár wrote:
> On Thursday 14 May 2020 20:01:19 Tom Rini wrote:
> > On Fri, May 15, 2020 at 12:41:52AM +0200, Pali Rohár wrote:
> > > On Saturday 09 May 2020 16:56:10 Tom Rini wrote:
> > > > On Sat, May 09, 2020 at 06:35:40PM +0200, Pali Rohár wrote:
> > > > > On Saturday 09 May 2020 21:58:19 Lokesh Vutla wrote:
> > > > > > On 08/05/20 6:40 PM, Tom Rini wrote:
> > > > > > > On Fri, May 08, 2020 at 02:52:55PM +0200, Pali Rohár wrote:
> > > > > > >> On Tuesday 28 April 2020 09:37:21 Pali Rohár wrote:
> > > > > > >>> On Monday 27 April 2020 14:00:47 Tom Rini wrote:
> > > > > > >>>> I'll take care of that shortly.  Otherwise:
> > > > > > >>>>
> > > > > > >>>> Reviewed-by: Tom Rini <trini at konsulko.com>
> > > > > > >>>
> > > > > > >>> Ok, thank you!
> > > > > > >>
> > > > > > >> Hello Tom! Will you take whole patch series? Or is there anything else
> > > > > > >> needed to be done from my side for this N900 patch series?
> > > > > > > 
> > > > > > > Lokesh is the custodian for the TI tree these days, so whenever it's in
> > > > > > > his next PR.  Thanks!
> > > > > > > 
> > > > > > 
> > > > > > I see gitlab is failing to build this patch[0]. Does cross compiler needs to be
> > > > > > changed?
> > > > > > 
> > > > > > [0] https://gitlab.denx.de/u-boot/custodians/u-boot-ti/-/jobs/91228
> > > > > 
> > > > > Cross compiler arm-linux-gnueabi-gcc needs to be in $PATH.
> > > > > 
> > > > > I figured out that on Travis it is available in ~/.buildman-toolchains
> > > > > but not exported to $PATH. So for Travis build I added...
> > > > > 
> > > > > export PATH=~/.buildman-toolchains/gcc-9.2.0-nolibc/arm-linux-gnueabi/bin/:$PATH
> > > > > 
> > > > > ... as can be seen in the last patch.
> > > > > 
> > > > > Do you know where is installed arm-linux-gnueabi toolchain on Gitlab?
> > > > > Maybe Tom knows it as he already wrote that would take care of updating
> > > > > Gitlab image.
> > > > 
> > > > All of the buildman-fetched toolchains are always in the same place, so
> > > > a similar change to gitlab/azure will fix those.  Thanks!
> > > 
> > > I see that all patches except this one were merged, thanks.
> > > 
> > > Tom, are you going to take look at this last patch?
> > > 
> > > It already passed on travis [1] [2] but I do not have those gitlab and
> > > azure accounts to trigger their jobs. But I think that only correct
> > > $PATH is needed for azure and gitlab.
> > > 
> > > [1] - https://github.com/u-boot/u-boot/pull/30
> > > [2] - https://travis-ci.org/github/u-boot/u-boot/jobs/679162986
> > 
> > No, I've been waiting for you to make an attempt at fixing the jobs.
> > Anyone can get Azure running and there's enough examples to make a
> > reasonable attempt at making it work without testing.
> 
> So can you give me pointers how to run it?

It's a well documented public service.  The only slight trick is you
need to point it at .azure-pipeline.yml and not whatever the default
non-dotfile name is.

> And is there something more needed for travis job?

All 3 CIs need to pass, but no, if Travis is passing, that part is fine.
Since Azure/GitLab share the same docker image (which sadly I don't see
how to make Travis also do), that's why fixing Azure should let you see
what to drop in for GitLab.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200515/b672d920/attachment.sig>