Force fetched submodule checkouts

Author: adam-ce

AddRepo.cmake

@@ -159,7 +159,7 @@ function(_alp_add_repo_prepare_submodules name repo_dir revision checkout_perfor
         message(STATUS "[alp/git] ${name}: fetching missing recursive submodule revisions in '${repo_dir}'.")
         execute_process(
             COMMAND "${GIT_EXECUTABLE}" -c protocol.file.allow=always submodule update
-                --init --recursive --checkout --depth 1
+                --init --recursive --checkout --force --depth 1
             WORKING_DIRECTORY "${repo_dir}"
             RESULT_VARIABLE _update_result
             OUTPUT_VARIABLE _output
@@ -168,23 +168,6 @@ function(_alp_add_repo_prepare_submodules name repo_dir revision checkout_perfor
             _alp_add_repo_fail("${name}" "${repo_dir}" "${revision}"
                 "updating recursive submodules" "${_error}${_output}")
         endif()
-
-        # A failed --no-fetch pass can leave shallow submodules checked out at
-        # commits other than the gitlinks recorded by their parents.  Once the
-        # fallback has fetched every missing object, converge all worktrees
-        # from local objects before checking the repository postconditions.
-        message(STATUS "[alp/git] ${name}: finalizing recursive submodules from local objects in '${repo_dir}'.")
-        execute_process(
-            COMMAND "${GIT_EXECUTABLE}" -c protocol.file.allow=always submodule update
-                --init --recursive --checkout --force --depth 1 --no-fetch
-            WORKING_DIRECTORY "${repo_dir}"
-            RESULT_VARIABLE _finalize_result
-            OUTPUT_VARIABLE _output
-            ERROR_VARIABLE _error)
-        if(NOT _finalize_result EQUAL 0)
-            _alp_add_repo_fail("${name}" "${repo_dir}" "${revision}"
-                "finalizing recursive submodules" "${_error}${_output}")
-        endif()
     endif()
 endfunction()