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

[Tom Rini]

On Fri, May 15, 2020 at 03:51:22PM +0200, Pali Rohár wrote:
> On Friday 15 May 2020 09:48:48 Tom Rini wrote:
> > On Fri, May 15, 2020 at 03:46:02PM +0200, Pali Rohár wrote:
> > > On Friday 15 May 2020 09:20:20 Tom Rini wrote:
> > > > 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.
> > > 
> > > Tom, sorry, but I grepped whole u-boot source code repository and I did
> > > not find any documentation nor README nor any other information how to
> > > run / extend or modify this service. That is why I asked for some
> > > information... e.g. how I can I run it and check if it is working or
> > > not.
> > > 
> > > > > 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.
> > > 
> > > I prepared this N900 Travis setup for your request [1] and I do not like
> > > to see it thrown away, just because there is unrelated issue on Azure.
> > > 
> > > I have used Travis before, so I know that opening pull request on github
> > > triggers Travis build and Github directly shows me links to result.
> > > 
> > > But whatever I did, I was not able to trigger that azure from github
> > > pull request.
> > > 
> > > [1] - https://lists.denx.de/pipermail/u-boot/2018-December/353019.html
> > 
> > Sorry, I mean Azure itself is a well documented public SaaS CI tool.  It
> > plugs in to GitHub just as easy as Travis does, but runs quicker.
> 
> So seems it is buggy, it was not triggered, see that only Travis was
> triggered in pull request: https://github.com/u-boot/u-boot/pull/30

Did you configure your Azure account?  If you've used Travis elsewhere
that's why U-Boot just runs there.

-- 
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/5325da73/attachment.sig>