mbox series

[for,2.10,v2,0/6] Various docker fixes

Message ID 20170725133425.436-1-alex.bennee@linaro.org
Headers show
Series Various docker fixes | expand

Message

Alex Bennée July 25, 2017, 1:34 p.m. UTC
Hi,

Phillipe pointed out a few anomalies with the travis.docker image
during review which has led to a couple more patches. As they are bug
fixes I didn't race to get a pull-req out today for hard-freeze. I
will roll a pull-req by the end of the week if there are no objections
as I'll be away for the rc1/2 cycles.

Alex Bennée (6):
  docker: ensure NOUSER for travis images
  docker: fix dirty/stash detection on some systems
  docker: ignore submodules when checking diff
  docker: docker.py make --no-cache skip checksum test
  docker: don't install device-tree-compiler build-deps in travis.docker
  docker: reduce noise when building travis.docker

 tests/docker/Makefile.include          | 3 ++-
 tests/docker/docker.py                 | 3 ++-
 tests/docker/dockerfiles/travis.docker | 6 ++++--
 3 files changed, 8 insertions(+), 4 deletions(-)

-- 
2.13.0

Comments

Fam Zheng Aug. 31, 2017, 6:59 a.m. UTC | #1
On Tue, 07/25 14:34, Alex Bennée wrote:
> Hi,

> 

> Phillipe pointed out a few anomalies with the travis.docker image

> during review which has led to a couple more patches. As they are bug

> fixes I didn't race to get a pull-req out today for hard-freeze. I

> will roll a pull-req by the end of the week if there are no objections

> as I'll be away for the rc1/2 cycles.

> 

> Alex Bennée (6):

>   docker: ensure NOUSER for travis images

>   docker: fix dirty/stash detection on some systems

>   docker: ignore submodules when checking diff

>   docker: docker.py make --no-cache skip checksum test

>   docker: don't install device-tree-compiler build-deps in travis.docker

>   docker: reduce noise when building travis.docker


Thanks, I've queued patches 1, 4, 5, 6 for 2.11. I intend to drop the code
touched by 2 and 3 with a dedicated script:

https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg06349.html
https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg06358.html

(It would be great if you can give some test/review there :)

Fam

> 

>  tests/docker/Makefile.include          | 3 ++-

>  tests/docker/docker.py                 | 3 ++-

>  tests/docker/dockerfiles/travis.docker | 6 ++++--

>  3 files changed, 8 insertions(+), 4 deletions(-)

> 

> -- 

> 2.13.0

>
Alex Bennée Sept. 5, 2017, 7:54 a.m. UTC | #2
Fam Zheng <famz@redhat.com> writes:

> On Tue, 07/25 14:34, Alex Bennée wrote:

>> Hi,

>>

>> Phillipe pointed out a few anomalies with the travis.docker image

>> during review which has led to a couple more patches. As they are bug

>> fixes I didn't race to get a pull-req out today for hard-freeze. I

>> will roll a pull-req by the end of the week if there are no objections

>> as I'll be away for the rc1/2 cycles.

>>

>> Alex Bennée (6):

>>   docker: ensure NOUSER for travis images

>>   docker: fix dirty/stash detection on some systems

>>   docker: ignore submodules when checking diff

>>   docker: docker.py make --no-cache skip checksum test

>>   docker: don't install device-tree-compiler build-deps in travis.docker

>>   docker: reduce noise when building travis.docker

>

> Thanks, I've queued patches 1, 4, 5, 6 for 2.11. I intend to drop the code

> touched by 2 and 3 with a dedicated script:

>

> https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg06349.html

> https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg06358.html

>

> (It would be great if you can give some test/review there :)


Cool - I shall have a look.

>

> Fam

>

>>

>>  tests/docker/Makefile.include          | 3 ++-

>>  tests/docker/docker.py                 | 3 ++-

>>  tests/docker/dockerfiles/travis.docker | 6 ++++--

>>  3 files changed, 8 insertions(+), 4 deletions(-)

>>

>> --

>> 2.13.0

>>



--
Alex Bennée