Skip to content

Commit

Permalink
builder-main: install platform as well as sdk if both are built
Browse files Browse the repository at this point in the history
Tested building an SDK+Runtime and a Runtime without custom Sdk (thus no
platform-id)

Fixes: #630
  • Loading branch information
dcbaker authored and TingPing committed Feb 28, 2025
1 parent f2d0567 commit a9d386b
Showing 1 changed file with 18 additions and 6 deletions.
24 changes: 18 additions & 6 deletions src/builder-main.c
Original file line number Diff line number Diff line change
Expand Up @@ -1149,13 +1149,25 @@ main (int argc,
passed and there were no changes, do nothing in that case */
if (export_repo == NULL)
g_printerr ("NOTE: No export due to --require-changes, ignoring --install\n");
else if (!do_install (build_context, flatpak_file_get_path_cached (export_repo),
builder_manifest_get_id (manifest),
builder_manifest_get_branch (manifest, build_context),
&error))
else
{
g_printerr ("Install failed: %s\n", error->message);
return 1;
if (!do_install (build_context, flatpak_file_get_path_cached (export_repo),
builder_manifest_get_id (manifest),
builder_manifest_get_branch (manifest, build_context),
&error))
{
g_printerr ("Install failed: %s\n", error->message);
return 1;
}
if (builder_manifest_get_id_platform (manifest) &&
(!do_install (build_context, flatpak_file_get_path_cached (export_repo),
builder_manifest_get_id_platform (manifest),
builder_manifest_get_branch (manifest, build_context),
&error)))
{
g_printerr ("Install failed: %s\n", error->message);
return 1;
}
}
}

Expand Down

0 comments on commit a9d386b

Please sign in to comment.