Hi Thomas,

On 1/8/25 17:12, Thomas Huth wrote:
From: Thomas Huth <th...@redhat.com>

The tests/functional folder has become quite crowded, thus move the
mips tests into a target-specific subfolder.

Signed-off-by: Thomas Huth <th...@redhat.com>
---
  MAINTAINERS                                   |  8 ++--
  tests/functional/meson.build                  | 47 ++-----------------
  tests/functional/mips/meson.build             | 11 +++++
  .../test_malta.py}                            |  0
  .../test_replay.py}                           |  0
  .../test_tuxrun.py}                           |  0
  tests/functional/mips64/meson.build           | 10 ++++
  .../test_malta.py}                            |  2 +-
  .../test_tuxrun.py}                           |  0
  tests/functional/mips64el/meson.build         | 14 ++++++
  .../test_fuloong2e.py}                        |  0
  .../test_loongson3v.py}                       |  0
  .../test_malta.py}                            |  4 +-
  .../test_replay.py}                           |  0
  .../test_tuxrun.py}                           |  0
  tests/functional/mipsel/meson.build           | 12 +++++
  .../test_malta.py}                            |  2 +-
  .../test_replay.py}                           |  0
  .../test_tuxrun.py}                           |  0
  19 files changed, 59 insertions(+), 51 deletions(-)
  create mode 100644 tests/functional/mips/meson.build
  rename tests/functional/{test_mips_malta.py => mips/test_malta.py} (100%)
  rename tests/functional/{test_mips_replay.py => mips/test_replay.py} (100%)
  rename tests/functional/{test_mips_tuxrun.py => mips/test_tuxrun.py} (100%)
  create mode 100644 tests/functional/mips64/meson.build
  rename tests/functional/{test_mips64_malta.py => mips64/test_malta.py} (96%)
  rename tests/functional/{test_mips64_tuxrun.py => mips64/test_tuxrun.py} 
(100%)
  create mode 100644 tests/functional/mips64el/meson.build
  rename tests/functional/{test_mips64el_fuloong2e.py => 
mips64el/test_fuloong2e.py} (100%)
  rename tests/functional/{test_mips64el_loongson3v.py => 
mips64el/test_loongson3v.py} (100%)
  rename tests/functional/{test_mips64el_malta.py => mips64el/test_malta.py} 
(98%)
  rename tests/functional/{test_mips64el_replay.py => mips64el/test_replay.py} 
(100%)
  rename tests/functional/{test_mips64el_tuxrun.py => mips64el/test_tuxrun.py} 
(100%)
  create mode 100644 tests/functional/mipsel/meson.build
  rename tests/functional/{test_mipsel_malta.py => mipsel/test_malta.py} (98%)
  rename tests/functional/{test_mipsel_replay.py => mipsel/test_replay.py} 
(100%)
  mode change 100644 => 100755
  rename tests/functional/{test_mipsel_tuxrun.py => mipsel/test_tuxrun.py} 
(100%)


+subdir('mips')
+subdir('mipsel')
+subdir('mips64')
+subdir('mips64el')

Just one single comment here. I'd rather have a single mips/ directory
and keep the variants inside it, i.e.:

  tests/functional/mips/test_mips64el_malta.py

Anyhow:

Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>


Reply via email to