docker: Remove LegacyKeyValueFormat warnings

[PATCH 0/7] docker: Remove LegacyKeyValueFormat warnings

Philippe Mathieu-Daudé posted 7 patches 1 week, 5 days ago

Download series mbox

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

Maintainers: Peter Maydell <peter.maydell@linaro.org>, "Alex Bennée" <alex.bennee@linaro.org>, Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>, Brian Cain <brian.cain@oss.qualcomm.com>, Song Gao <gaosong@loongson.cn>, Max Filippov <jcmvbkbc@gmail.com>, John Snow <jsnow@redhat.com>, Cleber Rosa <crosa@redhat.com>

MAINTAINERS                                        |  6 +++++-
tests/lcitool/refresh                              | 12 +++++++-----
tests/docker/dockerfiles/alpine.docker             |  2 +-
tests/docker/dockerfiles/centos9.docker            |  2 +-
.../dockerfiles/debian-all-test-cross.docker       | 10 ++++++----
tests/docker/dockerfiles/debian-amd64-cross.docker |  2 +-
tests/docker/dockerfiles/debian-arm64-cross.docker |  2 +-
tests/docker/dockerfiles/debian-armhf-cross.docker |  2 +-
.../docker/dockerfiles/debian-hexagon-cross.docker | 14 +++++++-------
tests/docker/dockerfiles/debian-i686-cross.docker  |  2 +-
.../dockerfiles/debian-loongarch-cross.docker      | 10 +++++-----
.../dockerfiles/debian-mips64el-cross.docker       |  2 +-
.../docker/dockerfiles/debian-mipsel-cross.docker  |  2 +-
.../docker/dockerfiles/debian-ppc64el-cross.docker |  2 +-
.../docker/dockerfiles/debian-riscv64-cross.docker |  2 +-
tests/docker/dockerfiles/debian-s390x-cross.docker |  2 +-
.../docker/dockerfiles/debian-tricore-cross.docker |  6 +++---
.../docker/dockerfiles/debian-xtensa-cross.docker  |  8 ++++----
tests/docker/dockerfiles/debian.docker             |  2 +-
.../docker/dockerfiles/fedora-rust-nightly.docker  |  2 +-
tests/docker/dockerfiles/fedora-win64-cross.docker |  2 +-
tests/docker/dockerfiles/fedora.docker             |  2 +-
tests/docker/dockerfiles/opensuse-leap.docker      |  2 +-
tests/docker/dockerfiles/python.docker             |  5 +++--
scripts/coverity-scan/coverity-scan.docker         | 12 ++++++------
scripts/git.orderfile                              |  4 ++++
26 files changed, 66 insertions(+), 53 deletions(-)

Expand all Fold all

[PATCH 0/7] docker: Remove LegacyKeyValueFormat warnings

Posted by Philippe Mathieu-Daudé 1 week, 5 days ago

Update MAINTAINERS entries for manually maintained Dockerfiles.

Remove warnings such:

 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 98)
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 64)
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 97)

See https://docs.docker.com/reference/build-checks/legacy-key-value-format/

Philippe Mathieu-Daudé (7):
  MAINTAINERS: Fix docker/dockerfiles/debian-hexagon-cross.docker path
  MAINTAINERS: Cover debian-loongarch-cross.docker with LoongArch
    section
  MAINTAINERS: Cover debian-xtensa-cross.docker with Xtensa section
  MAINTAINERS: Cover debian-tricore-cross.docker with TriCore section
  MAINTAINERS: Cover python.docker with Python library section
  docker: Remove LegacyKeyValueFormat warnings in non-generated files
  docker: Remove LegacyKeyValueFormat warnings in generated files

 MAINTAINERS                                        |  6 +++++-
 tests/lcitool/refresh                              | 12 +++++++-----
 tests/docker/dockerfiles/alpine.docker             |  2 +-
 tests/docker/dockerfiles/centos9.docker            |  2 +-
 .../dockerfiles/debian-all-test-cross.docker       | 10 ++++++----
 tests/docker/dockerfiles/debian-amd64-cross.docker |  2 +-
 tests/docker/dockerfiles/debian-arm64-cross.docker |  2 +-
 tests/docker/dockerfiles/debian-armhf-cross.docker |  2 +-
 .../docker/dockerfiles/debian-hexagon-cross.docker | 14 +++++++-------
 tests/docker/dockerfiles/debian-i686-cross.docker  |  2 +-
 .../dockerfiles/debian-loongarch-cross.docker      | 10 +++++-----
 .../dockerfiles/debian-mips64el-cross.docker       |  2 +-
 .../docker/dockerfiles/debian-mipsel-cross.docker  |  2 +-
 .../docker/dockerfiles/debian-ppc64el-cross.docker |  2 +-
 .../docker/dockerfiles/debian-riscv64-cross.docker |  2 +-
 tests/docker/dockerfiles/debian-s390x-cross.docker |  2 +-
 .../docker/dockerfiles/debian-tricore-cross.docker |  6 +++---
 .../docker/dockerfiles/debian-xtensa-cross.docker  |  8 ++++----
 tests/docker/dockerfiles/debian.docker             |  2 +-
 .../docker/dockerfiles/fedora-rust-nightly.docker  |  2 +-
 tests/docker/dockerfiles/fedora-win64-cross.docker |  2 +-
 tests/docker/dockerfiles/fedora.docker             |  2 +-
 tests/docker/dockerfiles/opensuse-leap.docker      |  2 +-
 tests/docker/dockerfiles/python.docker             |  5 +++--
 scripts/coverity-scan/coverity-scan.docker         | 12 ++++++------
 scripts/git.orderfile                              |  4 ++++
 26 files changed, 66 insertions(+), 53 deletions(-)

-- 
2.53.0

Re: [PATCH 0/7] docker: Remove LegacyKeyValueFormat warnings

Posted by Alex Bennée 1 week, 4 days ago

Philippe Mathieu-Daudé <philmd@linaro.org> writes:

> Update MAINTAINERS entries for manually maintained Dockerfiles.
>
> Remove warnings such:
>
>  - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 98)
>  - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 64)
>  - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 97)
>
> See
> https://docs.docker.com/reference/build-checks/legacy-key-value-format/

Queued to testing/next, thanks.

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro