builder-manifest: run appstreamcli compose with --no-partial-urls

Only when mirroring. This is to be compatible with existing flatpak deployment and appstream-glib See ximion/appstream#505 Should fix flatpak#573 Signed-off-by: Hubert Figuière <[email protected]>
hfiguiere · Jan 23, 2024 · 9f9aa46 · 9f9aa46
1 parent 63905e5
commit 9f9aa46
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/README.md b/README.md
@@ -28,9 +28,9 @@ Configure arguments are documented in `meson_options.txt`.
 
 ## Runtime dependencies
 
-The `flatpak-builder` tool requires `flatpak` being available on the host to
-function. Depending on the manifest used it also requires some commands be available on
-the host.
+The `flatpak-builder` tool requires `flatpak` and `appstreamcli` >=
+0.16.3 being available on the host to function. Depending on the
+manifest used it also requires some commands be available on the host.
 
 Very commonly used:
 

diff --git a/meson.build b/meson.build
@@ -41,7 +41,7 @@ endforeach
 debugedit = find_program('debugedit', version: '>= 5.0')
 
 # Require appstream with compose plugin installed
-appstreamcli = find_program('appstreamcli', version: '>= 0.15.0')
+appstreamcli = find_program('appstreamcli', version: '>= 0.16.3')
 appstreamcli_compose = run_command(appstreamcli, ['compose', '--help'], check: true)
 
 subdir('src')

diff --git a/src/builder-manifest.c b/src/builder-manifest.c
@@ -3053,6 +3053,7 @@ builder_manifest_cleanup (BuilderManifest *self,
               g_print ("Running appstreamcli compose\n");
               g_print ("Saving screenshots in %s\n", flatpak_file_get_path_cached (media_dir));
               if (!appstreamcli_compose (error,
+                                         "--no-partial-urls",
                                          "--prefix=/",
                                          origin,
                                          arg_base_url,