TODO

@@ -1399,9 +1399,8 @@ Features:
   - follow PropertiesChanged state more closely, to deal with quick logouts and
     relogins
   - (optionally?) spawn seat-manager@$SEAT.service whenever a seat shows up that as CanGraphical set
-  - expose details of boot entries on the bus. In particular, it should be possible
-    to query the list of boot entry titles that bootctl / sd-boot would show.
-    Currently we only expose their identifiers.
+
+* move logind udev rules to top-level rule.d/ directory
 
 * move multiseat vid/pid matches from logind udev rule to hwdb
 

docs/RELEASE.md

@@ -11,7 +11,7 @@ SPDX-License-Identifier: LGPL-2.1-or-later
 2. Update the contributors list in NEWS (`ninja -C build git-contrib`)
 3. Update the time and place in NEWS
 4. Update hwdb (`ninja -C build update-hwdb`, `ninja -C build update-hwdb-autosuspend`, commit separately).
-5. Update syscall numbers (`ninja -C build update-syscall-tables update-syscall-header`).
+5. Update syscall numbers (`ninja -C build update-syscall-tables update-syscall-headers`).
 6. [RC1] Update version and library numbers in `meson.build`
 7. Check dbus docs with `ninja -C build update-dbus-docs`
 8. Tag the release: `version=vXXX-rcY && git tag -s "${version}" -m "systemd ${version}"`

man/org.freedesktop.home1.xml

@@ -514,7 +514,12 @@ node /org/freedesktop/home1/home {
     </refsect2>
   </refsect1>
 
-  <xi:include href="org.freedesktop.locale1.xml" xpointer="versioning"/>
+  <refsect1>
+    <title>Versioning</title>
+
+    <para>These D-Bus interfaces follow <ulink url="http://0pointer.de/blog/projects/versioning-dbus.html">
+    the usual interface versioning guidelines</ulink>.</para>
+  </refsect1>
 
   <refsect1>
     <title>See Also</title>

man/org.freedesktop.hostname1.xml

@@ -379,7 +379,12 @@ node /org/freedesktop/hostname1 {
     name.</para>
   </refsect1>
 
-  <xi:include href="org.freedesktop.locale1.xml" xpointer="versioning"/>
+  <refsect1>
+    <title>Versioning</title>
+
+    <para>These D-Bus interfaces follow <ulink url="http://0pointer.de/blog/projects/versioning-dbus.html">
+    the usual interface versioning guidelines</ulink>.</para>
+  </refsect1>
 
   <refsect1>
     <title>Examples</title>

man/org.freedesktop.import1.xml

@@ -339,5 +339,10 @@ node /org/freedesktop/import1/transfer/_1 {
     </example>
   </refsect1>
 
-  <xi:include href="org.freedesktop.locale1.xml" xpointer="versioning"/>
+  <refsect1>
+    <title>Versioning</title>
+
+    <para>These D-Bus interfaces follow <ulink url="http://0pointer.de/blog/projects/versioning-dbus.html">
+    the usual interface versioning guidelines</ulink>.</para>
+  </refsect1>
 </refentry>

man/org.freedesktop.locale1.xml

@@ -179,7 +179,7 @@ $ gdbus introspect --system \
     </example>
   </refsect1>
 
-  <refsect1 id="versioning">
+  <refsect1>
     <title>Versioning</title>
 
     <para>These D-Bus interfaces follow <ulink url="http://0pointer.de/blog/projects/versioning-dbus.html">

man/org.freedesktop.login1.xml

@@ -1415,46 +1415,27 @@ node /org/freedesktop/login1/session/1 {
     <title>Examples</title>
 
     <example>
-      <title>Introspect the logind manager on the bus</title>
+      <title>Introspect <interfacename>org.freedesktop.login1.Manager</interfacename> on the bus</title>
 
       <programlisting>$ gdbus introspect --system --dest org.freedesktop.login1 \
   --object-path /org/freedesktop/login1
       </programlisting>
-
-      <para>or</para>
-
-      <programlisting>$ busctl introspect org.freedesktop.login1 /org/freedesktop/login1
-      </programlisting>
     </example>
 
     <example>
-      <title>Introspect the default seat on the bus</title>
+      <title>Introspect <interfacename>org.freedesktop.login1.Seat</interfacename> on the bus</title>
 
       <programlisting>$ gdbus introspect --system --dest org.freedesktop.login1 \
  --object-path /org/freedesktop/login1/seat/seat0
       </programlisting>
-
-      <para>or</para>
-
-      <programlisting>$ busctl introspect org.freedesktop.login1 /org/freedesktop/login1/seat/seat0
-      </programlisting>
-
-      <para>Seat <literal>seat0</literal> is the default seat, so it'll be present unless local configuation
-      is made to reassign all devices to a different seat. The list of seats and users can be acquired with
-      <command>loginctl list-sessions</command>.</para>
     </example>
 
     <example>
-      <title>Introspect a single user on the bus</title>
+      <title>Introspect <interfacename>org.freedesktop.login1.User</interfacename> on the bus</title>
 
       <programlisting>$ gdbus introspect --system --dest org.freedesktop.login1 \
   --object-path /org/freedesktop/login1/user/_1000
       </programlisting>
-
-      <para>or</para>
-
-      <programlisting>$ busctl introspect org.freedesktop.login1 /org/freedesktop/login1/user/_1000
-      </programlisting>
     </example>
 
     <example>
@@ -1463,13 +1444,13 @@ node /org/freedesktop/login1/session/1 {
       <programlisting>$ gdbus introspect --system --dest org.freedesktop.login1 \
   --object-path /org/freedesktop/login1/session/45
       </programlisting>
-
-      <para>or</para>
-
-      <programlisting>$ busctl introspect org.freedesktop.login1 /org/freedesktop/login1/session/45
-      </programlisting>
     </example>
   </refsect1>
 
-  <xi:include href="org.freedesktop.locale1.xml" xpointer="versioning"/>
+  <refsect1>
+    <title>Versioning</title>
+
+    <para>These D-Bus interfaces follow <ulink url="http://0pointer.de/blog/projects/versioning-dbus.html">
+    the usual interface versioning guidelines</ulink>.</para>
+  </refsect1>
 </refentry>

man/org.freedesktop.machine1.xml

@@ -639,5 +639,10 @@ $ gdbus introspect --system \
     </example>
   </refsect1>
 
-  <xi:include href="org.freedesktop.locale1.xml" xpointer="versioning"/>
+  <refsect1>
+    <title>Versioning</title>
+
+    <para>These D-Bus interfaces follow <ulink url="http://0pointer.de/blog/projects/versioning-dbus.html">
+    the usual interface versioning guidelines</ulink>.</para>
+  </refsect1>
 </refentry>

man/org.freedesktop.network1.xml

@@ -498,5 +498,10 @@ $ gdbus introspect --system \
     </example>
   </refsect1>
 
-  <xi:include href="org.freedesktop.locale1.xml" xpointer="versioning"/>
+  <refsect1>
+    <title>Versioning</title>
+
+    <para>These D-Bus interfaces follow <ulink url="http://0pointer.de/blog/projects/versioning-dbus.html">
+    the usual interface versioning guidelines</ulink>.</para>
+  </refsect1>
 </refentry>

man/org.freedesktop.oom1.xml

@@ -65,5 +65,10 @@ node /org/freedesktop/oom1 {
     </refsect2>
   </refsect1>
 
-  <xi:include href="org.freedesktop.locale1.xml" xpointer="versioning"/>
+  <refsect1>
+    <title>Versioning</title>
+
+    <para>These D-Bus interfaces follow <ulink url="http://0pointer.de/blog/projects/versioning-dbus.html">
+    the usual interface versioning guidelines</ulink>.</para>
+  </refsect1>
 </refentry>

man/org.freedesktop.portable1.xml

@@ -559,5 +559,10 @@ node /org/freedesktop/portable1 {
     </refsect2>
   </refsect1>
 
-  <xi:include href="org.freedesktop.locale1.xml" xpointer="versioning"/>
+  <refsect1>
+    <title>Versioning</title>
+
+    <para>These D-Bus interfaces follow <ulink url="http://0pointer.de/blog/projects/versioning-dbus.html">
+    the usual interface versioning guidelines</ulink>.</para>
+  </refsect1>
 </refentry>

man/org.freedesktop.resolve1.xml

@@ -891,5 +891,10 @@ $ gdbus introspect --system \
     </example>
   </refsect1>
 
-  <xi:include href="org.freedesktop.locale1.xml" xpointer="versioning"/>
+  <refsect1>
+    <title>Versioning</title>
+
+    <para>These D-Bus interfaces follow <ulink url="http://0pointer.de/blog/projects/versioning-dbus.html">
+    the usual interface versioning guidelines</ulink>.</para>
+  </refsect1>
 </refentry>

man/org.freedesktop.systemd1.xml

@@ -10656,5 +10656,10 @@ $ gdbus introspect --system --dest org.freedesktop.systemd1 \
     </example>
   </refsect1>
 
-  <xi:include href="org.freedesktop.locale1.xml" xpointer="versioning"/>
+  <refsect1>
+    <title>Versioning</title>
+
+    <para>These D-Bus interfaces follow <ulink url="http://0pointer.de/blog/projects/versioning-dbus.html">
+    the usual interface versioning guidelines</ulink>.</para>
+  </refsect1>
 </refentry>

man/org.freedesktop.timedate1.xml

@@ -190,7 +190,12 @@ $ gdbus introspect --system \
     </example>
   </refsect1>
 
-  <xi:include href="org.freedesktop.locale1.xml" xpointer="versioning"/>
+  <refsect1>
+    <title>Versioning</title>
+
+    <para>These D-Bus interfaces follow <ulink url="http://0pointer.de/blog/projects/versioning-dbus.html">
+    the usual interface versioning guidelines</ulink>.</para>
+  </refsect1>
 
   <refsect1>
     <title>See also</title>

meson.build

@@ -2068,6 +2068,7 @@ subdir('src/timedate')
 subdir('src/timesync')
 subdir('src/tmpfiles')
 subdir('src/userdb')
+subdir('src/vconsole')
 subdir('src/xdg-autostart-generator')
 
 subdir('src/systemd')

rules.d/meson.build

@@ -4,8 +4,8 @@ install_data(
         'README',
         install_dir : udevrulesdir)
 
-rules = [
-        [files('60-autosuspend.rules',
+rules = files(
+        '60-autosuspend.rules',
         '60-block.rules',
         '60-cdrom_id.rules',
         '60-drm.rules',
@@ -27,50 +27,31 @@ rules = [
         '75-probe_mtd.rules',
         '78-sound-card.rules',
         '80-net-setup-link.rules',
-               '81-net-dhcp.rules',
-              )],
+        '81-net-dhcp.rules')
 
-        [files('80-drivers.rules'),
-         conf.get('HAVE_KMOD') == 1],
-
-        [files('70-memory.rules'),
-         dmi_arches.contains(host_machine.cpu_family())],
-]
-
-all_rules = []
-
-foreach tuple : rules
-        if tuple.length() == 1 or tuple[1]
-                install_data(tuple[0],
-                             install_dir : udevrulesdir)
-                all_rules += tuple[0]
+if conf.get('HAVE_KMOD') == 1
+        rules += files('80-drivers.rules')
 endif
-endforeach
 
-rules_in = [
-        ['50-udev-default.rules'],
-        ['64-btrfs.rules'],
-        ['99-systemd.rules'],
+if dmi_arches.contains(host_machine.cpu_family())
+        rules += files('70-memory.rules')
+endif
 
-        ['70-uaccess.rules',   enable_logind and conf.get('HAVE_ACL') == 1],
-        ['71-seat.rules',      enable_logind],
-        ['73-seat-late.rules', enable_logind],
+install_data(rules,
+             install_dir : udevrulesdir)
 
-        ['90-vconsole.rules',  conf.get('ENABLE_VCONSOLE') == 1],
-]
+all_rules = rules
 
-foreach tuple : rules_in
-        want = tuple.length() == 1 or tuple[1]
+rules_in = ['50-udev-default.rules',
+            '64-btrfs.rules',
+            '99-systemd.rules']
 
-        rule = custom_target(
-                tuple[0],
-                input : tuple[0] + '.in',
-                output: tuple[0],
+foreach file : rules_in
+        all_rules += custom_target(
+                file,
+                input : file + '.in',
+                output: file,
                 command : [jinja2_cmdline, '@INPUT@', '@OUTPUT@'],
-                install : want,
+                install : true,
                 install_dir : udevrulesdir)
-
-        if want
-                all_rules += rule
-        endif
 endforeach

src/basic/strv.h

@@ -77,7 +77,7 @@ int strv_split_full(char ***t, const char *s, const char *separators, ExtractFla
 static inline char** strv_split(const char *s, const char *separators) {
         char **ret;
 
-        if (strv_split_full(&ret, s, separators, EXTRACT_RETAIN_ESCAPE) < 0)
+        if (strv_split_full(&ret, s, separators, 0) < 0)
                 return NULL;
 
         return ret;

src/basic/terminal-util.c

@@ -1278,7 +1278,12 @@ ColorMode get_color_mode(void) {
                         /* We only check for the presence of the variable; value is ignored. */
                         cached_color_mode = COLOR_OFF;
 
-                else if (getpid_cached() == 1) {
+                else if (STRPTR_IN_SET(getenv("COLORTERM"),
+                                       "truecolor",
+                                       "24bit"))
+                        cached_color_mode = COLOR_24BIT;
+
+                else if (getpid_cached() == 1)
                         /* PID1 outputs to the console without holding it open all the time.
                          *
                          * Note that the Linux console can only display 16 colors. We still enable 256 color
@@ -1287,23 +1292,9 @@ ColorMode get_color_mode(void) {
                          * map them to the closest color in the 16 color palette (since kernel 3.16). Doing
                          * 256 colors is nice for people who invoke systemd in a container or via a serial
                          * link or such, and use a true 256 color terminal to do so. */
-                        if (getenv_terminal_is_dumb())
-                                cached_color_mode = COLOR_OFF;
-                } else {
-                        if (terminal_is_dumb())
-                                cached_color_mode = COLOR_OFF;
-                }
-
-                if (cached_color_mode < 0) {
-                        /* We failed to figure out any reason to *disable* colors.
-                         * Let's see how many colors we shall use. */
-                        if (STRPTR_IN_SET(getenv("COLORTERM"),
-                                          "truecolor",
-                                          "24bit"))
-                                cached_color_mode = COLOR_24BIT;
+                        cached_color_mode = getenv_terminal_is_dumb() ? COLOR_OFF : COLOR_256;
                 else
-                                cached_color_mode = COLOR_256;
-                }
+                        cached_color_mode = terminal_is_dumb() ? COLOR_OFF : COLOR_256;
         }
 
         return cached_color_mode;

src/login/meson.build

@@ -65,22 +65,25 @@ pam_systemd_sym = 'src/login/pam_systemd.sym'
 pam_systemd_c = files('pam_systemd.c')
 
 enable_logind = conf.get('ENABLE_LOGIND') == 1
+in_files = [
+        ['logind.conf',        pkgsysconfdir, enable_logind and install_sysconfdir_samples],
+        ['70-uaccess.rules',   udevrulesdir,  enable_logind and conf.get('HAVE_ACL') == 1],
+        ['71-seat.rules',      udevrulesdir,  enable_logind],
+        ['73-seat-late.rules', udevrulesdir,  enable_logind],
+        ['systemd-user',       pamconfdir,    enable_logind]]
 
+foreach tuple : in_files
+        file = tuple[0]
+        dir = tuple[1]
+        install = (dir == pkgsysconfdir) ? install_sysconfdir_samples : (dir != 'no')
         custom_target(
-        'logind.conf',
-        input : 'logind.conf.in',
-        output : 'logind.conf',
+                file,
+                input : file + '.in',
+                output: file,
                 command : [jinja2_cmdline, '@INPUT@', '@OUTPUT@'],
-        install : enable_logind and install_sysconfdir_samples and pkgsysconfdir != 'no',
-        install_dir : pkgsysconfdir)
-
-custom_target(
-        'systemd-user',
-        input : 'systemd-user.in',
-        output : 'systemd-user',
-        command : [jinja2_cmdline, '@INPUT@', '@OUTPUT@'],
-        install : enable_logind and pamconfdir != 'no',
-        install_dir : pamconfdir)
+                install : tuple[2] and install,
+                install_dir : dir)
+endforeach
 
 if enable_logind
         install_data('org.freedesktop.login1.conf',
@@ -89,6 +92,8 @@ if enable_logind
                      install_dir : dbussystemservicedir)
         install_data('org.freedesktop.login1.policy',
                      install_dir : polkitpolicydir)
+        install_data('70-power-switch.rules',
+                     install_dir : udevrulesdir)
 endif
 
 ############################################################

src/test/test-strv.c

@@ -317,21 +317,6 @@ TEST(strv_split) {
 
         assert_se(strv_split_full(&l, "\\", NULL, EXTRACT_UNQUOTE | EXTRACT_RELAX | EXTRACT_UNESCAPE_RELAX) == 1);
         assert_se(strv_equal(l, STRV_MAKE("\\")));
-
-        l = strv_free_erase(l);
-
-        assert_se(l = strv_split("\\", NULL));
-        assert_se(strv_equal(l, STRV_MAKE("\\")));
-
-        l = strv_free_erase(l);
-
-        assert_se(l = strv_split("aa\\ bb\\", NULL));
-        assert_se(strv_equal(l, STRV_MAKE("aa\\", "bb\\")));
-
-        l = strv_free_erase(l);
-
-        assert_se(l = strv_split("aa\" bb'", NULL));
-        assert_se(strv_equal(l, STRV_MAKE("aa\"", "bb'")));
 }
 
 TEST(strv_split_empty) {

src/udev/udev-rules.c

@@ -2425,15 +2425,11 @@ static int udev_rule_apply_parent_token_to_event(
         UdevRuleToken *head;
         int r;
 
-        assert(rules);
-        assert(rules->current_file);
-        assert(event);
-
-        line = ASSERT_PTR(rules->current_file->current_line);
-        head = ASSERT_PTR(rules->current_file->current_line->current_token);
-        event->dev_parent = ASSERT_PTR(event->dev);
-
+        line = rules->current_file->current_line;
+        head = rules->current_file->current_line->current_token;
+        event->dev_parent = event->dev;
         for (;;) {
+                line->current_token = NULL;
                 LIST_FOREACH(tokens, token, head) {
                         if (!token_is_for_parents(token))
                                 return true; /* All parent tokens match. */
@@ -2445,8 +2441,8 @@ static int udev_rule_apply_parent_token_to_event(
                         if (r == 0)
                                 break;
                 }
-                if (r > 0)
-                        /* All parent tokens match, and no more token (except for GOTO) in the line. */
+                if (!line->current_token)
+                        /* All parent tokens match. But no assign tokens in the line. Hmm... */
                         return true;
 
                 if (sd_device_get_parent(event->dev_parent, &event->dev_parent) < 0) {

src/vconsole/meson.build

@@ -0,0 +1,9 @@
+# SPDX-License-Identifier: LGPL-2.1-or-later
+
+custom_target(
+        '90-vconsole.rules',
+        input : '90-vconsole.rules.in',
+        output : '90-vconsole.rules',
+        command : [jinja2_cmdline, '@INPUT@', '@OUTPUT@'],
+        install : conf.get('ENABLE_VCONSOLE') == 1,
+        install_dir : udevrulesdir)

test/udev-test.pl

@@ -176,7 +176,7 @@ EOF
                         {
                                 devpath         => "/devices/pci0000:00/0000:00:1f.2/host0/target0:0:0/0:0:0:0/block/sda/sda1",
                                 exp_links       => ["boot_disk1", "boot_disk1-4", "boot_disk1-5"],
-                                not_exp_links   => ["boot_disk1-1", "boot_disk1-2", "boot_disk1-3", "boot_disk1-6", "boot_disk1-7"]
+                                not_exp_links   => ["boot_disk1-1", "boot_disk1-2", "boot_disk1-3"]
                         }],
                 rules           => <<EOF
 SUBSYSTEMS=="scsi", ATTRS{vendor}=="?ATA", SYMLINK+="boot_disk%n-1"
@@ -185,12 +185,6 @@ SUBSYSTEMS=="scsi", ATTRS{vendor}=="A??", SYMLINK+="boot_disk%n"
 SUBSYSTEMS=="scsi", ATTRS{vendor}=="ATAS", SYMLINK+="boot_disk%n-3"
 SUBSYSTEMS=="scsi", ATTRS{vendor}=="AT?", SYMLINK+="boot_disk%n-4"
 SUBSYSTEMS=="scsi", ATTRS{vendor}=="??A", SYMLINK+="boot_disk%n-5"
-SUBSYSTEMS=="scsi", ATTRS{vendor}=="ATA", GOTO="skip-6"
-SUBSYSTEMS=="scsi", ATTRS{vendor}=="ATA", SYMLINK+="boot_disk%n-6"
-LABEL="skip-6"
-SUBSYSTEMS=="scsi", GOTO="skip-7"
-SUBSYSTEMS=="scsi", ATTRS{vendor}=="ATA", SYMLINK+="boot_disk%n-7"
-LABEL="skip-7"
 EOF
         },
         {