Quoting Julian Gilbey (2022-09-19 16:55:11) > > That is correct. But letting sbuild run autopkgtest is also a bad design > > choice. Since the $autopkgtest_opts are a list of options, it would also be > > bad to let the command line options just replace what was already set. So > > then you need some sort of interface that lets you add or remove arbitrary > > stuff. This can become very complicated very fast. We can also not change > > this now because sbuild is also heavily used from scripts that now rely on > > the existing functionality. > > Another good point (about the bad design). I wasn't suggesting changing the > existing behaviour for exactly the reason you indicate, rather I was > suggesting a new option to mean "clear all existing $autopkgtest_opts and > start with an empty list". But that also feels a bit clunky.
Yes, that could be done. But before I add another option to the already endless
list:
sbuild [-h|--help | -V|--version] [-v|--verbose | -q|--quiet] [-D|--de‐
bug] [-A|--arch-all] [--archive=archive] [-d|--dist=distribution]
[-c|--chroot=chroot] [--chroot-mode=schroot|sudo|autopkgtest|unshare]
[--arch=architecture] [--arch-any | --no-arch-any] [--build=architec‐
ture] [--host=architecture] [--profiles=profile[,...]] [-s|--source]
[--force-orig-source] [--make-binNMU=changelog-entry] [--binNMU=NMU-
version] [--append-to-version=string] [--binNMU-timestamp=timestamp]
[--binNMU-changelog=changelog] [--build-dir=directory] [--add-de‐
pends=dependency] [--add-conflicts=dependency] [--add-depends-arch=de‐
pendency] [--add-conflicts-arch=dependency] [--add-depends-indep=depen‐
dency] [--add-conflicts-indep=dependency] [-m|--maintainer=maintainer]
[-e|--uploader=uploader] [-k|--keyid=key-id] [--source-only-changes]
[--no-source-only-changes] [-j|--jobs=n] [--debbuildopt=option] [--deb‐
buildopts=options] [--dpkg-source-opt=options] [--dpkg-source-opts=op‐
tions] [--dpkg-file-suffix=suffix] [-p|--purge=purge-mode]
[--purge-build=purge-mode] [--purge-deps=purge-mode] [--purge-ses‐
sion=purge-mode] [-b|--batch] [-n|--nolog] [--clean-source]
[--no-clean-source] [--run-lintian] [--no-run-lintian] [--lin‐
tian-opt=options] [--lintian-opts=options] [--run-piuparts]
[--no-run-piuparts] [--piuparts-opt=options] [--piuparts-opts=options]
[--piuparts-root-arg=options] [--piuparts-root-args=options] [--run-au‐
topkgtest] [--no-run-autopkgtest] [--autopkgtest-opt=options] [--autop‐
kgtest-opts=options] [--autopkgtest-root-arg=options] [--autop‐
kgtest-root-args=options] [--pre-build-commands=string] [--ch‐
root-setup-commands=string] [--chroot-update-failed-commands=string]
[--build-deps-failed-commands=string] [--starting-build-com‐
mands=string] [--finished-build-commands=string] [--build-failed-com‐
mands=string] [--chroot-cleanup-commands=string] [--post-build-com‐
mands=string] [--post-build-failed-commands=string] [--any‐
thing-failed-commands=string] [--log-external-command-output]
[--log-external-command-error] [--setup-hook=hook-script]
[--build-dep-resolver=resolver] [--resolve-alternatives|--no-re‐
solve-alternatives] [--extra-package=package.deb] [--extra-reposi‐
tory=spec] [--extra-repository-key=file.asc] [--build-path=string]
[--autopkgtest-virt-server=schroot|lxc|chroot|qemu|ssh] [--autop‐
kgtest-virt-server-opt=string] [--autopkgtest-virt-server-opts=options]
[--purge-extra-packages] [--bd-uninstallable-explainer=dose3|apt|none]
[PACKAGE[.dsc]]
Before I do that, I would need a concrete use-case justifying it. If the %r
percent escape works for you, then I guess that's it.
Please close this bug if you think that there is nothing else to discuss.
Thanks!
cheers, josch
signature.asc
Description: signature
