gnuboot-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH v1 00/22] Fix release tests with debug images + website patches


From: Denis 'GNUtoo' Carikli
Subject: [PATCH v1 00/22] Fix release tests with debug images + website patches
Date: Fri, 20 Dec 2024 22:23:55 +0100

Hi,

This contains fixes for the test that fails due to the debug images
now being produced (as explained in the commit message we knew about
this issue at the time but decided not to fix yet for good reasons
that are explained in the commit message):

  release: tests: add debug images to the list of expected files.
  release: tests: whitespace fixes.
  release: tests: make it passs shellcheck.

These have been tested (I've launched a build with an older version of
this patch set (without many of the website improvements)), and it
worked fine:
    [ OK ] 
release/i945-thinkpads-install/i945-thinkpads-install-utilities-deb-pack.deb
    [ OK ] release/i945-thinkpads-install/packages_src.tar
    [ OK ] release/i945-thinkpads-install/gnuboot_src.tar
    [ OK ] 
release/i945-thinkpads-install/i945-thinkpads-install-utilities-tarball-pack.tar.gz
    [ OK ] Makefile: release target. See make-1734701870.log for the log.
    $ echo $?
    0

This patch set is to be applied on top of the previously not-reviewed
yet patch set that was sent yesterday ("[PATCH v1 0/7] Fix for the
LVM2 test + website improvements."). Several things motivated me to
send a new patch set so close to the previous one:

* The old patch set was tested with the newer commits from this patch
  set.

* I've discussed with Adrien in person and he prefers me to send it
  now because he will have a big chunk of time to do a lot of reviews
  soon in his schedule.

* Since isf also sent translations, and that I promised to include
  some of them in my next patch set, and that they were sent to the
  mailing list (as they should), it also enable to review them in
  patch form. This is also easier to review than files in zip files so
  it should also save time for everybody and/or enable more people to
  review the patches from their mail client.

* Finally this patch set also contains patches from neox that were to
  be resubmitted, so it also simplifies things and could also at the
  end save some time to neox.

In addition to the fixes, I've also added lot of patches for adding
tests in the status and for cleaning a bit the tables. All these have
been tested with the icecat browser from Guix. However for some
reasons it required to remove the "untitled" directory in the website
to make the global.css change taken into account.

I've also finally finished a French translation of the main
page. While I am a French native speaker, I tend to do a lot of
spelling mistakes (way more than in English), so it might be good to
double check if I didn't do any. Also note that I am also not very
familiar with computer terms in French (I need to catch up with the
translations their usage that are slowly integrating French), but
there any complicated computer terms in this translation.

Also neox sent patches from a previous serie and some were also added
in this serie. Here what was sent before:

> Adrien 'neox' Bourmault (6):
>   website: force-bordeaux-substitutes: fix e-mail address in copyright
>     notice
>   website: global.css: fix titles top and down margins
>   website: css/template: add the GNU Boot logo
>   website: status: add a RC4 section
>   website: status: mark Asus D16, Lenovo R400 and T500 as tested
>   website: status: mark Vikings D16 as tested

The first patch ("website: force-bordeaux-substitutes: [...]") was
already merged. The second one (global.css) is included. As for the
third one (GNU Boot logo) it still requires rework so I didn't include
it.

The one about adding an RC4 section is now obsolete, and the two last
ones were rebased and included.

This also makes the "GNU Boot logo" patch the only patch that still
needs to be reworked.

Since isf also sent multiple translations, I made a patch out of the
ones that could be integrated and they are included in this serie as
well. The last one could not be integrated yet in this patch set
because while the translation might be fine (I didn't review it but
the 2 previous only needed extremely small adjustments), it also
depended on another page that was not translated yet and that isf
planned to translate later on.

So even if we start reviewing the last translation, we cannot merge it
yet until we also get a translation for the page that it depends
on. My reply on the mailing list about this translation has more
details.

Adrien 'neox' Bourmault (3):
  website: global.css: fix titles top and down margins
  website: status: RC4: mark Asus D16, Lenovo R400 and T500 as tested
  website: status: 0.1 RC4: mark Vikings D16 as tested

Denis 'GNUtoo' Carikli (17):
  website: status: KCMA-D8: mention missing high resolution graphics
    images.
  website: status: KFSN4-DRE: mention missing high resolution graphics
    images.
  website: status: KGPE-D16: mention missing high resolution graphics
    images.
  website: status: D510-MO: mention missing high resolution graphics
    images.
  website: status: D945GCLF/D945GCLF2D: mention missing high resolution
    graphics images.
  website: status: G43T-AM3: mention missing high resolution graphics
    images.
  website: status: GA-G41M-ES2L: mention missing high resolution
    graphics images.
  website: status: D945GCLF/D945GCLF2D: mention missing GRUB images.
  website: status: KFSN4-DRE: mention missing GRUB images.
  release: tests: add debug images to the list of expected files.
  release: tests: whitespace fixes.
  release: tests: make it pass shellcheck.
  website: status: KGPE-D16: split status in ECC/non-ECC.
  website: status: KCMA-D8: split status in ECC/non-ECC.
  website: status: RC4/RC5: cosmetics: align cell text top-left.
  website: status: 0.1 RC5: set KGPE-D16 with GRUB and ECC RAM as
    tested.
  website: index: add French translation.

Jordán (isf) (2):
  website: docs: index: add Spanish translation.
  website: docs: install: index: add Spanish translation.

 resources/packages/release/test        |  18 +-
 tests/lint                             |   1 +
 website/pages/docs/index.es.md         |  52 ++++
 website/pages/docs/install/index.es.md | 121 +++++++++
 website/pages/global.css               |   5 +
 website/pages/index.fr.md              |  64 +++++
 website/pages/status.es.md             | 113 ++++++---
 website/pages/status.md                | 324 +++++++++++++++----------
 8 files changed, 532 insertions(+), 166 deletions(-)
 create mode 100644 website/pages/docs/index.es.md
 create mode 100644 website/pages/docs/install/index.es.md
 create mode 100644 website/pages/index.fr.md


base-commit: 0fdbfb3377dd75ef5aecc3d850397b467835ec97
-- 
2.46.0




reply via email to

[Prev in Thread] Current Thread [Next in Thread]