~postmarketos/pmbootstrap

41daa850d731f563bf320702b2d719b743416139 — Luca Weiss 11 months ago 5dcd846
parse.arguments: Allow providing multiple packages for kconfig_check

It can be quite useful to check the configs for a list of packages, so
adjust the argument parsing code to support that.

Reviewed-by: Oliver Smith <ollieparanoid@postmarketos.org>
Tested-by: Oliver Smith <ollieparanoid@postmarketos.org>
Reviewed-by: Clayton Craft <clayton@craftyguy.net>
Link: https://lists.sr.ht/~postmarketos/pmbootstrap-devel/%3C20231028085901.35205-1-luca@z3ntu.xyz%3E
2 files changed, 3 insertions(+), 5 deletions(-)

M pmb/helpers/frontend.py
M pmb/parse/arguments.py
M pmb/helpers/frontend.py => pmb/helpers/frontend.py +2 -4
@@ 393,10 393,8 @@ def kconfig(args):
            raise RuntimeError("kconfig check failed!")

        # Default to all kernel packages
        packages = []
        if args.package:
            packages = [args.package]
        else:
        packages = args.package
        if not args.package:
            for aport in pmb.helpers.pmaports.get_list(args):
                if aport.startswith("linux-"):
                    packages.append(aport.split("linux-")[1])

M pmb/parse/arguments.py => pmb/parse/arguments.py +1 -1
@@ 476,7 476,7 @@ def arguments_kconfig(subparser):
        check.add_argument(f"--{name}", action="store_true",
                           dest=f"kconfig_check_{name}",
                           help=f"check options needed for {name} too")
    add_kernel_arg(check)
    add_kernel_arg(check, nargs="*")

    # "pmbootstrap kconfig edit"
    edit = sub.add_parser("edit", help="edit kernel aport config")