On 15/01/2019 11:29, Wei Liu wrote: > On Tue, Jan 15, 2019 at 11:27:12AM +0000, Andrew Cooper wrote: >> On 15/01/2019 11:09, Wei Liu wrote: >>> Users may choose to export a snapshot of OVMF and build it >>> with xen.git supplied ovmf-makefile. In that case we don't >>> need to call `git submodule`. >>> >>> Fixes b16281870e. >>> >>> Reported-by: Olaf Hering <o...@aepfle.de> >>> Signed-off-by: Wei Liu <wei.l...@citrix.com> >>> --- >>> tools/firmware/ovmf-makefile | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/tools/firmware/ovmf-makefile b/tools/firmware/ovmf-makefile >>> index 3de2fc0300..55f9992145 100644 >>> --- a/tools/firmware/ovmf-makefile >>> +++ b/tools/firmware/ovmf-makefile >>> @@ -16,7 +16,7 @@ all: build >>> >>> .PHONY: build >>> build: >>> - $(GIT) submodule update --init --recursive >>> + if test -e .git ; then $(GIT) submodule update --init --recursive ; fi >> You want -d rather than -e. > Anthony said .git could be a link.
Hmm - so it can... You get some weird effects with it being a link, but git does seem to tolerate it. Sorry for the noise. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel