meson: add support for building static libsystemd and libudev

Merge pull request #8807 from ChrisLesiak/systemd-update-done-mtime-fix

fileio.c: fix incorrect mtime

Merge pull request #8775 from poettering/strip-cso

teach strip_tab_ansi() to strip ANSI CSO sequences

Merge pull request #8802 from keszybz/errno-reform

Errno reform

fileio.c: fix incorrect mtime

A regression was introduced that caused the mtime of /etc/.updated
and /var/.updated to be the current time when systemd-update-done
ran instead of being copied from /usr.

This was nearly fixed, but due to fflush being called after mtime
was carefully set, it was overwritten with the current time.

Regression introduced in 872c40396384f9fa89b01baf4e739d28ed590299

A fix was just missed in 39c38d773fbe2b4f76ff29ecd3344640efb9a86c

Fixes #8806

Merge pull request #8800 from keszybz/trivialities

Trivial cleanups

Merge pull request #8801 from fbuihuu/tmpfiles-fixes

Tmpfiles fixes

update TODO

string-util: use fflush_and_check() where appropriate

string-util: teach strip_tab_ansi() to deal with CSO sequences

With the recent terminal_urlify() APIs we'll now sometimes generate
clickable link CSO sequences. Hence we should also be able to remove
them again from strings. This beefs up the logic to do so.

Follow-up for: 23b27b39d2a3a002ad827a2e8a9872a51495d797

basic/log: do not use global errno in log_*_errno()

Quoting https://github.com/systemd/systemd/pull/8760#discussion_r183321060:

> When we originally added the errno patching we went for a "best of both
> worlds" approach, i.e. that we override errno if an error is specified, but
> if no error is specified (i.e. 0 is passed as error code) then we use the
> previously set errno, similar in style how plain `printf()` would do it. In
> retrospect I think we almost never purposefully made use of the second,
> i.e. the plain `printf()` logic, but we multiple times ran into this case
> accidentally and introduced a bug. Hence yes, it probably makes sense to
> switch this over, and consistently ignore the `errno` already set and always
> override it with the error passed in. The only problem I see with that is: I
> wonder if there might be a case or two lurking somewhere where we actually
> made use of the "best of both worlds" approach, and if so, if we can detect
> where... (But then again, even if there is, and we fail to find those cases,
> maybe that's not all bad, as it's just a few new bugs against probably fixing
> many more old and future bugs, if you follow what I mean).

I scanned our codebase, and found some bugs in the value passed to log_*_errno,
but no intentional cases of error=0 being passed.

core/device: avoid bogus errno use and invert ratelimit_test()

I'm not sure if I understand the original code. AFAICS, errno does not
have to be set at all in this callback.

ratelimit_test() returns positive if we are under limit. The code would only
log if the condition happened very often, which I assume is not inteded, and
this check was supposed to prevent too much logging.

sd-lldp: eliminate unnecessary variable

basic/audit-util: always log the reason when disabling audit logs

This state is cached, and it's seems OK to log at least once.

Fix three uses of bogus errno value in logs (and returned value in one case)

tree-wide: avoid assignment of r just to use in a comparison

This changes
  r = ...;
  if (r < 0)
to
  if (... < 0)
when r will not be used again.

Revert "bus-unit-util: fix bus_wait_for_jobs() debug output (#8760)"

This reverts commit d6b87637c52148f5861aca557de2711dcd0c2efd.

Let's try a different approach.

tmpfiles: 'e' is supposed to accept shell-style globs

tmpfiles: 'e' is supposed to operate on directory only

systemd-hwdb: reflow help() to avoid a line break

CGROUP_DELEGATION: typo and grammar fixes

fs-util: introduce fchmod_opath()

fchmod(2) still doesn't take file descriptors opened with O_PATH.

tmpfiles: remove spurious curly braces in create_item()

core/manager: make manager_enumerate() static

core/manager: trivial simplification

nspawn: fix copy/reflink fallback when directory already exists (#8781)

Merge pull request #8796 from yuwata/remove-unused-variables

Remove unused variables

test: drop unused variables

Follow-up for 49eb36596b72e568d4b9b785cd4544e58c89bdee.

login: drop an unused variable

Follow-up for 99f1229d76da4b805f8f6c6e5e4a878d17d42f93.

man/os-release: Document that blank lines are permitted (#8777)

Fixes: https://github.com/systemd/systemd/issues/8773

hwdb: Add accelerometer orientation quirk for the Linx 1010B tablet (#8784)

hwdb: Use wlan keycode for all Dell systems (#8762)

Once the seat is acquired, gnome-settings-daemon (GSD) takes full
control of rfkill by sending RFKILL_IOCTL_NOINPUT ioctl to disable
rfkill control in kernel.

Since GSD expects wlan keycode when the hotkey gets pressed, we should
change the "unknown" to "wlan" accordingly.

Tested under both GNOME and KDE.

doc: add $SYSTEMD_TIMEDATED_NTP_SERVICES= in ENVIRONMENT.md (#8785)

bus-message: use streq_ptr() (#8786)

po: update Turkish translation (#8783)

Merge pull request #8520 from drinkcat/upstream-udevadm

udevadm/hwdb: Return non-zero exit code on error

hoestnamed: Also parse HOME_URL from /usr/lib/os-release (#8779)

64928aa54520b8cad3cda14d7aab0f4f59409acf added parsing to
/etc/os-release.

Merge pull request #8675 from fbuihuu/make-sure-device-enter-in-plugged-state

device: make sure to always retroactively start device dependencies

Merge pull request #8769 from keszybz/followups

Three trivial followups for recent patches

bus-unit-util: fix bus_wait_for_jobs() debug output (#8760)

We shouldn't print 'errno' if its not initialized properly.

systemd-hwdb update: Return non-zero exit code on error when --strict is used

 - Add a new flag --strict to tell systemd-hwdb to return a
   non-zero code on error.
 - Make systemd-hwdb update return an error when any parsing
   error occurs (only if strict flag is set).

udevadm/hwdb: Return non-zero exit code on error when --strict is used

 - Add a new flag --strict to tell udevadm hwdb to return a
   non-zero code on error.
 - Make udevadm hwdb --update return an error when any parsing
   error occurs (only if strict flag is set).

Merge pull request #8778 from poettering/cgroup-delegation-doc

some docs regarding cgroup delegation and systemd

doc: add a bit more documentation about systemd and cgroups and cgroupsv2 and delegation

Ultimately we should replace the relevant wiki texts with documentation
maintained as part of our project tree. This is a start. It can't
replace the wiki documentation in full yet, but I think it's already
very useful.

update TODO

device: skip deserialization of device units when udevd is not running

Do not try to party initialize a device during deserialization if it's not
known by udev (anymore) and therefore hasn't been seen during device
enumeration.

The device unit in this case has not been initialized properly and setting it
in the "plugged" state can be confusing.

Actually this happens during every boots when PID switches to the new rootfs:
PID is reexecuted and enumerates devices but since udev is not running, the
list of enumerated devices is empty.

device: make sure to always retroactively start device dependencies

PID1 updates the state of device units upon 2 different events:

 - when it processes an event sent by udev and in this case the device deps are
   started if the device enters in the "plugged" state.

 - when it enumerates all devices during its startup or when it is asked to
   reload its configuration data but in this case the device deps (if any) are
   not retroactively started.

When udev processes a new "add" kernel event, it first registers the new device
in its databases then sends an event to systemd.

If for any reason, systemd is asked to reload its configuration between the
previous 2 steps, it might see for the first time the new device while scanning
/sys for all devices. Only during a second step, udev will send the event for
the new device.

In this peculiar case the device deps wont be started (even though the device
is first seen by PID1).

Indeed when reloading its configurations, PID1 will put the device unit in the
"plugged" state but without starting the device deps. Thereafter PID1 will get
the event from udev for the new device but the device unit will be in "plugged"
state already therefore it won't see any need to start the device dependencies.

Rather than assuming that during the reloading of systemd manager configuration
all devices listed in udev DBs have been already processed and should be put in
the "plugged" state (done by device_coldplug()), this patch does that only for
devices which have been processed via an udev event (device_dispatch_io())
previously. In this case we set "d->found" to "DEVICE_FOUND_UDEV" and we make
also sure to no more initialize "d->found" while enumerating devices. Instead
this field is now saved/restored while devices are serialized.

networkd: more specific link down while enslaving (#8771)

Issue #5853 introduced a link_down for every netdev enslaved. This behaviour is
not required on other slave interfaces.

fixes 14b6bb7

man: document the new $SYSTEMD_DEBUGGER variable

Follow-up for c5896b6a.

sysusers: use uniform order for variables

Follow-up for 43e948eea6.

remove unused variables (#8768)

clang 5.0 complains with -Wunused-variable

networkd: add support to configure IPv6 MTU (#8664)

This patch supports to configure IPv6 MTU.

Closes #8632

nspawn: when running nspawn, set a $PATH including both bin + sbin by default (#8756)

We don't know what the container payload needs, hence default to a PATH
with both bin and sbin included, as well as / and /usr.

Follow-up for #8324

Fixes: #8698

Fix typo

Merge pull request #8754 from poettering/sysusers-fix

two minor memleak fixes for sysusers, fixing #8718

Merge pull request #8767 from poettering/urlify-all-things

try to generate clickable links in our output if we can

systemctl: format unit file and dropin paths as clickable links in status output

systemctl: format documentation links in status output as clickable links

hostnamectl: make the distribution name a pretty clickable link if we can

hostnamed: expose HOME_URL os-release field on the bus

terminal: add internal API to format URLs for display in capable terminals

Newer terminals (in particular gnome-terminal) understand special escape
sequence for formatting clickable links. Let's support that to make our
tool output more clickable where that's appropriate.

For details see this:

https://gist.github.com/egmontkob/eb114294efbcd5adb1944c9f3cb5feda

The one big issue is that 'less' currently doesn't grok this, and
doesn't ignore sequence like regular terminal implementations do if they
don't support it. Hence for now, let's disable URL output if a pager is
used. We should revisit that though as soon as less added support for it
and enough time passed for it to enter various distributions.

Merge pull request #8615 from yuwata/fix-1329

timedated: support multiple NTP services

Merge pull request #8765 from poettering/test-fixes

some short fixes for the tests

Merge pull request #8623 from yuwata/resolvectl

resolvectl: rename systemd-resolve to resolvectl

tree-wide: drop spurious newlines (#8764)

Double newlines (i.e. one empty lines) are great to structure code. But
let's avoid triple newlines (i.e. two empty lines), quadruple newlines,
quintuple newlines, …, that's just spurious whitespace.

It's an easy way to drop 121 lines of code, and keeps the coding style
of our sources a bit tigther.

user-util: trivial coding style fixes

Use C's downgrade-to-bool feature when comparing pointers against NULL,
as we usually do.

mkosi: drop dumping all test output to console again

As it appears meson's test cases nowadays (?) show useful logs for
failing tests anyway, hence there's no need to show them unconditionally
in full every time anymore. Let's hence simplify and drop this.

tests: compare with correct shell

We synthesize the passwd record for UID 0, hence we need to compare with
our synthesized data and not with the data stored in /etc/passwd

core: don't export per-unit metadata files in test mode

We shouldn't clobber the host's /run directories with metadata we export
for our units when we run in test mode.

mkosi: ensure the 'hostname' tool is available in the build environment

Our test-execute test uses 'hostname' to validate specifiers, hence it
should be available in our mkosi build too, so that the test can pass.

Merge pull request #8758 from keszybz/improved-slice-checks

Improved slice checks

coredumpctl: Add debug as a gdb alias, and add lldb support (closes #8722) (#8744)

bash-completion: add completion for resolvectl

man: create man page for resolvectl

resolvectl: change syntax to use verb_dispatch()

This makes `resolvectl` use the verb style command line, e.g.,
`resolvectl status` or `resolvectl tlsa tcp fedoraproject.org:443`.
For compatibility, if the invocation name is `systemd-resolve`,
then it accepts the old syntax, e.g. `systemd-resolve --status`.

resolvectl: rename systemd-resolve to resolvectl

For the compatibility, `systemd-resolve` will be created as a symbolic
link to `resolvectl`.

core: do not allow Delegate= on unsupported unit types

core: fix resetting of Delegate= and properly ignore invalid assignment

The default is false not true. If we say "ignoring" we must return 0.

man: don't place nginx socket in /tmp (#8757)

First of all, it's frickin' ugly and wrong, as IPC sockets should be
placed in /run and definitely not under a guessable name under
world-writable /tmp. Secondly, it can't even work as we set
PrivateTmp=yes on the service.

Hence, let's clean up the example, and simply use a socket in /run
instead.

Fixes: #8419

Merge pull request #8708 from poettering/namespace-repeat

pid1 namespacing fixes

sysusers: make sure to reset the returned value when EOF is reached in fget*ent_sane() wrappers (#8737)

To indicate that the there're no more entries, these wrappers return false but
did leave the passed pointed unmodified.

However EOF is not an error and is a very common case so initialize the output
argument to NULL even in this case so callers don't need to do that.

Fixes: #8721

sysusers: fix minor memory leak

sysusers: clarify that we knowingly ignore process_items() return values

sysusers: fix memory leak when /etc/passwd contains multiple identical lines

Fixes: #8718

uid-range: remove spurious newline

Merge pull request #8709 from poettering/format-table

generic table formatter

path-lookup: properly chase paths when reducing with root dir (#8750)

Let's make this correct.

util-lib: introduce new empty_or_root() helper (#8746)

We check the same condition at various places. Let's add a trivial,
common helper for this, and use it everywhere.

It's not going to make things much faster or much shorter, but I think a
lot more readable

namespace: rework how we resolve symlinks in mount points

Before this patch we'd resolve all symlinks of bind mounts and other
mount points to establish for a service in advance, and only then start
mounting them. This is problematic, if symlink chains jump around
between directories in a namespace tree, so that to resolve a specific
symlink chain we need to establish another mount already. A typical case
where this happens is if /etc/resolv.conf is a symlink to some file in
/run: in that case we'd normally resolve and mount /etc/resolv.conf
early on, but that's broken, as to do this properly we'd need to resolve
/etc/resolv.conf first, then figure out that /run needs to be mounted
before we can proceed, and thus reorder the order in which we apply
mounts dynamically.

With this change, whenever we are about to apply a mount, we'll do a
single step of the symlink normalization process, patch the mount entry
accordingly, and then sort the list of mounts to establish again, taking
the new path into account. This means that we can correctly deal with
the example above: we might start with wanting to mount /etc/resolv.conf
early, but after resolving it to the path in /run/ we'd push it to the
end of the list, ensuring that /run is mounted first.

(Note that this also fixes another bug: we were following symlinks on
the bind mount source relative to the root directory of the service,
rather than of the host. That's wrong though as we explicitly document
tha the source of bind mounts is always on the host.)

namespace: improve logging when creating mount source nodes

namespace: split out calls to normalize mount entry list into new function

namespace: don't consider raw image read-only if /home in it is writable

fs-util: export how many iterations chase_symlinks() executes at max

fs-util: add new CHASE_STEP flag to chase_symlinks()

If the flag is set only a single step of the normalization is executed,
and the resulting path is returned.

This allows callers to normalize piecemeal, taking into account every
single intermediary path of the normalization.

update TODO

test-locale-util: show special glyphs

This is mostly useful as a sanity check.

test-utf8: add a smoke test for utf8_console_width()

loginctl: port loginctl to format-table.[ch]

machinectl: port machinectl to format-table.[ch]

basic: add minimalistic table formatter

We have plenty of code in our codebase that outputs tables to the
console, and all is homegrown and awful. Let's replace it with a generic
implementation that can do automatically what the old implementations
did manually.

Features:

1. Ellipsation (for fields overly long) and alignment (for
   fields overly short)

2. Sorting of rows

3. automatically copies formatting from the same cell in the row above

4. Heavy use of varargs to make putting together tables easy

5. can expand and compress tables, with weights

6. Has a minimal understanding of unicode wide characters in order to
   match unicode strings to character cell terminals.

7. Columns can be reordered and individually turned off.

8. pretty printing for various data types

And more.