clone: warn on failure to repo_init()

The --bundle-uri option was added in 55568919616 (clone: add
--bundle-uri option, 2022-08-09), but this also introduced a call to
repo_init() whose return value was ignored. Fix that ignored value by
warning that the bundle URI process could not continue if it failed.

Signed-off-by: Derrick Stolee <derrickstolee@github.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/builtin/clone.c b/builtin/clone.c
index 4463789680bdb901993eb6f74fd9932dae4e7c36..e21d42dfee54338c4beba9431b469d1f285c0bdf 100644 (file)
--- a/builtin/clone.c
+++ b/builtin/clone.c
@@ -1245,8 +1245,9 @@ int cmd_clone(int argc, const char **argv, const char *prefix)
         */
        if (bundle_uri) {
                /* At this point, we need the_repository to match the cloned repo. */
-               repo_init(the_repository, git_dir, work_tree);
-               if (fetch_bundle_uri(the_repository, bundle_uri))
+               if (repo_init(the_repository, git_dir, work_tree))
+                       warning(_("failed to initialize the repo, skipping bundle URI"));
+               else if (fetch_bundle_uri(the_repository, bundle_uri))
                        warning(_("failed to fetch objects from bundle URI '%s'"),
                                bundle_uri);
        }