linux-kernel - Re: [PATCH 1/2] kbuild: pacman-pkg: move common commands to a separate function

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <7d0ec58b-4704-4779-82ab-94ecdcbcd448@weissschuh.net>
Date: Fri, 16 Aug 2024 20:59:10 +0200 (GMT+02:00)
From: linux@...ssschuh.net
To: Masahiro Yamada <masahiroy@...nel.org>
Cc: linux-kbuild@...r.kernel.org, linux-kernel@...r.kernel.org,
	Christian Heusel <christian@...sel.eu>,
	Nathan Chancellor <nathan@...nel.org>,
	Nicolas Schier <nicolas@...sle.eu>,
	Thomas Weißschuh <linux@...ssschuh.net>
Subject: Re: [PATCH 1/2] kbuild: pacman-pkg: move common commands to a
 separate function

Aug 16, 2024 16:18:53 Masahiro Yamada <masahiroy@...nel.org>:

> All build and package functions share the following commands:
>
>   export MAKEFLAGS="${KBUILD_MAKEFLAGS}"
>   cd "${objtree}"
>
> Factor out the common code.
>
> Signed-off-by: Masahiro Yamada <masahiroy@...nel.org>

Acked-by:  Thomas Weißschuh <linux@...ssschuh.net>

> ---
>
> scripts/package/PKGBUILD | 17 ++++++++++-------
> 1 file changed, 10 insertions(+), 7 deletions(-)
>
> diff --git a/scripts/package/PKGBUILD b/scripts/package/PKGBUILD
> index fbd7eb10a52c..e2d9c2601ca9 100644
> --- a/scripts/package/PKGBUILD
> +++ b/scripts/package/PKGBUILD
> @@ -36,11 +36,15 @@ makedepends=(
> )
> options=(!debug !strip !buildflags !makeflags)
>
> -build() {
> +_prologue() {
>     # MAKEFLAGS from makepkg.conf override the ones inherited from kbuild.
>     # Bypass this override with a custom variable.
>     export MAKEFLAGS="${KBUILD_MAKEFLAGS}"
>     cd "${objtree}"
> +}
> +
> +build() {
> +   _prologue
>
>     ${MAKE} KERNELRELEASE="${KERNELRELEASE}" KBUILD_BUILD_VERSION="${pkgrel}"
> }
> @@ -48,10 +52,10 @@ build() {
> _package() {
>     pkgdesc="The ${pkgdesc} kernel and modules"
>
> -   export MAKEFLAGS="${KBUILD_MAKEFLAGS}"
> -   cd "${objtree}"
>     local modulesdir="${pkgdir}/usr/${MODLIB}"
>
> +   _prologue
> +
>     echo "Installing boot image..."
>     # systemd expects to find the kernel here to allow hibernation
>     # https://github.com/systemd/systemd/commit/edda44605f06a41fb86b7ab8128dcf99161d2344
> @@ -76,10 +80,10 @@ _package() {
> _package-headers() {
>     pkgdesc="Headers and scripts for building modules for the ${pkgdesc} kernel"
>
> -   export MAKEFLAGS="${KBUILD_MAKEFLAGS}"
> -   cd "${objtree}"
>     local builddir="${pkgdir}/usr/${MODLIB}/build"
>
> +   _prologue
> +
>     if grep -q CONFIG_MODULES=y include/config/auto.conf; then
>         echo "Installing build files..."
>         "${srctree}/scripts/package/install-extmod-build" "${builddir}"
> @@ -100,8 +104,7 @@ _package-api-headers() {
>     provides=(linux-api-headers)
>     conflicts=(linux-api-headers)
>
> -   export MAKEFLAGS="${KBUILD_MAKEFLAGS}"
> -   cd "${objtree}"
> +   _prologue
>
>     ${MAKE} headers_install INSTALL_HDR_PATH="${pkgdir}/usr"
> }
> --
> 2.43.0