snapcraft: edk2 submodules fixes

- Move downloading submodules into override-pull stage. - edk2: remove unnecessary fix subhook submodule url Signed-off-by: Thomas Parrott <[email protected]> (cherry picked from commit b34e573) (cherry picked from commit 86c7759)
canonical · Nov 13, 2024 · 7e6ab2f · 7e6ab2f
1 parent 98c70c3
commit 7e6ab2f
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/snapcraft.yaml b/snapcraft.yaml
@@ -249,15 +249,14 @@ parts:
       set -ex
       git clone https://github.com/tianocore/edk2 . -b edk2-stable202208
 
+      # Pull submodules after switching to source-commit
+      git submodule update --init --recursive
+
       git config user.email "[email protected]"
       git config user.name "LXD snap builder"
     override-build: |-
       [ "$(uname -m)" != "x86_64" ] && [ "$(uname -m)" != "aarch64" ] && exit 0
 
-      # Fix submodules
-      sed -i "s#https://git.cryptomilk.org/projects/cmocka#https://gitlab.com/cmocka/cmocka#g" .gitmodules
-      git submodule update --init --recursive
-
       # Apply patches
       patch -p1 < "${SNAPCRAFT_PROJECT_DIR}/patches/edk2-0001-force-DUID-LLT.patch"
       cp "${SNAPCRAFT_PROJECT_DIR}/patches/edk2-0002-logo.bmp" MdeModulePkg/Logo/Logo.bmp