lib/repo-finder-config: Add some more debug output
authorPhilip Withnall <withnall@endlessm.com>
Fri, 11 Aug 2017 09:14:58 +0000 (10:14 +0100)
committerAtomic Bot <atomic-devel@projectatomic.io>
Fri, 11 Aug 2017 18:07:29 +0000 (18:07 +0000)
This makes diagnosing false negatives a little easier.

Signed-off-by: Philip Withnall <withnall@endlessm.com>
Closes: #1075
Approved by: jlebon

src/libostree/ostree-repo-finder-config.c

index 2f9841dba0f2deca0f77c6f99cd6eb95843c7248..ada8479e96b2ff7e28b2393bdf5e0a8e8bc90bf0 100644 (file)
@@ -114,6 +114,7 @@ ostree_repo_finder_config_resolve_async (OstreeRepoFinder                  *find
       g_autoptr(GHashTable) remote_refs = NULL;  /* (element-type OstreeCollectionRef utf8) */
       const gchar *checksum;
       g_autofree gchar *remote_collection_id = NULL;
+      gboolean resolved_a_ref = FALSE;
 
       remote_name = remotes[i];
 
@@ -147,6 +148,7 @@ ostree_repo_finder_config_resolve_async (OstreeRepoFinder                  *find
                * @supported_ref_to_checksum. */
               g_debug ("Resolved ref (%s, %s) to remote ā€˜%s’.",
                        refs[j]->collection_id, refs[j]->ref_name, remote_name);
+              resolved_a_ref = TRUE;
 
               supported_ref_to_checksum = g_hash_table_lookup (repo_name_to_refs, remote_name);
 
@@ -162,6 +164,9 @@ ostree_repo_finder_config_resolve_async (OstreeRepoFinder                  *find
                                    (gpointer) refs[j], g_strdup (checksum));
             }
         }
+
+      if (!resolved_a_ref)
+        g_debug ("Ignoring remote ā€˜%s’ due to it not advertising any of the requested refs.", remote_name);
     }
 
   /* Aggregate the results. */