shared: add format helpers for printing MAC addresses

Use these in networctl.

update TODO

rc-local: drop SysVStartPriority= field which is now obsolete

bus-proxy: don't log bus policy every single time we run

busctl: fix 'command line' style output of properties

systemctl: show unit file preset state in "systemctl status" output"

manager: log deserialization errors only at LOG_DEBUG level

During upgrades and when transitioning between different systemd
versions in initrd and on the host we have to expect that some
serialization fields are unknown or parse incorrectly. This shouldn't
really be considered an error, hence downgrade the log messages about
it to debug. This way we can still trace it, but it doesn't confuse
users.

This kinda reverts 46849c3f.

update TODO

core: OOM really shouldn't be considered a deserialization parse failure

networkctl: print the Gateway in the status output

This is the IP address of the default route on the link, if present. A
description is printed when available (the manufacturer of the gateway NIC based
on its MAC address).

In the future we should prefer LLDP information over MAC info.

sd-rtnl: route - allow GETROUTE with AF_UNSPEC

shared: udev-util - add hwdb cleanup macro

sd-rtnl: add sd_rtnl_message_{new_neigh,neigh_get_{family,ifindex}}

sd-rtnl: add typesystem for RTM_*NEIGH

sd-rtnl: add a bit more debugging in case a message is dropped

sd-rtnl: message - allow checking for attributes without reading out their contents

sd-rtnl: add sd_rtnl_message_route_get_family

build-sys: configure the list of system users, files and directories

Choose which system users defined in sysusers.d/systemd.conf and files
or directories in tmpfiles.d/systemd.conf, should be provided depending
on comile-time configuration.

man: BusName= is not optional for Type!=bus

http://lists.freedesktop.org/archives/systemd-devel/2014-November/025492.html

man: delete ZX as sole compression; "Compress=" as boolean

In man journald.conf, removes reference to XZ as sole form of
compression. See commit d89c8fdf48c7bad5816b9f2e77e8361721f22517.

In man coredump.conf, clarifies that "Compression=" controls existence,
not type, of compression.

resolved: make TXT RR generation and parsing more in-line with RFC 6763, section 6.1

The RFC says to encode an single empty TXT string instead of an empty
TXT array. It also says to treat a zero-length TXT RR as a TXT array
with a single zero-length string.

update TODO

core: warn and ignore SysVStartPriority=

Option was being parsed but not used for anything.

When warning about unsupported options, be more detailed

update TODO

cgroup-util: Don't send SIGCONT after SIGKILL

timesyncd: minor simplification

journald: close passed fds we cannot make sense of

This is mostly likely the audit socket, and we really should close it
if we cannot make sense of it, since as long as it is open the kernel
might disable the kmsg forwarding of audit msgs, and we should avoid
that, since audit msgs might get completely lost then.

I also downgraded the log message we show a bit, after all things should
really work fine, and we proceed fine with it.

man: document that we set both soft and hard limits for Limit directives

See
http://cgit.freedesktop.org/systemd/systemd/tree/src/core/load-fragment.c#n1100

man: fix typos

man: document equivalence between Limit directives and ulimit

See https://bugs.freedesktop.org/show_bug.cgi?id=80341

man: fix link to systemd-networkd-wait-online.service in systemd-networkd.service(8)

man: add a link to systemd-coredump(8) in Description of coredump.conf(5)

terminal/screen: fix GL/GR double-mapping

We must not call screen_map() multiple times for the same input char. Fix
the double-mapping for GL/GR maps are correctly applied.

delta: Fix broken separator support

systemd-delta man page promises that multiple types of deltas will be
concatenated if they are listed with a comma as separator.  Replace
FOREACH_WORD() with FOREACH_WORD_SEPARATOR() to restore the functionality.

build-sys: add missing tests files to EXTRA_DIST

core: remove unused variables

network: remove unused variable

tests: use assert_se instead of assert

Otherwise they can be optimized away with -DNDEBUG

resolve: remove unused variable

systemctl: remove unused variable

NEWS: Document new .conf.d configuration directories

Also provide guidance to distributions, to make sure they don't start
dropping files in the configuration directories in /etc/.

udev: strings in C are NUL-terminated anyway, no need to add a second NUL...

journald-remote,journal-upload: Support .d directories in the usual search paths

timesyncd: Support timesyncd.conf.d directories in the usual search paths

resolved: Support resolved.conf.d directories in the usual search paths

coredump: Support coredump.conf.d directories in the usual search paths

bootchart: Support bootchart.conf.d directories in the usual search paths

systemd-sleep: Support sleep.conf.d directories in the usual search paths

journald: Support journald.conf.d directories in the usual search paths

core: Support system.conf.d and user.conf.d directories in the usual search paths

man: Factor out a common snippet for .d directories and precedence

Several manpages contain duplicate text describing a standard set of .d
configuration directories, with the usual sorting, precedence,
overrides, and so on.  Factor this common text out using XInclude before
proliferating it even further.

man/logind.conf.xml: Make "man logind.conf.d" work too

systemctl: add edit verb

It helps editing units by either creating a drop-in file, like
/etc/systemd/system/my.service.d/override.conf, or by copying the
original unit from /usr/lib/systemd/ to /etc/systemd/ if the --full
option is specified.

It invokes an editor on temporary files related to the unit files and
if the editor exited successfully, then it renames the temporary files
to their original names (e.g. my.service or override.conf) and
daemon-reload is invoked.

If the temporary file is empty the modification is canceled.

See https://bugzilla.redhat.com/show_bug.cgi?id=906824

man: update description of default extension for isolate

nspawn: fix unused variable warning

delta: diff returns 1 when files differ, ignore this

https://bugs.debian/org/771397

shared: install - report error if mask symlink can not be marked for removal

shared: time-dst - ensure nulstr is null terminated

Fixes CID #1237772.

README: notice kernel config for CPUQuota

mount: use DEFINE_TRIVIAL_CLEANUP_FUNC

udev: rules - ignore the lack of trailing newline

Also accept '\r' as newline character.

This dropps warnings of the type:

  invalid key/value pair in file /usr/lib/udev/rules.d/40-usb-media-players.rules
  on line 26, starting at character 25 ('')

udev: rules - print the first invalid character

The current code would print the character following the first invalid
character.

Given an udev rules-file without a trailing newline we would otherwise print
garbage:

  invalid key/value pair in file /usr/lib/udev/rules.d/40-usb-media-players.rules
  on line 26, starting at character 25 ('m')

This is now changed to print

  invalid key/value pair in file /usr/lib/udev/rules.d/40-usb-media-players.rules
  on line 26, starting at character 25 ('')

(still not very good as printing \0 just gives the empty string)

udev: rules - modernise add_rule a bit

sd-bus: move packet header definition to bus-protocol.h

treewide: convert some left-over (usec_t) -1 to USEC_INFINITY

treewide: introduce UID_INVALID (and friends) as macro for (uid_t) -1

udevadm hwdb: discard extra leading whitespaces in hwdb

Currently a property in the form of
  FOO=bar
is stored as " FOO=bar", i.e. the property name contains a leading space.
That's quite hard to spot.

This patch discards all extra whitespaces but the first one which is required
by libudev's hwdb_add_property.

[zj: modify the check a bit]

https://bugs.freedesktop.org/show_bug.cgi?id=82311

mount: constify MountParameters

mount: deal with inotify queue overflow

Overflow is very unlikely, since we are watching a privileged directory,
but could be triggered if thousands of mounts are suddently executed.

mount: create directory before adding watches on it

mount: be more careful about errors when parsing mtab

Fixup for 4a3a9ef610.

mount: simplify mount_needs_network check

mount: check options as well as fstype for network mounts

When creating a new mount unit after an event on /proc/self/mountinfo,
check the mount options as well as the fstype to determine if this is a
remote mount that requires network access.

mount: add remote-fs dependencies if needed after change

This is an attempt to add it the remote-fs dependencies to a mount unit
if the options change, like when the utab options are picked up after
mountinfo has already been processed.  It just adds the remote-fs
dependencies, leaving the local-fs ones in place.

With this change I always get mount units with proper remote-fs
dependencies when mounted with the _netdev option.

mount: monitor for utab changes with inotify

Parsing the mount table with libmount races against the mount command,
which will handle the actual mounting before updating utab.  This means
the poll event on /proc/self/mountinfo can kick of a reparse in systemd
before the utab information is available.

This change adds in an additional event source using inotify to watch
for changes to utab.  It only watches for IN_MOVED_TO events, matching
libmount behavior of always overwriting this file using rename(2).

This does add a second pass through the mount table parsing when utab is
updated.

mount: use libmount to enumerate /proc/self/mountinfo

This lets libmount add in user options from /run/mount/utab, like
_netdev which is needed to get proper ordering against remote-fs.target

coredump: simplify a few things by allocating small fields on the stack rather than heap

coredump: rework compose_open_fds()

Use FOREACH_DIRENT() and FOREACH_LINE() macros instead of manual loops.

Don't clobber return parameters on failure.

Simplify some other things.

update TODO

sd-bus: add support for matches against arrays of strings in messages

sd-bus: if a NULL signatures is passed to sd_bus_message_skip(), make it skip a single element of any type

sd-bus: add arrays of strings in the bloom filter

Let's do this right from the beginning, to prepare ground for udev
messages that most likely want to store list of strings (for device
tags) in messages, and filter on them.

update TODO

sd-bus: make more connection properties readable

sd-bus: add new sd_bus_get_address() for querying the current bus address

Also, update "busctl" to show this in its output.

sd-bus: rename sd_bus_get_owner_id() → sd_bus_get_bus_id()

The ID returned really doesn't identify the owner, but the bus instance,
hence fix this misnaming.

Also, update "busctl status" to show the ID in its output.

treewide: another round of simplifications

Using the same scripts as in f647962d64e "treewide: yet more log_*_errno
+ return simplifications".

treewide: use log_*_errno whenever %m is in the format string

If the format string contains %m, clearly errno must have a meaningful
value, so we might as well use log_*_errno to have ERRNO= logged.

Using:
find . -name '*.[ch]' | xargs sed -r -i -e \
's/log_(debug|info|notice|warning|error|emergency)\((".*%m.*")/log_\1_errno(errno, \2/'

Plus some whitespace, linewrap, and indent adjustments.

core: fix return value in error path after sd_event_add_io() failure

sd_event_add_io() does not set errno, it returns negative errno.

Noticed during log_*_errno conversions.

treewide: a few more log_*_errno + return simplifications

The one in tmpfiles.c:create_item() even looks like it fixes a bug.

treewide: yet more log_*_errno + return simplifications

Using:
find . -name '*.[ch]' | while read f; do perl -i.mmm -e \
 'local $/;
  local $_=<>;
  s/(if\s*\([^\n]+\))\s*{\n(\s*)(log_[a-z_]*_errno\(\s*([->a-zA-Z_]+)\s*,[^;]+);\s*return\s+\g4;\s+}/\1\n\2return \3;/msg;
  print;'
 $f
done

And a couple of manual whitespace fixups.

Revert "systemctl: append default suffix only if none present"

This reverts a chunk out of commit 5e03c6e3b517286bbd65b48d88f60e5b83721894
which was trying to pass NULL to the the '.service' default. Anyway,
it seems better to be explicit.

treewide: more log_*_errno + return simplifications

systemctl: append default suffix only if none present

Simplify unit_name_mangle() and unit_name_mangle_with_suffix() to
always behave the same, and only append a suffix if there is no
type suffix. If a user says 'isolate blah.device' it is better to
return an error that the type cannot be isolated, than to try to
isolate blah.device.target.

udevd: don't set receive buffer size when socket activated

The socket unit does this, so no need to redo it in udevd.

treewide: simplify log_*_errno(r,...) immediately followed by "return r"

treewide: more log_*_errno() conversions, multiline calls

Basically:

find . -name '*.[ch]' | while read f; do perl -i.mmm -e \
'local $/;
 local $_=<>;
 s/log_(debug|info|notice|warning|error|emergency)\("([^"]*)%s"([^;]*),\s*strerror\(-?([->a-zA-Z_]+)\)\);/log_\1_errno(\4, "\2%m"\3);/gms;print;' \
 $f; done

Plus manual indentation fixups.

hostnamectl: Exit with zero on success

In show_all_names(), bus_map_all_properties() returns 1 on success which is
then used as the return code of show_all_names() and eventually main(). Exit
with zero in main() on all nonnegative results to guard against similar errors.

udev,update-done: more log_xyz_errno() conversions