ACK/Cmnt: [Unstable/Lunar][PATCH] UBUNTU: [Packaging] Remove update-version-dkms
Andrei Gherzan
andrei.gherzan at canonical.com
Fri Apr 21 11:15:45 UTC 2023
On 23/04/20 01:15PM, Juerg Haefliger wrote:
> It's a legacy script that doesn't do anthing since 2021 so finally
> remove it.
>
> Fixes: 5cc78bf956c7 ("UBUNTU: [Packaging] switch to kernel-versions")
> Signed-off-by: Juerg Haefliger <juerg.haefliger at canonical.com>
> ---
> update-version-dkms | 4 ----
> 1 file changed, 4 deletions(-)
> delete mode 100755 update-version-dkms
>
> diff --git a/update-version-dkms b/update-version-dkms
> deleted file mode 100755
> index a266fecc06f8..000000000000
> --- a/update-version-dkms
> +++ /dev/null
> @@ -1,4 +0,0 @@
> -#!/bin/bash
> -
> -echo "EE: run update-dkms-versions after link-to-tracker" 1>&2
> -exit 1
> --
> 2.37.2
This looks good to me but I'm thinking, shouldn't this be a cranky thing
as opposed to having a in tree script? Also, if we end up dropping it,
we will need to also update the cranky docs to avoid confusion where the
script is not found[1].
[1] https://canonical-kteam-docs.readthedocs-hosted.com/en/latest/docs/kteam_tools/cranking-the-kernel.html#dkms-package-update-stage-update-dkms-versions
Acked-by: Andrei Gherzan <andrei.gherzan at canonical.com>
--
Andrei Gherzan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20230421/d3ff7b58/attachment.sig>
More information about the kernel-team
mailing list