deploy: Be a bit more verbose about SELinux bits
authorColin Walters <walters@verbum.org>
Mon, 28 Mar 2022 21:46:59 +0000 (17:46 -0400)
committerColin Walters <walters@verbum.org>
Mon, 28 Mar 2022 21:46:59 +0000 (17:46 -0400)
Let's log when we don't find the expected CLI argument which
will help debug things.

src/libostree/ostree-sysroot-deploy.c

index 3f301891a51310428b585c93ee6089013f08e52b..5219e2a4f958f8e53f558ab79785a77981a40a8c 100644 (file)
@@ -2926,9 +2926,12 @@ sysroot_finalize_selinux_policy (int deployment_dfd, GError **error)
                           SEMODULE_HELP_ARGC, &exit_status, &stdout, error))
     return FALSE;
   if (!g_spawn_check_exit_status (exit_status, error))
-    return FALSE;
+    return glnx_prefix_error (error, "failed to run semodule");
   if (!strstr(stdout, "--rebuild-if-modules-changed"))
-    return TRUE;
+    {
+      ot_journal_print (LOG_INFO, "semodule does not have --rebuild-if-modules-changed");
+      return TRUE;
+    }
 
   static const gchar * const SEMODULE_REBUILD_ARGV[] = {
     "semodule", "-N", "--rebuild-if-modules-changed"