All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v5 00/16] buildman: Correct various issues with missing blobs
@ 2022-11-10  2:14 Simon Glass
  2022-11-10  2:14 ` [PATCH v5 01/16] image: Correct strncpy() warning with image_set_name() Simon Glass
                   ` (15 more replies)
  0 siblings, 16 replies; 45+ messages in thread
From: Simon Glass @ 2022-11-10  2:14 UTC (permalink / raw)
  To: U-Boot Mailing List
  Cc: Tom Rini, Rasmus Villemoes, Simon Glass, AKASHI Takahiro,
	Alper Nebi Yasak, Andre Przywara, Chia-Wei Wang,
	Heinrich Schuchardt, Jan Kiszka, Joe Hershberger, Joel Stanley,
	Marek Behún, Marek Vasut, Masahiro Yamada, Pali Rohár,
	Philippe Reynes, Quentin Schulz, Quentin Schulz, Rick Chen,
	Sean Anderson, Steven Lawrance

This series attempts to tidy up the problem of missing blobs, making
U-Boot default to failing the build, but providing ways to configure this
so that buildman can still be used for build testing by people who don't
have the blobs or don't care about them.

This is based on discussion in [1] as well as some experimentation.

It also moves the buildman docs to rST and tidies up some toolchain
settings and documentation.

[1] https://patchwork.ozlabs.org/project/uboot/patch/20221010151831.3376759-2-trini@konsulko.com/

Changes in v5:
- Add a note that -W requires -M
- Update ftest use of -W to avoid confusion
- Update exit-status docs to mention -W also
- Rename flag to --ignore-missing
- Update commit message
- Correct help for -M flag
- Correct --- Warning --- section
- Convert the buildman args to a list
- Add new patch with a separate section about environment variables
- Add new patch to ensure config_fname is inited
- Add new patch to reinstate removal of temp output dir in tests
- Split out config_fname change into a separate patch
- Rename GetGlobalValue to GetGlobalItemValue
- Fix 'items' typo
- Simplify implementation of GetGlobalItemValue()
- Use '[global]' for the doc heading
- Use double tick for arguments
- Drop redundant allow_missing code
- Split out the tests more
- Add new patch with documentation for the command line args

Changes in v4:
- Add a patch to update the arc toolchain
- Add new patch to update the default settings file
- Leave the arc toolchain in there
- Various changes that were unfortunately not recorded

Changes in v3:
- Add new patch to convert documentation to rST
- Add new patch to drop mention of MAKEALL
- Add new patch to drop mention of old architectures
- Add tests docs and a settings-file option

Changes in v2:
- Use a separate rule for running binman

Simon Glass (14):
  image: Correct strncpy() warning with image_set_name()
  Makefile: Correct the binman rule
  doc: Correct the path to the Makefile documentation
  binman: Use an exit code when blobs are missing
  buildman: Convert documentation to rST
  buildman: Drop mention of MAKEALL
  buildman: Update the arc toolchain
  buildman: Update the default settings file
  buildman: Drop mention of old architectures
  buildman: Detect binman reporting missing blobs
  binman: Add a separate section about environment variables
  buildman: Ensure config_fname is inited
  buildman: Reinstate removal of temp output dir in tests
  binman: Add documentation for the command line args

Tom Rini (2):
  global: Do not default to faking missing binaries for buildman
  buildman: Add --allow-missing flag to allow missing blobs

 .azure-pipelines.yml            |    2 +-
 .gitlab-ci.yml                  |    6 +-
 Makefile                        |   15 +-
 doc/build/buildman.rst          |    1 +
 doc/build/index.rst             |    1 +
 include/image.h                 |    8 +-
 scripts/Kbuild.include          |    2 +-
 tools/binman/binman.rst         |  352 +++++++-
 tools/binman/cmdline.py         |    5 +-
 tools/binman/control.py         |    9 +-
 tools/binman/ftest.py           |   19 +-
 tools/buildman/README           | 1349 -------------------------------
 tools/buildman/README.rst       |    1 +
 tools/buildman/bsettings.py     |   17 +-
 tools/buildman/builder.py       |    5 +-
 tools/buildman/builderthread.py |    8 +-
 tools/buildman/buildman.rst     | 1328 ++++++++++++++++++++++++++++++
 tools/buildman/cmdline.py       |    6 +
 tools/buildman/control.py       |   28 +-
 tools/buildman/func_test.py     |  112 ++-
 20 files changed, 1893 insertions(+), 1381 deletions(-)
 create mode 120000 doc/build/buildman.rst
 delete mode 100644 tools/buildman/README
 create mode 120000 tools/buildman/README.rst
 create mode 100644 tools/buildman/buildman.rst

-- 
2.38.1.431.g37b22c650d-goog


^ permalink raw reply	[flat|nested] 45+ messages in thread

end of thread, other threads:[~2022-12-05 23:57 UTC | newest]

Thread overview: 45+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-11-10  2:14 [PATCH v5 00/16] buildman: Correct various issues with missing blobs Simon Glass
2022-11-10  2:14 ` [PATCH v5 01/16] image: Correct strncpy() warning with image_set_name() Simon Glass
2022-11-23  2:12   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 02/16] Makefile: Correct the binman rule Simon Glass
2022-11-10  8:01   ` Pali Rohár
2022-11-10 20:40     ` Simon Glass
2022-11-23  2:11       ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 03/16] doc: Correct the path to the Makefile documentation Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 04/16] binman: Use an exit code when blobs are missing Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 05/16] buildman: Convert documentation to rST Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 06/16] buildman: Drop mention of MAKEALL Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 07/16] buildman: Update the arc toolchain Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 08/16] buildman: Update the default settings file Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 09/16] buildman: Drop mention of old architectures Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 10/16] buildman: Detect binman reporting missing blobs Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-12-05 23:13   ` Peter Robinson
2022-12-05 23:23     ` Tom Rini
2022-12-05 23:29       ` Peter Robinson
2022-12-05 23:34         ` Tom Rini
2022-12-05 23:43           ` Peter Robinson
2022-12-05 23:46             ` Tom Rini
2022-12-05 23:49               ` Peter Robinson
2022-12-05 23:55               ` Simon Glass
2022-12-05 23:56                 ` Tom Rini
2022-12-05 23:57                   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 11/16] binman: Add a separate section about environment variables Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 12/16] global: Do not default to faking missing binaries for buildman Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 13/16] buildman: Ensure config_fname is inited Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 14/16] buildman: Reinstate removal of temp output dir in tests Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 15/16] buildman: Add --allow-missing flag to allow missing blobs Simon Glass
2022-11-23  2:11   ` Simon Glass
2022-11-10  2:14 ` [PATCH v5 16/16] binman: Add documentation for the command line args Simon Glass
2022-11-23  2:11   ` Simon Glass

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.