Skip to content

Commit

Permalink
builder-manifest: run appstreamcli compose with --no-partial-urls
Browse files Browse the repository at this point in the history
Only when mirroring. This is to be compatible with existing flatpak
deployment and appstream-glib

See ximion/appstream#505

Should fix #573

Signed-off-by: Hubert Figuière <[email protected]>
  • Loading branch information
hfiguiere committed Jan 23, 2024
1 parent 63905e5 commit c9f7ad0
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 5 deletions.
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -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:

Expand Down
2 changes: 1 addition & 1 deletion configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ AC_DEFINE([GLIB_VERSION_MAX_ALLOWED], [GLIB_VERSION_2_70], [Glib max version])
GLIB_REQS=2.66
OSTREE_REQS=2017.14
FLATPAK_REQS=0.99.1
APPSTREAMCLI_REQS=0.15.0
APPSTREAMCLI_REQS=0.16.3
SYSTEM_DEBUGEDIT_REQS=5.0
LIBDW_REQS=0.172

Expand Down
2 changes: 1 addition & 1 deletion meson.build
Original file line number Diff line number Diff line change
Expand Up @@ -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')
Expand Down
1 change: 1 addition & 0 deletions src/builder-manifest.c
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down

0 comments on commit c9f7ad0

Please sign in to comment.