All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Philippe Mathieu-Daudé" <philmd@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Philippe Mathieu-Daudé" <philmd@redhat.com>,
	qemu-stable@nongnu.org, "Laszlo Ersek" <lersek@redhat.com>,
	"Michael Roth" <mdroth@linux.vnet.ibm.com>,
	"Bruce Rogers" <brogers@suse.com>
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	[thread overview]
Message-ID: <20191007133540.30623-2-philmd@redhat.com> (raw)
In-Reply-To: <20191007133540.30623-1-philmd@redhat.com>

From: Michael Roth <mdroth@linux.vnet.ibm.com>

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 <lersek@redhat.com>
Cc: Bruce Rogers <brogers@suse.com>
Cc: qemu-stable@nongnu.org # v4.1.0
Reported-by: Bruce Rogers <brogers@suse.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
Message-Id: <20190912231202.12327-2-mdroth@linux.vnet.ibm.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 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=.git -cjf ${destination}.tar.bz2 ${destination}
 rm -rf ${destination}
-- 
2.21.0



  reply	other threads:[~2019-10-07 13:38 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-07 13:35 [PULL 0/5] edk2-next patches for 2019-10-07 Philippe Mathieu-Daudé
2019-10-07 13:35 ` Philippe Mathieu-Daudé [this message]
2019-10-07 13:35 ` [PULL 2/5] roms/Makefile.edk2: don't pull in submodules when building from tarball Philippe Mathieu-Daudé
2019-10-07 13:35 ` [PULL 3/5] roms: Add a 'make help' target alias Philippe Mathieu-Daudé
2019-10-07 13:35 ` [PULL 4/5] edk2 build scripts: honor external BaseTools flags with uefi-test-tools Philippe Mathieu-Daudé
2019-10-07 13:35 ` [PULL 5/5] edk2 build scripts: work around TianoCore#1607 without forcing Python 2 Philippe Mathieu-Daudé
2019-10-08 14:08 ` [PULL 0/5] edk2-next patches for 2019-10-07 Peter Maydell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191007133540.30623-2-philmd@redhat.com \
    --to=philmd@redhat.com \
    --cc=brogers@suse.com \
    --cc=lersek@redhat.com \
    --cc=mdroth@linux.vnet.ibm.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-stable@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.