TODO

@@ -1113,6 +1113,7 @@ External:
   - <command> <verb> -<TAB> should complete options, but currently does not
   - systemctl add-wants,add-requires
 
+* systemctl status foo.service should say that it is trigger by foo.timer
 * systemctl status should know about 'systemd-analyze calendar ... --iterations='
 * If timer has just OnInactiveSec=..., it should fire after a specified time
   after being started.

src/systemctl/systemctl.c

@@ -4050,9 +4050,6 @@ typedef struct UnitStatusInfo {
 
         char **dropin_paths;
 
-        char **triggered_by;
-        char **triggers;
-
         const char *load_error;
         const char *result;
 
@@ -4137,8 +4134,6 @@ static void unit_status_info_free(UnitStatusInfo *info) {
 
         strv_free(info->documentation);
         strv_free(info->dropin_paths);
-        strv_free(info->triggered_by);
-        strv_free(info->triggers);
         strv_free(info->listen);
 
         while ((c = info->conditions)) {
@@ -4152,17 +4147,6 @@ static void unit_status_info_free(UnitStatusInfo *info) {
         }
 }
 
-static void format_active_state(const char *active_state, const char **active_on, const char **active_off) {
-        if (streq_ptr(active_state, "failed")) {
-                *active_on = ansi_highlight_red();
-                *active_off = ansi_normal();
-        } else if (STRPTR_IN_SET(active_state, "active", "reloading")) {
-                *active_on = ansi_highlight_green();
-                *active_off = ansi_normal();
-        } else
-                *active_on = *active_off = "";
-}
-
 static void print_status_info(
                 sd_bus *bus,
                 UnitStatusInfo *i,
@@ -4182,7 +4166,14 @@ static void print_status_info(
         /* This shows pretty information about a unit. See
          * print_property() for a low-level property printer */
 
-        format_active_state(i->active_state, &active_on, &active_off);
+        if (streq_ptr(i->active_state, "failed")) {
+                active_on = ansi_highlight_red();
+                active_off = ansi_normal();
+        } else if (STRPTR_IN_SET(i->active_state, "active", "reloading")) {
+                active_on = ansi_highlight_green();
+                active_off = ansi_normal();
+        } else
+                active_on = active_off = "";
 
         printf("%s%s%s %s", active_on, special_glyph(SPECIAL_GLYPH_BLACK_CIRCLE), active_off, strna(i->id));
 
@@ -4222,7 +4213,7 @@ static void print_status_info(
                        on, strna(i->load_state), off);
 
         if (i->transient)
-                printf("  Transient: yes\n");
+                printf("Transient: yes\n");
 
         if (!strv_isempty(i->dropin_paths)) {
                 _cleanup_free_ char *dir = NULL;
@@ -4287,18 +4278,6 @@ static void print_status_info(
         else
                 printf("\n");
 
-        STRV_FOREACH(t, i->triggered_by) {
-                UnitActiveState state = _UNIT_ACTIVE_STATE_INVALID;
-
-                (void) get_state_one_unit(bus, *t, &state);
-                format_active_state(unit_active_state_to_string(state), &on, &off);
-
-                printf("%s %s%s%s %s\n",
-                       t == i->triggered_by ? "TriggeredBy:" : "            ",
-                       on, special_glyph(SPECIAL_GLYPH_BLACK_CIRCLE), off,
-                       *t);
-        }
-
         if (endswith(i->id, ".timer")) {
                 char tstamp1[FORMAT_TIMESTAMP_RELATIVE_MAX],
                      tstamp2[FORMAT_TIMESTAMP_MAX];
@@ -4320,18 +4299,6 @@ static void print_status_info(
                         printf("n/a\n");
         }
 
-        STRV_FOREACH(t, i->triggers) {
-                UnitActiveState state = _UNIT_ACTIVE_STATE_INVALID;
-
-                (void) get_state_one_unit(bus, *t, &state);
-                format_active_state(unit_active_state_to_string(state), &on, &off);
-
-                printf("%s %s%s%s %s\n",
-                       t == i->triggers ? "   Triggers:" : "            ",
-                       on, special_glyph(SPECIAL_GLYPH_BLACK_CIRCLE), off,
-                       *t);
-        }
-
         if (!i->condition_result && i->condition_timestamp > 0) {
                 UnitCondition *c;
                 int n = 0;
@@ -4339,7 +4306,7 @@ static void print_status_info(
                 s1 = format_timestamp_relative(since1, sizeof(since1), i->condition_timestamp);
                 s2 = format_timestamp(since2, sizeof(since2), i->condition_timestamp);
 
-                printf("  Condition: start %scondition failed%s at %s%s%s\n",
+                printf("Condition: start %scondition failed%s at %s%s%s\n",
                        ansi_highlight_yellow(), ansi_normal(),
                        s2, s1 ? "; " : "", strempty(s1));
 
@@ -5543,8 +5510,6 @@ static int show_one(
                 { "DropInPaths",                    "as",              NULL,           offsetof(UnitStatusInfo, dropin_paths)                      },
                 { "LoadError",                      "(ss)",            map_load_error, offsetof(UnitStatusInfo, load_error)                        },
                 { "Result",                         "s",               NULL,           offsetof(UnitStatusInfo, result)                            },
-                { "TriggeredBy",                    "as",              NULL,           offsetof(UnitStatusInfo, triggered_by)                      },
-                { "Triggers",                       "as",              NULL,           offsetof(UnitStatusInfo, triggers)                          },
                 { "InactiveExitTimestamp",          "t",               NULL,           offsetof(UnitStatusInfo, inactive_exit_timestamp)           },
                 { "InactiveExitTimestampMonotonic", "t",               NULL,           offsetof(UnitStatusInfo, inactive_exit_timestamp_monotonic) },
                 { "ActiveEnterTimestamp",           "t",               NULL,           offsetof(UnitStatusInfo, active_enter_timestamp)            },