docker: Sort packages

[Qemu-devel] [PATCH 0/4] docker: Sort packages

Philippe Mathieu-Daudé posted 4 patches 7 years, 2 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180818015344.797-1-f4bug@amsat.org

Test checkpatch passed

Test docker-mingw@fedora passed

Test docker-clang@ubuntu passed

Test docker-quick@centos7 passed

tests/docker/dockerfiles/centos7.docker       |   2 +-
.../dockerfiles/fedora-i386-cross.docker      |   9 +-
tests/docker/dockerfiles/fedora.docker        | 102 ++++++++++++++----
tests/docker/dockerfiles/ubuntu.docker        |  61 +++++++++--
4 files changed, 141 insertions(+), 33 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH 0/4] docker: Sort packages

Posted by Philippe Mathieu-Daudé 7 years, 2 months ago

Trivial cleanup patches.

Per the Docker recommendation, "this helps to avoid duplication of
packages and make the list much easier to update. This also makes PRs
a lot easier to read and review."

Regards,

Phil.

Philippe Mathieu-Daudé (4):
  docker: Sort Fedora packages
  docker: Sort Fedora packages (i386-cross)
  docker: Sort Ubuntu packages
  docker: Sort CentOS packages

 tests/docker/dockerfiles/centos7.docker       |   2 +-
 .../dockerfiles/fedora-i386-cross.docker      |   9 +-
 tests/docker/dockerfiles/fedora.docker        | 102 ++++++++++++++----
 tests/docker/dockerfiles/ubuntu.docker        |  61 +++++++++--
 4 files changed, 141 insertions(+), 33 deletions(-)

-- 
2.18.0

Re: [Qemu-devel] [PATCH 0/4] docker: Sort packages

Posted by Fam Zheng 7 years, 2 months ago

On Fri, 08/17 22:53, Philippe Mathieu-Daudé wrote:
> Trivial cleanup patches.
> 
> Per the Docker recommendation, "this helps to avoid duplication of
> packages and make the list much easier to update. This also makes PRs
> a lot easier to read and review."
> 
> Regards,
> 
> Phil.
> 
> Philippe Mathieu-Daudé (4):
>   docker: Sort Fedora packages
>   docker: Sort Fedora packages (i386-cross)
>   docker: Sort Ubuntu packages
>   docker: Sort CentOS packages
> 
>  tests/docker/dockerfiles/centos7.docker       |   2 +-
>  .../dockerfiles/fedora-i386-cross.docker      |   9 +-
>  tests/docker/dockerfiles/fedora.docker        | 102 ++++++++++++++----
>  tests/docker/dockerfiles/ubuntu.docker        |  61 +++++++++--
>  4 files changed, 141 insertions(+), 33 deletions(-)
> 
> -- 
> 2.18.0
> 

Queued, thanks.

Fam