Re: [PATCH] kbuild: deb-pkg: Explicitly define optional user-defined variables
From: Nicolas Schier
Date: Sun Jun 16 2024 - 14:47:17 EST
On Sun, Jun 16, 2024 at 06:37:16PM +0200 Nicolas Schier wrote:
> mkdebian supports some optional environment variables for controlling
> the Debian package generation. Declare those variables explicitly to
> allow enabling of 'set -u' (error on unset variables).
>
> Use of ': "${VAR:=}"' for variable assignment is POSIX compatible [1].
>
> [1]: https://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#colon
>
> Signed-off-by: Nicolas Schier <nicolas@xxxxxxxxx>
> ---
> scripts/package/mkdebian | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/scripts/package/mkdebian b/scripts/package/mkdebian
> index 070149c985fe..81e81906ad07 100755
> --- a/scripts/package/mkdebian
> +++ b/scripts/package/mkdebian
> @@ -6,6 +6,18 @@
>
> set -e
>
> +# Optional user-specified environment variables
> +
> +# Set target Debian architecture (skip auto-detection)
> +: "${KBUILD_DEBARCH:=}"
> +
> +# Set target Debian distribution (skipping auto-detection)
> +: "${KDEB_CHANGELOG_DIST:=}"
> +
> +# Overwrite the automatically determined package version.
> +: "${KDEB_PKGVERSION:=}"
> +
> +
> is_enabled() {
> grep -q "^$1=y" include/config/auto.conf
> }
> --
> 2.39.2
Please ignore, cp. https://lore.kernel.org/linux-kbuild/CAK7LNARPP_OsaOhUHT+5-vi5XJ36hDs0bNsiM7pwuAmj4S+MNg@xxxxxxxxxxxxxx/
Sorry for the noise.
Kind regards,
Nicolas