From nobody Fri Apr 19 02:59:37 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1570455502; cv=none; d=zoho.com; s=zohoarc; b=fZH7h+4+XB9D1vpTPO0YdimSDgIOqoHMjpCnDbTqQsjyomPWfpZsWhscAzzfotH6p6jKpsJHXseh9zGrOVRLgBZCPf3LrkhEfhY86QP7uEUuLmQkG9QfQF+YYS5FOFyTjT3VvcCgB0JnZC6mmLV0sc2L+F721PObHDSarltSLwQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1570455502; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=FdILkDQdVuAEyRXJ/cMICJOCbNJqZMdly5B7S862Qe4=; b=E+30+kjaPFzFk+fhPaRGqIJm7sQ55pMOgOq6MjeBR+7/ec2ZGaoymK7/t9Ld08oqpyRl545Tq1t6drnBqLd+43NWzMiKxLc5G41EKU7NAEDOndFi1uyLQvDpuMQsFL4c+XL6y8J/rPM5en9axhlKWOrkUHTDJxlcxgUQqrq45TE= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1570455502599962.0889454629474; Mon, 7 Oct 2019 06:38:22 -0700 (PDT) Received: from localhost ([::1]:44892 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iHTDP-0005nn-6Q for importer@patchew.org; Mon, 07 Oct 2019 09:38:19 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:60080) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iHTB1-0003ef-1Q for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:35:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iHTAz-0007IJ-VV for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:35:50 -0400 Received: from mx1.redhat.com ([209.132.183.28]:54238) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1iHTAz-0007Hz-Pf; Mon, 07 Oct 2019 09:35:49 -0400 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 07AB7C04FFE0; Mon, 7 Oct 2019 13:35:49 +0000 (UTC) Received: from x1w.redhat.com (unknown [10.40.206.44]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 0B62060127; Mon, 7 Oct 2019 13:35:46 +0000 (UTC) From: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Subject: [PULL 1/5] make-release: pull in edk2 submodules so we can build it from tarballs Date: Mon, 7 Oct 2019 15:35:36 +0200 Message-Id: <20191007133540.30623-2-philmd@redhat.com> In-Reply-To: <20191007133540.30623-1-philmd@redhat.com> References: <20191007133540.30623-1-philmd@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.31]); Mon, 07 Oct 2019 13:35:49 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 209.132.183.28 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , qemu-stable@nongnu.org, Laszlo Ersek , Michael Roth , Bruce Rogers Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" From: Michael Roth The `make efi` target added by 536d2173 is built from the roms/edk2 submodule, which in turn relies on additional submodules nested under roms/edk2. The make-release script currently only pulls in top-level submodules, so these nested submodules are missing in the resulting tarball. We could try to address this situation more generally by recursively pulling in all submodules, but this doesn't necessarily ensure the end-result will build properly (this case also required other changes). Additionally, due to the nature of submodules, we may not always have control over how these sorts of things are dealt with, so for now we continue to handle it on a case-by-case in the make-release script. Cc: Laszlo Ersek Cc: Bruce Rogers Cc: qemu-stable@nongnu.org # v4.1.0 Reported-by: Bruce Rogers Reviewed-by: Philippe Mathieu-Daud=C3=A9 Tested-by: Philippe Mathieu-Daud=C3=A9 Signed-off-by: Michael Roth Message-Id: <20190912231202.12327-2-mdroth@linux.vnet.ibm.com> Signed-off-by: Philippe Mathieu-Daud=C3=A9 --- scripts/make-release | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/scripts/make-release b/scripts/make-release index b4af9c9e52..a2a8cda33c 100755 --- a/scripts/make-release +++ b/scripts/make-release @@ -20,6 +20,14 @@ git checkout "v${version}" git submodule update --init (cd roms/seabios && git describe --tags --long --dirty > .version) (cd roms/skiboot && ./make_version.sh > .version) +# Fetch edk2 submodule's submodules, since it won't have access to them via +# the tarball later. +# +# A more uniform way to handle this sort of situation would be nice, but we +# don't necessarily have much control over how a submodule handles its +# submodule dependencies, so we continue to handle these on a case-by-case +# basis for now. +(cd roms/edk2 && git submodule update --init) popd tar --exclude=3D.git -cjf ${destination}.tar.bz2 ${destination} rm -rf ${destination} --=20 2.21.0 From nobody Fri Apr 19 02:59:37 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1570455643; cv=none; d=zoho.com; s=zohoarc; b=L0qze2DtiNAMr+PHit7BoPyNg+EC/Qhl+d77gsQRJXwaOREpx0/TaruN9g4e0SSl4WaVUVn/eSzxG19ph4borZF/HP7HPcWHOsLrCw6qfy9ZmGmUrawQjmDzfss3pNSmIYpd0mMxUsu0uoYNtDdjg3JdhNPCVTL7l4rXA037z3c= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1570455643; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=XUuRLMzn5W6IKKahG807w1yWNQVinic+hWFOPunDVjI=; b=a0mIjr+lL3in1psG3aoUOVm+pa0wt2l3ETBmFm/DnhXgtZ8ywjgrV47A/tfdARrd6+YbV7nSkW8abfTpeSb2c1QilEpF+qEPHSdh2vFQ+T7IU9lm2o+2hdmFENa7zvUTE55z/ThlnP2rxmjFaCqu8OcBW0lYSZtrgAmhd9a6X7M= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1570455643726164.27974093048942; Mon, 7 Oct 2019 06:40:43 -0700 (PDT) Received: from localhost ([::1]:44932 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iHTFe-0000Nq-L5 for importer@patchew.org; Mon, 07 Oct 2019 09:40:38 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:60110) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iHTB3-0003hv-Nw for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:35:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iHTB2-0007Jo-HX for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:35:53 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49488) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1iHTB2-0007JP-Bz; Mon, 07 Oct 2019 09:35:52 -0400 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 8F8ED18CB8E2; Mon, 7 Oct 2019 13:35:51 +0000 (UTC) Received: from x1w.redhat.com (unknown [10.40.206.44]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 94431600C1; Mon, 7 Oct 2019 13:35:49 +0000 (UTC) From: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Subject: [PULL 2/5] roms/Makefile.edk2: don't pull in submodules when building from tarball Date: Mon, 7 Oct 2019 15:35:37 +0200 Message-Id: <20191007133540.30623-3-philmd@redhat.com> In-Reply-To: <20191007133540.30623-1-philmd@redhat.com> References: <20191007133540.30623-1-philmd@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.6.2 (mx1.redhat.com [10.5.110.63]); Mon, 07 Oct 2019 13:35:51 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 209.132.183.28 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , qemu-stable@nongnu.org, Laszlo Ersek , Michael Roth , Bruce Rogers Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" From: Michael Roth Currently the `make efi` target pulls submodules nested under the roms/edk2 submodule as dependencies. However, when we attempt to build from a tarball this fails since we are no longer in a git tree. A preceding patch will pre-populate these submodules in the tarball, so assume this build dependency is only needed when building from a git tree. Cc: Laszlo Ersek Cc: Bruce Rogers Cc: qemu-stable@nongnu.org # v4.1.0 Reported-by: Bruce Rogers Reviewed-by: Laszlo Ersek Reviewed-by: Philippe Mathieu-Daud=C3=A9 Tested-by: Philippe Mathieu-Daud=C3=A9 Signed-off-by: Michael Roth Message-Id: <20190912231202.12327-3-mdroth@linux.vnet.ibm.com> Signed-off-by: Philippe Mathieu-Daud=C3=A9 --- roms/Makefile.edk2 | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/roms/Makefile.edk2 b/roms/Makefile.edk2 index c2f2ff59d5..33a074d3a4 100644 --- a/roms/Makefile.edk2 +++ b/roms/Makefile.edk2 @@ -46,8 +46,13 @@ all: $(foreach flashdev,$(flashdevs),../pc-bios/edk2-$(f= lashdev).fd.bz2) \ # files. .INTERMEDIATE: $(foreach flashdev,$(flashdevs),../pc-bios/edk2-$(flashdev)= .fd) =20 +# Fetch edk2 submodule's submodules. If it is not in a git tree, assume +# we're building from a tarball and that they've already been fetched by +# make-release/tarball scripts. submodules: - cd edk2 && git submodule update --init --force + if test -d edk2/.git; then \ + cd edk2 && git submodule update --init --force; \ + fi =20 # See notes on the ".NOTPARALLEL" target and the "+" indicator in # "tests/uefi-test-tools/Makefile". --=20 2.21.0 From nobody Fri Apr 19 02:59:37 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1570455611; cv=none; d=zoho.com; s=zohoarc; b=k8txKHAOC62VqpcWE7klGXk2KARxt4aznoyElwIksvtpGE+iRmWNQDbtZ64/LF/dJPpGToIhsThMnxokuINiuCE5G9qey1RBnx7G7knIpq1F4W1D397waEP1T49aat5vVMMccpVDdMYBpjKsoIRBZbvZyJ+jldYE+S7yYocUQOA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1570455611; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=bGgiFsVinZ8hqBpwbTjLdPmgEJ5OkqULuCfcxqoDs0Q=; b=ji/uNp7PW4Rw2jMxtasUxN6PBpqV28FUWyR9dFkl4ygaxO3OWVlLOg0pqn0Kd3+aF1LWLFzrKaxd7gXZ1sHVW71fWCcBQW+hea8AZXEuIkaNy4ELX5KGQThRtuqxSh6kCWiiq6PPxhDg4UQCL5bWbOJXuFVe951tUpcrhtIYZa8= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1570455611965318.35662617886965; Mon, 7 Oct 2019 06:40:11 -0700 (PDT) Received: from localhost ([::1]:44930 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iHTFA-0008W6-Eu for importer@patchew.org; Mon, 07 Oct 2019 09:40:08 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:60151) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iHTB8-0003pV-3Y for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:35:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iHTB7-0007MS-2U for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:35:57 -0400 Received: from mx1.redhat.com ([209.132.183.28]:58234) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1iHTB6-0007M6-TQ for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:35:57 -0400 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 296F1308212F for ; Mon, 7 Oct 2019 13:35:56 +0000 (UTC) Received: from x1w.redhat.com (unknown [10.40.206.44]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 494CB600C1; Mon, 7 Oct 2019 13:35:51 +0000 (UTC) From: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Subject: [PULL 3/5] roms: Add a 'make help' target alias Date: Mon, 7 Oct 2019 15:35:38 +0200 Message-Id: <20191007133540.30623-4-philmd@redhat.com> In-Reply-To: <20191007133540.30623-1-philmd@redhat.com> References: <20191007133540.30623-1-philmd@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.47]); Mon, 07 Oct 2019 13:35:56 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 209.132.183.28 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: John Snow , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , Laszlo Ersek Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Various C projects provide a 'make help' target. Our root directory does so. The roms/ directory lacks a such rule, but already displays a help output when the default target is called. Add a 'help' target aliased to the default one, to avoid: $ make -C roms help make: *** No rule to make target 'help'. Stop. Reviewed-by: John Snow Reviewed-by: Laszlo Ersek Message-Id: <20190920171159.18633-1-philmd@redhat.com> Signed-off-by: Philippe Mathieu-Daud=C3=A9 --- roms/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/roms/Makefile b/roms/Makefile index 6cf07d3b44..3ffd13cc7e 100644 --- a/roms/Makefile +++ b/roms/Makefile @@ -51,7 +51,7 @@ SEABIOS_EXTRAVERSION=3D"-prebuilt.qemu.org" # EDK2_EFIROM =3D edk2/BaseTools/Source/C/bin/EfiRom =20 -default: +default help: @echo "nothing is build by default" @echo "available build targets:" @echo " bios -- update bios.bin (seabios)" --=20 2.21.0 From nobody Fri Apr 19 02:59:37 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1570455727; cv=none; d=zoho.com; s=zohoarc; b=QBmnxB4gS7zrYptoHNtTjXu0ZWBzHZ3A48f1YLl4t6ygeIf2GPyGvJAzW3vIMkFJPW87a92RxXkpHZBv08fuVsXQUBOzrlPrrdYdL3XPsdjXF/DfcDxBE/yv/asLNIRo3y7++UVA68hWNKuPBrkOgEYbH9B2RJ/BBTOAB2U0R7U= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1570455727; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=zDK0Fd3LecdaWlJHM9PWw4pX+aajUD+fyx+wKAXIGJ4=; b=d1Xesp4QbSrlvBxUe5lSGkFmtCW1tDNiz5z7GC95BC5KqwEaiPk7IkJsfRcDhYHYDhf9ybYjS6EvyEcea8aZ1y/p43fNA4D19e90vzkj0IEO8xn6LToXaZOY8UwZEkX2qS0AZBiP3bMx84WozPhJo77B5awLZALAnVD+asKxD84= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1570455727224853.9322054638873; Mon, 7 Oct 2019 06:42:07 -0700 (PDT) Received: from localhost ([::1]:44952 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iHTH4-00023y-6E for importer@patchew.org; Mon, 07 Oct 2019 09:42:06 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:60163) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iHTBB-0003sX-03 for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:36:01 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iHTB9-0007Ok-Va for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:36:00 -0400 Received: from mx1.redhat.com ([209.132.183.28]:36876) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1iHTB9-0007OJ-QM for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:35:59 -0400 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 16E7E30842A8 for ; Mon, 7 Oct 2019 13:35:58 +0000 (UTC) Received: from x1w.redhat.com (unknown [10.40.206.44]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 92A0860127; Mon, 7 Oct 2019 13:35:56 +0000 (UTC) From: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Subject: [PULL 4/5] edk2 build scripts: honor external BaseTools flags with uefi-test-tools Date: Mon, 7 Oct 2019 15:35:39 +0200 Message-Id: <20191007133540.30623-5-philmd@redhat.com> In-Reply-To: <20191007133540.30623-1-philmd@redhat.com> References: <20191007133540.30623-1-philmd@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.40]); Mon, 07 Oct 2019 13:35:58 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 209.132.183.28 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: John Snow , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , Laszlo Ersek Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" From: Laszlo Ersek Unify the recipe for "build-edk2-tools" in "tests/uefi-test-tools/Makefile" with the recipe for "edk2-basetools" in "roms/Makefile". Cc: Philippe Mathieu-Daud=C3=A9 Reviewed-by: Philippe Mathieu-Daud=C3=A9 Tested-by: Philippe Mathieu-Daud=C3=A9 Reviewed-by: John Snow Signed-off-by: Laszlo Ersek Message-Id: <20190920083808.21399-2-lersek@redhat.com> Signed-off-by: Philippe Mathieu-Daud=C3=A9 --- tests/uefi-test-tools/Makefile | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/uefi-test-tools/Makefile b/tests/uefi-test-tools/Makefile index 1d78bc14d5..7e0177d733 100644 --- a/tests/uefi-test-tools/Makefile +++ b/tests/uefi-test-tools/Makefile @@ -99,7 +99,9 @@ Build/bios-tables-test.%.efi: build-edk2-tools +./build.sh $(edk2_dir) BiosTablesTest $* $@ =20 build-edk2-tools: - $(MAKE) -C $(edk2_dir)/BaseTools + $(MAKE) -C $(edk2_dir)/BaseTools \ + EXTRA_OPTFLAGS=3D'$(EDK2_BASETOOLS_OPTFLAGS)' \ + EXTRA_LDFLAGS=3D'$(EDK2_BASETOOLS_LDFLAGS)' =20 clean: rm -rf Build Conf log --=20 2.21.0 From nobody Fri Apr 19 02:59:37 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1570455846; cv=none; d=zoho.com; s=zohoarc; b=lqC5/Nku1eVmfwfr3oaEiuyM/zrqGWaxckmflRo+HtXG887glXkNNWpErXq2xh4OXOCxLN7+uj6aKnlBeURiqo7IpLl/kp3wdLmHguIPiH3U2dOEgS3MLdcBSMDuEAdt6i0+ol3hNMs2g84Qr2i/mT/aC/yS0rLZQ0vkUpScRhc= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1570455846; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=d7LK5JqRYqs9DMyEsPHVQx1fFwIS/qp65iYt1PgGzFY=; b=cj1npdMWRGjkKjMAoA12eUce9SswjBvBVAaW7ZtDrB5QcoLkQvHw9z6Kowh6XMUUVH01p7p9qKvzm9wfGucLqwd3ztEFWRg7V5P6533fsIlAEvo3EKlLnrGrZtigG1gIIcl1tQg2MIGi1q0OEL5ek6InAWZfQElep1E8hRpo+Cw= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1570455846322824.3945125986945; Mon, 7 Oct 2019 06:44:06 -0700 (PDT) Received: from localhost ([::1]:44970 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iHTIw-0003uH-Sz for importer@patchew.org; Mon, 07 Oct 2019 09:44:02 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:60178) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iHTBF-0003yd-QZ for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:36:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iHTBE-0007Qv-Co for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:36:05 -0400 Received: from mx1.redhat.com ([209.132.183.28]:55630) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1iHTBE-0007Qa-4h for qemu-devel@nongnu.org; Mon, 07 Oct 2019 09:36:04 -0400 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 597DC116BB23 for ; Mon, 7 Oct 2019 13:36:03 +0000 (UTC) Received: from x1w.redhat.com (unknown [10.40.206.44]) by smtp.corp.redhat.com (Postfix) with ESMTPS id A724C60127; Mon, 7 Oct 2019 13:35:58 +0000 (UTC) From: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Subject: [PULL 5/5] edk2 build scripts: work around TianoCore#1607 without forcing Python 2 Date: Mon, 7 Oct 2019 15:35:40 +0200 Message-Id: <20191007133540.30623-6-philmd@redhat.com> In-Reply-To: <20191007133540.30623-1-philmd@redhat.com> References: <20191007133540.30623-1-philmd@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.6.2 (mx1.redhat.com [10.5.110.65]); Mon, 07 Oct 2019 13:36:03 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 209.132.183.28 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: John Snow , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , Laszlo Ersek , Eduardo Habkost Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" From: Laszlo Ersek It turns out that forcing python2 for running the edk2 "build" utility is neither necessary nor sufficient. Forcing python2 is not sufficient for two reasons: - QEMU is moving away from python2, with python2 nearing EOL, - according to my most recent testing, the lacking dependency information in the makefiles that are generated by edk2's "build" utility can cause parallel build failures even when "build" is executed by python2. And forcing python2 is not necessary because we can still return to the original idea of filtering out jobserver-related options from MAKEFLAGS. So do that. While at it, cut short edk2's auto-detection of the python3.* minor version, by setting PYTHON_COMMAND to "python3" (which we expect to be available wherever we intend to build edk2). With this patch, the guest UEFI binaries that are used as part of the BIOS tables test, and the OVMF and ArmVirtQemu platform firmwares, will be built strictly in a single job, regardless of an outermost "-jN" make option. Alas, there appears to be no reliable way to build edk2 in an (outer make, inner make) environment, with a jobserver enabled. Cc: Eduardo Habkost Cc: John Snow Cc: Philippe Mathieu-Daud=C3=A9 Reported-by: John Snow Reviewed-by: Philippe Mathieu-Daud=C3=A9 Tested-by: Philippe Mathieu-Daud=C3=A9 Reviewed-by: John Snow Signed-off-by: Laszlo Ersek Message-Id: <20190920083808.21399-3-lersek@redhat.com> Signed-off-by: Philippe Mathieu-Daud=C3=A9 --- roms/Makefile | 1 + roms/edk2-build.sh | 4 ++-- roms/edk2-funcs.sh | 17 +++++++++++++++++ tests/uefi-test-tools/Makefile | 1 + tests/uefi-test-tools/build.sh | 6 ++++-- 5 files changed, 25 insertions(+), 4 deletions(-) diff --git a/roms/Makefile b/roms/Makefile index 3ffd13cc7e..da4efa47a9 100644 --- a/roms/Makefile +++ b/roms/Makefile @@ -141,6 +141,7 @@ build-efi-roms: build-pxe-roms # edk2-basetools: $(MAKE) -C edk2/BaseTools \ + PYTHON_COMMAND=3D$${EDK2_PYTHON_COMMAND:-python3} \ EXTRA_OPTFLAGS=3D'$(EDK2_BASETOOLS_OPTFLAGS)' \ EXTRA_LDFLAGS=3D'$(EDK2_BASETOOLS_LDFLAGS)' =20 diff --git a/roms/edk2-build.sh b/roms/edk2-build.sh index 4f46f8a6a2..d5391c7637 100755 --- a/roms/edk2-build.sh +++ b/roms/edk2-build.sh @@ -27,8 +27,7 @@ shift $num_args =20 cd edk2 =20 -# Work around . -export PYTHON_COMMAND=3Dpython2 +export PYTHON_COMMAND=3D${EDK2_PYTHON_COMMAND:-python3} =20 # Source "edksetup.sh" carefully. set +e +u +C @@ -43,6 +42,7 @@ fi # any), for the edk2 "build" utility. source ../edk2-funcs.sh edk2_toolchain=3D$(qemu_edk2_get_toolchain "$emulation_target") +MAKEFLAGS=3D$(qemu_edk2_quirk_tianocore_1607 "$MAKEFLAGS") edk2_thread_count=3D$(qemu_edk2_get_thread_count "$MAKEFLAGS") qemu_edk2_set_cross_env "$emulation_target" =20 diff --git a/roms/edk2-funcs.sh b/roms/edk2-funcs.sh index a9fae7ee89..3f4485b201 100644 --- a/roms/edk2-funcs.sh +++ b/roms/edk2-funcs.sh @@ -251,3 +251,20 @@ qemu_edk2_get_thread_count() printf '1\n' fi } + + +# Work around by +# filtering jobserver-related flags out of MAKEFLAGS. Print the result to = the +# standard output. +# +# Parameters: +# $1: the value of the MAKEFLAGS variable +qemu_edk2_quirk_tianocore_1607() +{ + local makeflags=3D"$1" + + printf %s "$makeflags" \ + | LC_ALL=3DC sed --regexp-extended \ + --expression=3D's/--jobserver-(auth|fds)=3D[0-9]+,[0-9]+//' \ + --expression=3D's/-j([0-9]+)?//' +} diff --git a/tests/uefi-test-tools/Makefile b/tests/uefi-test-tools/Makefile index 7e0177d733..1dcddcdbba 100644 --- a/tests/uefi-test-tools/Makefile +++ b/tests/uefi-test-tools/Makefile @@ -100,6 +100,7 @@ Build/bios-tables-test.%.efi: build-edk2-tools =20 build-edk2-tools: $(MAKE) -C $(edk2_dir)/BaseTools \ + PYTHON_COMMAND=3D$${EDK2_PYTHON_COMMAND:-python3} \ EXTRA_OPTFLAGS=3D'$(EDK2_BASETOOLS_OPTFLAGS)' \ EXTRA_LDFLAGS=3D'$(EDK2_BASETOOLS_LDFLAGS)' =20 diff --git a/tests/uefi-test-tools/build.sh b/tests/uefi-test-tools/build.sh index 8aa7935c43..3b78f30840 100755 --- a/tests/uefi-test-tools/build.sh +++ b/tests/uefi-test-tools/build.sh @@ -29,8 +29,7 @@ export PACKAGES_PATH=3D$(realpath -- "$edk2_dir") export WORKSPACE=3D$PWD mkdir -p Conf =20 -# Work around . -export PYTHON_COMMAND=3Dpython2 +export PYTHON_COMMAND=3D${EDK2_PYTHON_COMMAND:-python3} =20 # Source "edksetup.sh" carefully. set +e +u +C @@ -46,12 +45,15 @@ fi source "$edk2_dir/../edk2-funcs.sh" edk2_arch=3D$(qemu_edk2_get_arch "$emulation_target") edk2_toolchain=3D$(qemu_edk2_get_toolchain "$emulation_target") +MAKEFLAGS=3D$(qemu_edk2_quirk_tianocore_1607 "$MAKEFLAGS") +edk2_thread_count=3D$(qemu_edk2_get_thread_count "$MAKEFLAGS") qemu_edk2_set_cross_env "$emulation_target" =20 # Build the UEFI binary mkdir -p log build \ --arch=3D"$edk2_arch" \ + -n "$edk2_thread_count" \ --buildtarget=3DDEBUG \ --platform=3DUefiTestToolsPkg/UefiTestToolsPkg.dsc \ --tagname=3D"$edk2_toolchain" \ --=20 2.21.0