github: expressly ask for logs in github issue template

Apparently it's not obvious that logs are a good thing to provide, hence
let's explicitly ask for them.

.github/ISSUE_TEMPLATE/Bug_report.md

@@ -31,3 +31,15 @@ about: A report of an error in a recent systemd version
 
 **Steps to reproduce the problem**
  > …
+
+**Additional program output to the terminal or log subsystem illustrating the issue**
+
+<!-- Please paste relevant program terminal or journal output here, ideally
+     when generated in debug mode (try setting the SYSTEMD_LOG_LEVEL=debug
+     environment variable). For very long copy/pasted data consider using a
+     service like https://gist.github.com/. Where copy/paste is not possible
+     (for example early boot or late shutdown), a photo of the screen might do
+     too, but text is always much preferred. -->
+```text
+…
+```

src/analyze/analyze-security.c

@@ -1545,7 +1545,7 @@ static int assess(const struct security_info *info, Table *overview_table, Analy
 
                 if (a->default_dependencies_only && !info->default_dependencies) {
                         badness = UINT64_MAX;
-                        d = strdup("Service runs in special boot phase, option does not apply");
+                        d = strdup("Service runs in special boot phase, option is not appropriate");
                         if (!d)
                                 return log_oom();
                 } else {

src/shared/clock-util.c

@@ -96,8 +96,7 @@ int clock_is_localtime(const char* adjtime_path) {
         return false;
 }
 
-int clock_set_timezone(int *min) {
+int clock_set_timezone(int *ret_minutesdelta) {
-        const struct timeval *tv_null = NULL;
         struct timespec ts;
         struct tm tm;
         int minutesdelta;
@@ -107,36 +106,32 @@ int clock_set_timezone(int *min) {
         assert_se(localtime_r(&ts.tv_sec, &tm));
         minutesdelta = tm.tm_gmtoff / 60;
 
-        tz.tz_minuteswest = -minutesdelta;
+        tz = (struct timezone) {
-        tz.tz_dsttime = 0; /* DST_NONE */
+                .tz_minuteswest = -minutesdelta,
+                .tz_dsttime = 0, /* DST_NONE */
+        };
 
-        /*
+        /* If the RTC does not run in UTC but in local time, the very first call to settimeofday() will set
-         * If the RTC does not run in UTC but in local time, the very first
+         * the kernel's timezone and will warp the system clock, so that it runs in UTC instead of the local
-         * call to settimeofday() will set the kernel's timezone and will warp the
+         * time we have read from the RTC. */
-         * system clock, so that it runs in UTC instead of the local time we
+        if (settimeofday(NULL, &tz) < 0)
-         * have read from the RTC.
+                return -errno;
-         */
+
-        if (settimeofday(tv_null, &tz) < 0)
+        if (ret_minutesdelta)
-                return negative_errno();
+                *ret_minutesdelta = minutesdelta;
 
-        if (min)
-                *min = minutesdelta;
         return 0;
 }
 
 int clock_reset_timewarp(void) {
-        const struct timeval *tv_null = NULL;
+        static const struct timezone tz = {
-        struct timezone tz;
+                .tz_minuteswest = 0,
+                .tz_dsttime = 0, /* DST_NONE */
+        };
 
-        tz.tz_minuteswest = 0;
+        /* The very first call to settimeofday() does time warp magic. Do a dummy call here, so the time
-        tz.tz_dsttime = 0; /* DST_NONE */
+         * warping is sealed and all later calls behave as expected. */
-
+        if (settimeofday(NULL, &tz) < 0)
-        /*
-         * The very first call to settimeofday() does time warp magic. Do a
-         * dummy call here, so the time warping is sealed and all later calls
-         * behave as expected.
-         */
-        if (settimeofday(tv_null, &tz) < 0)
                 return -errno;
 
         return 0;

src/test/test-string-util.c

@@ -917,7 +917,7 @@ static void test_strverscmp_improved(void) {
                 "123-3.1",
                 "123^patch1",
                 "123^1",
-                "123.a-1"
+                "123.a-1",
                 "123.1-1",
                 "123a-1",
                 "124",