[pve-devel] applied: [RFC ifupdown2] d/maintscripts: use postinst arguments to determine first install

[Thomas Lamprecht]

Am 13.11.24 um 20:03 schrieb Stoiko Ivanov:
> Following https://www.debian.org/doc/debian-policy/ap-flowcharts.html
> postinst gets called as 'postinst configure' w/o second argument on
> first installation, use that information instead of creating a
> flag-file in preinst.
> 
> Technically this changes the semantics and the first-install parts
> will not be run, in case ifupdown2 had been installed on the system,
> but was subsequently removed, but not purged ('rc' in dpkg-output).
> Since the functionality was added quite a while ago (released with PVE
> 6.2 to support ovs-setups with ifupdown2 - the potential for
> regression should be limited. Originally introduced in:
> 849ae55de6454ea4631de0899182c148c954e46a

FWIW, I got a git-alias named show-short that resolves to

show --no-patch --pretty='format: %C(auto)%h ("%s")'

which I use for such references. The main benefit is having the subject
of a commit, which gives some more context without having to check the
history, or allows quickly searching on other sources (e.g., lore for
kernel or our stuff) if the commit would reference another repo.

I do not have all to hard feelings here and am *very* happy about any
Fixes tag, whatever form it is, compared to none at all, just mentioning
it, maybe you see some similar benefits in a similar format.

> 
> Fixes: 8d5303c35044b612afc68eec1dc5bc265a5dd328
> Signed-off-by: Stoiko Ivanov <s.ivanov at proxmox.com>
> ---
> sending as RFC, as I'm not sure that the small cosmetic fix is worth
> the potetial regressions at all, but after spending a bit too long
> reading the commit-history and the debian-policy manual thought I'll
> send it anyways.
> (Tested with mmebstrap and in an install on top of debian)
> 
>  debian/ifupdown2.postinst |  3 +--
>  debian/ifupdown2.preinst  | 20 --------------------
>  2 files changed, 1 insertion(+), 22 deletions(-)
>  delete mode 100644 debian/ifupdown2.preinst
> 
>

applied, thanks!