https://bugzilla.redhat.com/show_bug.cgi?id=1769843
--- Comment #6 from Michael Catanzaro <mcatanzaro(a)gnome.org> ---
(In reply to Bastien Nocera from comment #5)
It's not really a configuration file, and shouldn't be
modified by users, or
even
administrators.
I notice there is also %{_datadir}/dbus-1/system.d/. Not sure about the history
behind these two locations, but how about we put it there instead? Seems like a
better place?
I would actually want the file to be replaced to automatically use
the new
defaults
set in the package, rather than keep the old ones. So the current tag is
correct for
this file.
That is allowed if you add a comment to the spec file, but it's discouraged:
"""
As a rule of thumb, use %config(noreplace) instead of plain %config unless your
best, educated guess is that doing so will break things. In other words, think
hard before overwriting local changes in configuration files on package
upgrades. An example case when /not/ to use noreplace is when a package’s
configuration file changes so that the new package revision wouldn’t work with
the config file from the previous package revision. Whenever plain %config is
used, add a brief comment to the specfile explaining why.
"""
Basically users won't ever be able to safely edit this file if you replace
their changes on package upgrade. Seems better for low-memory-monitor to assume
default values for anything it doesn't find in its configuration file, which
you probably do already anyway.
But if you really want to replace it despite that guidance, you can add a
comment.
That expands to:
%systemd_requires \
Requires(post): systemd \
Requires(preun): systemd \
Requires(postun): systemd \
%{nil}
Which I think is what we want. Or am I missing something?
Final sentence here:
"""
If package scriptlets call other systemd tools, for example systemd-tmpfiles,
the package SHOULD declare appropriate dependencies. The %systemd_requires
macro is a shortcut to require systemd for the %pre, %post, and %postun
scriptlets. Note that those dependencies are not required for the
%systemd_{post,preun,postun_with_restart,user_post,user_preun} macros listed
above.
"""
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component