From patchwork Sat Jul 29 14:38:13 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Masahiro Yamada X-Patchwork-Id: 128149 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:918b:0:b0:3e4:2afc:c1 with SMTP id s11csp1072793vqg; Sat, 29 Jul 2023 07:57:56 -0700 (PDT) X-Google-Smtp-Source: APBJJlFANFtjmAwaChHETK4Spit80oCIf3q8yUGqAJKfNEqsbsuVUhBmFCLs3K6kj1uzZmm7SR+C X-Received: by 2002:a17:90a:be10:b0:268:169e:8497 with SMTP id a16-20020a17090abe1000b00268169e8497mr6067665pjs.13.1690642675269; Sat, 29 Jul 2023 07:57:55 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1690642675; cv=none; d=google.com; s=arc-20160816; b=c95wCNCOlYs2SAeup2DDSKPqCL2Chiynp6kFwYfzIpUq2MZ83k8vsNsiA8/qotB9Qf qIxiAbgM2NFtxgC1ZoN0TQ2n2XtayZXHyCcnIKtGoJh/AOuUPmPfY0qZSvPM0DZ39arM yuABcZP5z6iroXFc+FqOVdC73iQ3DzmXX786tAXySKBfocgqkx+GJzYOf8tjNlQWhwj/ TWbAgfbNQ6bDh+kMvU8jkBrt8w4BRWvtFUsEcZh2dPgLE0oM1ZUeDntpwBLlPfNO3WA/ 6sy+G1HOVu734fBTzAOeIWzQ+GzrYEVjUqspbO5whLEvcR3ATdIX0YAX1gwcZH5894QB aXIQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=9q0L8r5uPOIxzq+jYXctBicxflOE3K3IpsEyR/kuNFM=; fh=eslil6/SEaAa2rUAmnzCWClLUU/9b0t1QC9lKZrgFu4=; b=Th+VjdTfP4EChES1FHcGK/r9bVQNoTG5+unKkoUH4i9KAKv4LPfvZMbW90qTzYPVY5 Zlzopa4Em1mfA1j6q5+a87o6KkgrKef/qyGYh879q3nEvYHEnWjoY6i9gH71F/AKK/xq +nElMfCDX5A6Eeu0nlsIVZrTa3LkaDHY10kmY3IB87dvaiOuo4rZGRyQ3R6JQcgVMPbz XT1WGx4FVhB4zNBRT4yUOVDG4lUamBmswB8toSu7t7RauHtPxAzi2ljQnt99QBlW+0dJ Pl3CYp6kSZKejkpzboAN5uOsLUu/dzqYKwMYl0+HHJz9rYiMNIh/Rq2bBax3lRc/NFv0 qBdA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=r4R8iXlr; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id c8-20020a17090a674800b00268097354bfsi6372770pjm.0.2023.07.29.07.57.41; Sat, 29 Jul 2023 07:57:55 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=r4R8iXlr; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231370AbjG2OiX (ORCPT + 99 others); Sat, 29 Jul 2023 10:38:23 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60514 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229510AbjG2OiW (ORCPT ); Sat, 29 Jul 2023 10:38:22 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2CA05A8; Sat, 29 Jul 2023 07:38:21 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id B097860C68; Sat, 29 Jul 2023 14:38:20 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A4A30C433C9; Sat, 29 Jul 2023 14:38:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1690641500; bh=vtWsqKGXBtO/1q4tL+rXkzYaevQSYzf89qO8jTqbrC4=; h=From:To:Cc:Subject:Date:From; b=r4R8iXlrXje1MsaRytB+OZuXp3YStlS5JXZp3VQmqNDR3riil7B8ZX7IhyCGxvZZP acAjWZ98luBVkiCyV/0hehf5vnTjxu191BN2KLTOuyDWbPK/XY+Wte2Kjm8IsEhM7+ 7D45b/Q+DKvyeSBmzg8RZmi3VjSErcGCiPqiB+avzeBZramPivgoOcSm5FSdEfFbrC ZU/xMAxY0D62OWD+lTixW19H1vhMW1FqfrqWzZbqo7iSg5MQu+BPLeNgAgf/cL86SF P/ZISwPIf3dFrrKjtOoJlBdPv3a1u8AtzwyQy8FSIeKvYaJJvmYhFxAtcEPTpmEbz5 2QQn37XoitM2w== From: Masahiro Yamada To: linux-kbuild@vger.kernel.org Cc: linux-kernel@vger.kernel.org, Ben Hutchings , Masahiro Yamada , Nathan Chancellor , Nick Desaulniers , Nicolas Schier Subject: [PATCH 1/2] kbuild: deb-pkg: use Debian compliant shebang for debian/rules Date: Sat, 29 Jul 2023 23:38:13 +0900 Message-Id: <20230729143814.1509196-1-masahiroy@kernel.org> X-Mailer: git-send-email 2.39.2 MIME-Version: 1.0 X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1772767334154413702 X-GMAIL-MSGID: 1772767334154413702 Debian Policy "4.9. Main building script: debian/rules" requires "debian/rules must start with the line #!/usr/bin/make -f". [1] Currently, Kbuild does not follow this policy. When Kbuild generates debian/rules, "#!$(command -v $MAKE) -f" is expanded by shell. The result may not be "#!/usr/bin/make -f". There was a reason to opt out the Debian policy. If you run '/path/to/my/custom/make deb-pkg', debian/rules must also be invoked by the same Make program. If #!/usr/bin/make were hard-coded in debian/rules, the sub-make would be executed by a possibly different Make version. This is problematic due to the MAKEFLAGS incompatibility, especially the job server flag. Old Make versions used --jobserver-fds to propagate job server file descriptors, but Make >= 4.2 uses --jobserver-auth. The flag disagreement between the parent and the child Make would result in a process fork explosion. However, having a non-standard path in the shebang causes another issue; the generated source package is not portable as such a path does not exist in other build environments. This commit solves those conflicting demands. Hard-code '#!/usr/bin/make -f' in debian/rules to create a portable and Debian-compliant source package. Pass '--rules-file=$(MAKE) -f debian/rules' when dpkg-buildpackage is invoked from Makefile so that debian/rules is executed by the same Make program as used to start Kbuild. [1] https://www.debian.org/doc/debian-policy/ch-source.html#main-building-script-debian-rules Signed-off-by: Masahiro Yamada Tested-by: Nicolas Schier Reviewed-by: Nicolas Schier --- scripts/Makefile.package | 2 +- scripts/package/mkdebian | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/Makefile.package b/scripts/Makefile.package index 85beab0363d7..f8a948ec2c6b 100644 --- a/scripts/Makefile.package +++ b/scripts/Makefile.package @@ -148,7 +148,7 @@ deb-pkg srcdeb-pkg bindeb-pkg: $(if $(findstring source, $(build-type)), \ --unsigned-source --compression=$(KDEB_SOURCE_COMPRESS)) \ $(if $(findstring binary, $(build-type)), \ - -r$(KBUILD_PKG_ROOTCMD) -a$$(cat debian/arch), \ + --rules-file='$(MAKE) -f debian/rules' -r$(KBUILD_PKG_ROOTCMD) -a$$(cat debian/arch), \ --no-check-builddeps) \ $(DPKG_FLAGS)) diff --git a/scripts/package/mkdebian b/scripts/package/mkdebian index 9105abab9728..2829f5b8aea6 100755 --- a/scripts/package/mkdebian +++ b/scripts/package/mkdebian @@ -264,7 +264,7 @@ EOF fi cat < debian/rules -#!$(command -v $MAKE) -f +#!/usr/bin/make -f srctree ?= . KERNELRELEASE = ${KERNELRELEASE} From patchwork Sat Jul 29 14:38:14 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Masahiro Yamada X-Patchwork-Id: 128152 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:918b:0:b0:3e4:2afc:c1 with SMTP id s11csp1075190vqg; Sat, 29 Jul 2023 08:02:10 -0700 (PDT) X-Google-Smtp-Source: APBJJlH0vA4vy92oQCOacEXDDyTRx/NDw/EcYqjZmERSstN39Gw+Gt2JV29jpx6N91KIpxTrMTVn X-Received: by 2002:a17:90b:1914:b0:263:e133:b9c9 with SMTP id mp20-20020a17090b191400b00263e133b9c9mr4514579pjb.34.1690642930002; Sat, 29 Jul 2023 08:02:10 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1690642929; cv=none; d=google.com; s=arc-20160816; b=XHmXZ0h3a5LUzL7LA38jwK1bUeOS/NXzIpclTFYvn4B/w0fdP0Bdnc/4Xa2YWO9b98 NV8BC8YZLV23xefPUGgU3wb3DiThtj1on5fo9fCe5xfZIc2AbWDM2SZ2TV+E16fHFaFX BziJypyda9CBvT84CY3hauaPbwaBmZ+U9h271KHVg1mfpl9IS3xZTQs1ZVtVZDcSGPnZ Nh9NzXtb9h1nzJp6AvAg43KlAJBB4Xo2D21zyYNOrlZHJEaW/fY7WbV2LsTOeTUWrLrz T39S/im03AvrmKYkjwCuRShESO/sP3nbXhUhSysqAzgLPuiRZZdkfCObVLqBKEgPzR/E cEyg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=7DF7vnMk9z3bp9q1+ZIrWXy8TiBJ58UlUwUAm6Y+x2I=; fh=eslil6/SEaAa2rUAmnzCWClLUU/9b0t1QC9lKZrgFu4=; b=klClOfTZ9kAX/KscxTX+ZZI0NGTys7iRkItQ6MeCnZhbc83MKY59HRT6bk8nRVdfIW C2+BgH3Ef8nICewgWib8rU1yuIJKa4VKiNLZWWHDkDIgVliDl7JKTHRxdvuYx7aGFmYQ bYIIf1EA/rMM6I6tX6111j2Ev317TCxiV6fGXOnEk10w9hQo+fq1xg41GcCl8bRqarfX Mghcx9IFqQVf0JKgP0+dff5YeK+J+QPtmsNRf2gv0BgWVIGlyr+RztXZUL1kK2B2bVVu iZtFHO28f4A7rriyjfJoe8bW/sAeHaOj4DIfu09vdGERkkqQaobc1wP3f3asGyJcWq8e 62gw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=mQ7JCIPW; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id o3-20020a17090a744300b00268256817fasi6197009pjk.105.2023.07.29.08.01.56; Sat, 29 Jul 2023 08:02:09 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=mQ7JCIPW; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231710AbjG2Oi1 (ORCPT + 99 others); Sat, 29 Jul 2023 10:38:27 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60520 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231634AbjG2OiY (ORCPT ); Sat, 29 Jul 2023 10:38:24 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 14D55A8; Sat, 29 Jul 2023 07:38:23 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id A74DB60C68; Sat, 29 Jul 2023 14:38:22 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 79C52C433CB; Sat, 29 Jul 2023 14:38:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1690641502; bh=PpYGpsj8tfX+WSYA1L58aqHkY/e/BiqQbpLrV6MGLQE=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=mQ7JCIPWJZ7xuTZO2VlsdgACH+6Zwk5PwJgK+otQOnmpzr0MKj5zIyWYt6wxu7Ndn /BG0mM5EIeuL9aWONTLwsE/pjpIWqLjr4BblSZ1foPmi80vlSAgsfb/YTeXTIFr69g iiN250sX8GJUXeUt8h3EBZJ6sY3c8/8dtIwT5xqv9ZrgBRm12HItDkCg16cCg98kfh KVjVTRBCNI/XzDYYYUjXbgLwFGnFObUV23KzxZIDDrHCXpUbzKwRijZYFJUEPM6UnZ 2DnVvwB8gLhUIEWjYap4Bg8v7RwPHKwB1OLAowVt6h6wfY0N3CoL8uvHwY4HkNTh4V LyD4FWt24ogYw== From: Masahiro Yamada To: linux-kbuild@vger.kernel.org Cc: linux-kernel@vger.kernel.org, Ben Hutchings , Masahiro Yamada , Nathan Chancellor , Nick Desaulniers , Nicolas Schier Subject: [PATCH 2/2] kbuild: deb-pkg: split debian/rules Date: Sat, 29 Jul 2023 23:38:14 +0900 Message-Id: <20230729143814.1509196-2-masahiroy@kernel.org> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20230729143814.1509196-1-masahiroy@kernel.org> References: <20230729143814.1509196-1-masahiroy@kernel.org> MIME-Version: 1.0 X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1772767600960219845 X-GMAIL-MSGID: 1772767600960219845 debian/rules is generated by shell scripting, but the escape sequence (\$) is unreadable. debian/rules embeds only two variables (ARCH and KERNELRELEASE). Split them out to debian/rules.vars, and check-in the rest of Makefile code to scripts/package/debian/rules. Signed-off-by: Masahiro Yamada Tested-by: Nicolas Schier Reviewed-by: Nicolas Schier --- scripts/package/debian/rules | 28 ++++++++++++++++++++++++++++ scripts/package/mkdebian | 32 ++++---------------------------- 2 files changed, 32 insertions(+), 28 deletions(-) create mode 100755 scripts/package/debian/rules diff --git a/scripts/package/debian/rules b/scripts/package/debian/rules new file mode 100755 index 000000000000..a4e5ab5abdd9 --- /dev/null +++ b/scripts/package/debian/rules @@ -0,0 +1,28 @@ +#!/usr/bin/make -f +# SPDX-License-Identifier: GPL-2.0-only + +srctree ?= . + +include debian/rules.vars + +.PHONY: binary binary-indep binary-arch +binary: binary-arch binary-indep +binary-indep: build-indep +binary-arch: build-arch + $(MAKE) -f $(srctree)/Makefile ARCH=$(ARCH) \ + KERNELRELEASE=$(KERNELRELEASE) \ + run-command KBUILD_RUN_COMMAND=+$(srctree)/scripts/package/builddeb + +.PHONY: build build-indep build-arch +build: build-arch build-indep +build-indep: +build-arch: + $(MAKE) -f $(srctree)/Makefile ARCH=$(ARCH) \ + KERNELRELEASE=$(KERNELRELEASE) \ + $(shell $(srctree)/scripts/package/deb-build-option) \ + olddefconfig all + +.PHONY: clean +clean: + rm -rf debian/files debian/linux-* + $(MAKE) -f $(srctree)/Makefile ARCH=${ARCH} clean diff --git a/scripts/package/mkdebian b/scripts/package/mkdebian index 2829f5b8aea6..498b579c6320 100755 --- a/scripts/package/mkdebian +++ b/scripts/package/mkdebian @@ -263,35 +263,11 @@ Description: Linux kernel debugging symbols for $version EOF fi -cat < debian/rules -#!/usr/bin/make -f - -srctree ?= . +cat < debian/rules.vars +ARCH = ${ARCH} KERNELRELEASE = ${KERNELRELEASE} - -.PHONY: clean build build-arch build-indep binary binary-arch binary-indep - -build-indep: -build-arch: - \$(MAKE) -f \$(srctree)/Makefile ARCH=${ARCH} \ - KERNELRELEASE=\$(KERNELRELEASE) \ - \$(shell \$(srctree)/scripts/package/deb-build-option) \ - olddefconfig all - -build: build-arch - -binary-indep: -binary-arch: build-arch - \$(MAKE) -f \$(srctree)/Makefile ARCH=${ARCH} \ - KERNELRELEASE=\$(KERNELRELEASE) \ - run-command KBUILD_RUN_COMMAND=+\$(srctree)/scripts/package/builddeb - -clean: - rm -rf debian/files debian/linux-* - \$(MAKE) -f \$(srctree)/Makefile ARCH=${ARCH} clean - -binary: binary-arch EOF -chmod +x debian/rules + +cp ${srctree}/scripts/package/debian/rules debian/ exit 0