[igt-dev] [PATCH i-g-t v3 1/1] gitlab-ci: add build and tests for MIPS

[Ser, Simon]

On Thu, 2019-06-27 at 16:19 +0300, Arkadiusz Hiler wrote:
> On Thu, Jun 27, 2019 at 04:14:53PM +0300, Ser, Simon wrote:
> > On Thu, 2019-06-27 at 11:02 +0100, Guillaume Tucker wrote:
> > > On 27/06/2019 08:02, Ser, Simon wrote:
> > > > On Tue, 2019-06-25 at 14:08 +0100, Guillaume Tucker wrote:
> > > > > On 18/06/2019 13:42, Guillaume Tucker wrote:
> > > > > > Add Docker image and Gitlab CI steps to run builds and tests for
> > > > > > the
> > > > > > MIPS architecture using Debian Stretch with backports.
> > > > > > 
> > > > > > Signed-off-by: Guillaume Tucker <guillaume.tucker at collabora.com>
> > > > > > ---
> > > > > > 
> > > > > > Notes:
> > > > > >     v2: use stretch-backports and require libatomic1
> > > > > >     v3: add mips ci tests and require Debian libatomic1 for mips
> > > > > 
> > > > > The series to use portable atomics functions was merged today, so
> > > > > I think this one should now be good to go as well.  It applies
> > > > > cleanly on top of the current master branch and the Gitlab CI
> > > > > pipeline passed:
> > > > > 
> > > > >   
> > > > > https://gitlab.freedesktop.org/gtucker/igt-gpu-tools/pipelines/44704
> > > > > 
> > > > > Please let me know if you want me to resubmit it to get another
> > > > > Patchwork CI run or if anything else needs to be done.
> > > > 
> > > > LGTM!
> > > > 
> > > > Reviewed-by: Simon Ser <simon.ser at intel.com>
> > > > 
> > > > And pushed:
> > > > 
> > > > To gitlab.freedesktop.org:drm/igt-gpu-tools.git
> > > >    15ad66453441..439a9f5d615f  master -> master
> > > 
> > > Thanks!
> > > 
> > > Err, however it looks like you pushed the v2 which had only
> > > builds rather than this v3 which does builds and tests:
> > > 
> > >   439a9f5d615f gitlab-ci: add build for MIPS
> > > 
> > > I've made another patch with the difference between v2 and v3 and
> > > pushed it to my branch:
> > > 
> > >   https://gitlab.freedesktop.org/gtucker/igt-gpu-tools/commit/9693e28871f27efb7340ad29d54de4be7b5461a9
> > > 
> > > I'll wait for the Gitlab CI pipeline to complete and then I guess
> > > I should send that to the mailing list.
> > 
> > Bleh, I'm sorry about this! It seems like patchwork got confused.
> > 
> > I'll gladly review and merge a fix, feel free to Cc me :)
> 
> The title of the first patch has changed, so patchwork treats it as a
> separate series instead of a revision to existing one.
> 
> It's safer to take the patchwork links (both to series and the mbox)
> from the CI results instead of trying to browse for them yourself.

Yeah, this makes sense. Thanks for the info!