Hello community, here is the log from the commit of package systemd for openSUSE:Factory checked in at 2016-05-17 17:04:33 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/systemd (Old) and /work/SRC/openSUSE:Factory/.systemd.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "systemd" Changes: -------- --- /work/SRC/openSUSE:Factory/systemd/systemd-mini.changes 2016-04-14 13:00:38.000000000 +0200 +++ /work/SRC/openSUSE:Factory/.systemd.new/systemd-mini.changes 2016-05-17 17:04:34.000000000 +0200 @@ -1,0 +2,6 @@ +Wed May 4 06:37:51 UTC 2016 - [email protected] + +- fix warning about missing install info during preset (boo#970293) + 0001-systemctl-pid1-do-not-warn-about-missing-install-inf.patch + +------------------------------------------------------------------- systemd.changes: same change New: ---- 0001-systemctl-pid1-do-not-warn-about-missing-install-inf.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ systemd-mini.spec ++++++ --- /var/tmp/diff_new_pack.sJfWEZ/_old 2016-05-17 17:04:39.000000000 +0200 +++ /var/tmp/diff_new_pack.sJfWEZ/_new 2016-05-17 17:04:39.000000000 +0200 @@ -259,6 +259,8 @@ Patch527: 0001-core-fix-bus-name-synchronization-after-daemon-reloa.patch # PATCH-FIX-UPSTREAM -- fixed after 228 Patch528: 0001-core-re-sync-bus-name-list-after-deserializing-durin.patch +# PATCH-FIX-UPSTREAM -- fixed after 228 +Patch529: 0001-systemctl-pid1-do-not-warn-about-missing-install-inf.patch # UDEV PATCHES # ============ @@ -606,6 +608,7 @@ %patch526 -p1 %patch527 -p1 %patch528 -p1 +%patch529 -p1 # udev patches %patch1002 -p1 systemd.spec: same change ++++++ 0001-systemctl-pid1-do-not-warn-about-missing-install-inf.patch ++++++ >From b8d6cc0513adadea9b5048fa320d7f49c10f004b Mon Sep 17 00:00:00 2001 From: Thomas Blume <[email protected]> Date: Wed, 4 May 2016 08:51:52 +0200 Subject: [PATCH] systemctl,pid1: do not warn about missing install info with "preset" When "preset" was executed for a unit without install info, we'd warn similarly as for "enable" and "disable". But "preset" is usually called for all units, because the preset files are provided by the distribution, and the units are under control of individual programs, and it's reasonable to call "preset" for all units rather then try to do it only for the ones that can be installed. We also don't warn about missing info for "preset-all". Thus it seems reasonable to silently ignore units w/o install info when presetting. (In addition, when more than one unit was specified, we'd issue the warning only if none of them had install info. But this is probably something to fix for enable/disable too.) --- man/systemctl.xml | 26 +++++++++++++------------- src/systemctl/systemctl.c | 10 +++++----- 2 files changed, 18 insertions(+), 18 deletions(-) diff --git a/man/systemctl.xml b/man/systemctl.xml index f342e26..1c2c919 100644 --- a/man/systemctl.xml +++ b/man/systemctl.xml @@ -1082,22 +1082,22 @@ kobject-uevent 1 systemd-udevd-kernel.socket systemd-udevd.service <term><command>preset <replaceable>NAME</replaceable>...</command></term> <listitem> - <para>Reset one or more unit files, as specified on the - command line, to the defaults configured in the preset - policy files. This has the same effect as - <command>disable</command> or <command>enable</command>, - depending how the unit is listed in the preset files.</para> + <para>Reset the enable/disable status one or more unit files, as specified on + the command line, to the defaults configured in the preset policy files. This + has the same effect as <command>disable</command> or + <command>enable</command>, depending how the unit is listed in the preset + files.</para> - <para>Use <option>--preset-mode=</option> to control - whether units shall be enabled and disabled, or only - enabled, or only disabled.</para> + <para>Use <option>--preset-mode=</option> to control whether units shall be + enabled and disabled, or only enabled, or only disabled.</para> + + <para>If the unit carries no install information, it will be silently ignored + by this command.</para> - <para>For more information on the preset policy format, - see + <para>For more information on the preset policy format, see <citerefentry><refentrytitle>systemd.preset</refentrytitle><manvolnum>5</manvolnum></citerefentry>. - For more information on the concept of presets, please - consult the <ulink - url="http://freedesktop.org/wiki/Software/systemd/Preset";>Preset</ulink> + For more information on the concept of presets, please consult the + <ulink url="http://freedesktop.org/wiki/Software/systemd/Preset";>Preset</ulink> document.</para> </listitem> </varlistentry> diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c index 1f42d1a..9dc5971 100644 --- a/src/systemctl/systemctl.c +++ b/src/systemctl/systemctl.c @@ -5399,6 +5399,7 @@ static int enable_unit(int argc, char *argv[], void *userdata) { UnitFileChange *changes = NULL; unsigned n_changes = 0; int carries_install_info = -1; + bool ignore_carries_install_info = false; int r; if (!argv[1]) @@ -5412,8 +5413,7 @@ static int enable_unit(int argc, char *argv[], void *userdata) { if (r < 0) return r; - /* If the operation was fully executed by the SysV compat, - * let's finish early */ + /* If the operation was fully executed by the SysV compat, let's finish early */ if (strv_isempty(names)) return 0; @@ -5430,7 +5430,6 @@ static int enable_unit(int argc, char *argv[], void *userdata) { r = unit_file_link(arg_scope, arg_runtime, arg_root, names, arg_force, &changes, &n_changes); else if (streq(verb, "preset")) { r = unit_file_preset(arg_scope, arg_runtime, arg_root, names, arg_preset_mode, arg_force, &changes, &n_changes); - carries_install_info = r; } else if (streq(verb, "mask")) r = unit_file_mask(arg_scope, arg_runtime, arg_root, names, arg_force, &changes, &n_changes); else if (streq(verb, "unmask")) @@ -5450,7 +5449,7 @@ static int enable_unit(int argc, char *argv[], void *userdata) { } else { _cleanup_bus_message_unref_ sd_bus_message *reply = NULL, *m = NULL; _cleanup_bus_error_free_ sd_bus_error error = SD_BUS_ERROR_NULL; - int expect_carries_install_info = false; + bool expect_carries_install_info = false; bool send_force = true, send_preset_mode = false; const char *method; sd_bus *bus; @@ -5481,6 +5480,7 @@ static int enable_unit(int argc, char *argv[], void *userdata) { method = "PresetUnitFiles"; expect_carries_install_info = true; + ignore_carries_install_info = true; } else if (streq(verb, "mask")) method = "MaskUnitFiles"; else if (streq(verb, "unmask")) { @@ -5540,7 +5540,7 @@ static int enable_unit(int argc, char *argv[], void *userdata) { r = 0; } - if (carries_install_info == 0) + if (carries_install_info == 0 && !ignore_carries_install_info) log_warning("The unit files have no [Install] section. They are not meant to be enabled\n" "using systemctl.\n" "Possible reasons for having this kind of units are:\n" -- 2.6.6
