On 8/22/2022 5:24 PM, Junio C Hamano wrote:
> "Derrick Stolee via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes:
>
>> + /*
>> + * Before fetching from the remote, download and install bundle
>> + * data from the --bundle-uri option.
>> + */
>> + 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))
>> + warning(_("failed to fetch objects from bundle URI '%s'"),
>> + bundle_uri);
>> + }
>
> I do not offhand know why I suddenly started seeing the issue for
> this relatively old commit I have had in my tree for at least 10
> days, but I am getting this
>
> builtin/clone.c: In function 'cmd_clone':
> builtin/clone.c:1248:17: error: ignoring return value of 'repo_init' declared with attribute 'warn_unused_result' [-Werror=unused-result]
> 1248 | repo_init(the_repository, git_dir, work_tree);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
>
> with the commit merged in 'seen'.
Thank you for pointing this out. Here is a patch that fixes
the issue from this patch:
--- >8 ---
>From 6df8bc6d7ffdc1b115d85ef9550bab5dcf1811f8 Mon Sep 17 00:00:00 2001
From: Derrick Stolee <derrickstolee@xxxxxxxxxx>
Date: Tue, 23 Aug 2022 09:53:47 -0400
Subject: [PATCH] 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@xxxxxxxxxx>
---
builtin/clone.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/builtin/clone.c b/builtin/clone.c
index 4463789680b..e21d42dfee5 100644
--- 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);
}
--
2.37.1.vfs.0.0.rebase
