[PATCH v4 0/2] automatically strip btool_ from btool names

Quentin Schulz foss+uboot at 0leil.net
Mon Nov 7 13:54:52 CET 2022


Commit daa2da754afe ("binman: btool: gzip: fix packer name so that binary can be
found") introduced some sort of work-around to make the gzip binary found by
bintool. However, this work-around would need to be applied to any btool whose
name is prefixed with btool_ (usually to avoid conflict with other python
modules).

Instead, let's handle this specific case directly inside bintool so that the
btools don't have to handle it themselves. This also reverts the now-unnecessary
work-around for gzip btool.

To: Simon Glass <sjg at chromium.org>
To: Alper Nebi Yasak <alpernebiyasak at gmail.com>
Cc: Quentin Schulz <foss+uboot at 0leil.net>
Cc: u-boot at lists.denx.de
Signed-off-by: Quentin Schulz <quentin.schulz at theobroma-systems.com>

---
Changes in v4:
- fix buildman test by enforcing class name to be Bintool<packer> even for
  Bintools in files whose filename is prefixed with btool_,
- Link to v3: https://lore.kernel.org/r/20220930-upstream-gzip-binman-v3-v3-0-17c99d6d87ac@theobroma-systems.com

---
Quentin Schulz (2):
      binman: bintool: remove btool_ prefix from btool names
      Revert "binman: btool: gzip: fix packer name so that binary can be found"

 tools/binman/bintool.py          | 3 ++-
 tools/binman/btool/btool_gzip.py | 4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)
---
base-commit: 898bd53e6a930080cee7cd7b1a09120c4dfd9467
change-id: 20220930-upstream-gzip-binman-v3-91b83bf16795

Best regards,
-- 
Quentin Schulz <quentin.schulz at theobroma-systems.com>


More information about the U-Boot mailing list