From: Michal Schmidt <mschmidt@redhat.com>
Date: Wed, 27 Feb 2013 22:31:35 +0000 (+0100)
Subject: core/transaction: replace a bare status_printf()
X-Git-Tag: v198~157
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=297d0749dd82ea2442203d53c23ee401bdf46fca;p=elogind.git

core/transaction: replace a bare status_printf()

Like other status messages, this one too should not be printed
unconditionally, but it should take the manager state into account.
unit_status_printf() does that.
---

diff --git a/src/core/transaction.c b/src/core/transaction.c
index 032936635..4a8d90e6e 100644
--- a/src/core/transaction.c
+++ b/src/core/transaction.c
@@ -396,8 +396,8 @@ static int transaction_verify_order_one(Transaction *tr, Job *j, Job *from, unsi
                                        "Job %s/%s deleted to break ordering cycle starting with %s/%s",
                                        delete->unit->id, job_type_to_string(delete->type),
                                        j->unit->id, job_type_to_string(j->type));
-                        status_printf(ANSI_HIGHLIGHT_RED_ON " SKIP " ANSI_HIGHLIGHT_OFF, true,
-                                      "Ordering cycle found, skipping %s", unit_description(delete->unit));
+                        unit_status_printf(delete->unit, ANSI_HIGHLIGHT_RED_ON " SKIP " ANSI_HIGHLIGHT_OFF,
+                                           "Ordering cycle found, skipping %s");
                         transaction_delete_unit(tr, delete->unit);
                         return -EAGAIN;
                 }