shared: add process-util.[ch]

shared: add formats-util.h

dbus: typo fix in log

tmpfiles: fix build with clang

Clang is not happy about using the cleanup attribute in switches

efivars: fix build for non-efi

shared: boot-timestamps - remove ifdef

No need to ifdef out efi code as the functions are always defined.

shared: efivars - fix compile on non-EFI systems

systemctl and logind were unconditionally using functions that were not compiled
on non-EFI systems. Add stubs returning -EOPNOTSUPP to fix compile again.

shared: efivars - is_efi_*() returns bool instead of int

There was a bug where is_efi_*() could return a negative error value, which would be treated as 'true',
just make this a bool in the helper library to avoid the problem.

Revert "sd-dhcp-client: fix strict aliasing issue"

This reverts commit 6ec8e7c763b7dfa82e25e31f6938122748d1608f.

This doesn't fix any issues, just makes the code harder to read.

sd-event: simplify sd_event_run()

tmpfiles: add specifier expansion for L and C lines, too

update TODO

bus-util: be more verbose if dbus job fails

Users might have hard time figuring out why exactly their systemctl request
failed. If dbus job fails try to figure out more details about failure by
examining Result property of the service.

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

core: set_put never returns -EEXIST

When the value is already there it returns 0.

Also add a test to ensure this

tmpfiles: enforce ordering when executing lines

Always create files first, and then adjust their ACLs, xattrs, file
attributes, never the opposite. Previously the order was not
deterministic, thus possibly first adjusting ACLs/xattrs/file
attributes before actually creating the items.

tmpfiles: eat up empty columns

tmpfiles: substitute % specifiers in arguments for writing files and xattrs

tmpfiles: warn if we get an argument on lines that don't take any

tmpfiles: mostly revert 71044f609b829d802e0eb81270e13b4f55d76476

Add a comment why returning a positive error is OK and intended in this
case.

(It's still a nasty hack to do this though!)

device-nodes: minor simplifications

util: unify how we parse mode_t strings

util: remove normalize_env_assignment(), it's unused

shared: fix memleak

path was used for 2 purposes but it was not freed before being reused.

util: mark unquote() static, so that new code doesn't use this anymore

util: fix unicode decoding in unquote_first_word()

util: when unescaping C escape sequences support C++11 \u and \U unicode literals

We simply recode them in utf8.

util: fix a typo in comment

terminal/page: drop unneeded DISABLE_WARNING_*

The macros in shared/macro.h have been updated to no longer use fixed
variable names. This will silence any shadowing-warnings, so drop the
DISABLE_WARNING_* annotations.

bus: simplify bus_message_read_strv_extend()

Avoid unbound for(;;) loop and use the established coding-style:
  while ((r = sd_bus_message_read*(...)) > 0) {
  }
  if (r < 0)
          return r;

This is much easier to read and used all over the code base.

bus: use free_and_strdup()

Save some LOCs by replacing strdup()+error-handling+free+assign with
free_and_strdup().

terminal/idev: fix NULL terminator for bus-property mapping

We require a NULL entry as sentinal in bus_properties_map. Fix this!

test-journal-flush: fix memleak

util: cunescape - fix memleak in error path

util: add shell_maybe_quote() call for preparing a string for shell cmdline inclusion

If necessary the passed string is enclosed in "", and all special
characters escapes.

This also ports over usage in bus-util.c and job.c to use this, instead
of a incorrect local implementation that forgets to properly escape.

vconsole: some modernizations, indentation fixes

tmpfiles: properly return error code from path_set_acl()

tmpfiles: add file attribute calls to list of glob needing commands

update TODO

vconsole: stick to kernel-style function error handling

This undoes part of 8931278c8a9ef7b60f6b6963486a9e9febb20a0d.

We really should stick to kernel-style "int" return values from
functions, and not covnert success into boolean returns.

hwdb: mouse DPI data, Roccat Lua (ROC-11-310)

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

networkd: do not assign the variable twice

update TODO

tmpfiles: rework file attribute code

- Stick to one type for the flags field: unsigned. This appears to be
  what the kernel uses, and there's no point in using something else.

- compress the flags array by avoiding sparse entries

- extend some error messages to not use abbreviated words

- avoid TTOCTTOU issues by invoking fstat() after open() when applying
  file flags

- add explanation why we need to check the file type with fstat().

- don't needlessly abbreviate "attribute" as "attrib", in particually as
  "chattr" abbreviates it as "attr" rather than "attrib".

util: merge change_attr_fd() and chattr_fd()

fstab-generator: don't continue on OOM

timesyncd: print an INFO log message when we successfully reached a time server

man: avoid confusion regarding "time synchronization state"

Also, update example in the timedatectl man page to match the actual
timedatectl output.

systemctl: Use logind for --firmware-setup if possible

Indentation

man: remove 'fs' from 'rootfsflags'

rootfsflags does not appear anywhere else.

core/automount: beef up error message

This should not happen... but when it does more information is nice.

shell-completion: systemctl switch-root verb

The completion is rudimentary (all files). I think this is OK since
this is used so rarely. But not having it proposed at all is annoying.

update TODO

resolved: maintain order when writing resolv.conf entries

http://lists.freedesktop.org/archives/systemd-devel/2015-March/029850.html

util: add minimal OrderedSet type

OrderedSet implements a Set-like structure, but maintains insertion
ordered. It is hence to Set what OrderedHashmap is for Hashmap.

Internally, this is only a wrapper around OrderedHashmap for now, but
this could one day be improved and be added to hashmap.c natively.

TODO: yeah, now it makes sense

missing.h: Define IFA_F_NOPREFIXROUTE

Fixes second systemd compilation problem against Linux 3.12 uapi
headers.

IFA_F_NOPREFIXROUTE is a usual #define appeared in Linux 3.14, so
AC_CHECK_DECLS is not necessary.

sd-rtnl: properly size attribute array if IFA_FLAGS was missing

po: update French translation

Add strings introduced by 5bdf22430e367799dfa66c724144b624c5479518

remove duplicated includes

timedatectl: trim --help output to 80 chars

zsh-completion: update hostnamectl

zsh-completion: add missing -M completion for journalctl

zsh-completion: update loginctl

test-socket-util: add test for in_addr_to_string

configure: sign-compare is part of extra

Remove sign-compare as we already get it from extra

device: remove unused null check

We dereference the variable right before the null check. We never reach
this point with a null value anyway so let's just remove it.

logind: drop "interactive" parameter from SetRebootToFirmwareSetup() bus call

Newer dbus versions have an "allow interactive authentication" bit in
the message header, hence it is not necessary to take a boolean for this
explicitly.

update TODO

missing.h: define IFA_FLAGS if it is missing

test-util: don't mix declarations and code

update TODO

socket: minor modernizations

logind,systemctl: add reboot to EFI firmware support

update TODO

man: fix typos in machinectl

The s/now/not/ unfortunately inverted the meaning.

man: Resolve how to refer to derived names for machinectl containers

"the name it is" is clumsy english, and since the most recently referred
to thing was a name anyway we can just leave the "it is".

This matches later uses in the same document.

update TODO

timedatectl: avoid specifically referring to NTP

systemd-timesyncd not only does NTP, but also manages clock monotonicity
using a flags file. In future, it might learn PTP support. Hence don't
expose its enablement state as "NTP" but use the more generic term
"network time synchronization". After all, for similar reasons
systemd-timesyncd is not called systemd-ntpd.

man: update timedatectl(1)'s examples

The output didn't match what we output these days. Also, "set-ntp"
cannot be used to control chronyd anymore.

man: document in timedatectl(1) that the NTP state shown reflects whether timesyncd is enabled

path-util: fix more path_is_mount e792e890f fallout

update TODO

man: document default timeouts and start limit parameters

polkit: rename bus_verify_polkit() to bus_test_polkit() and make it strictly non-interactive

Interactive authorization should only happen asynchronously, hence
disallow it in synchronous bus_verify_polkit(), and rename it to
bus_test_polkit(). This way even if the bus message header asks for
interactive authorization, we'll ask for non-interactive authorization
which is actually the desired behaviour if CanSuspend, CanHibernate and
friends, which call this function.

util: rework cunescape(), improve error handling

Change cunescape() to return a normal error code, so that we can
distuingish OOM errors from parse errors.

This also adds a flags parameter to control whether "relaxed" or normal
parsing shall be done. If set no parse failures are generated, and the
only reason why cunescape() can fail is OOM.

core: fix mount setup to work with non-existing mount points

We must not fail on ENOENT. We properly create the mount-point in
mount-setup, so there's really no reason to skip the mount. Make sure we
just skip the mount on unexpected failures or if it's already mounted.

device: fix sysattr newline cutter

Right now, we always drop the last character of all values we write to
sysfs. Fix this!

udev: input_id: tag accelerometers as ID_INPUT_ACCELEROMETER

input_id already (tries to) tag accelerometers as such, but this only works
for absolute accelerometers. Recent kernels mark accelerometers through an
input prop. Trust that prop and always tag devices with it with
ID_INPUT_ACCELEROMETER.

Note that detection by the prop bit works the same as the existing detection
and will ensure that no other tags get set on the device.

udev: input_id: tag pointing sticks as ID_INPUT_POINTINGSTICK

Also referred to as trackpoint, trackstick. These are marked by recent kernels
through an input prop. Forward that prop as udev property so userspace can
easily determine whether there is a pointing stick present.

These devices were previously marked as ID_INPUT_MOUSE, for backwards
compatibility we keep that in place, the new property is an addition.

core: mount-setup: handle non-existing mountpoints gracefully

Commit e792e890f ("path-util: don't eat up ENOENT in
path_is_mount_point()") changed path_is_mount_point() so it doesn't hide
-ENOENT from its caller. This causes all boots to fail early in case
any of the mount points does not exist (for instance, when kdbus isn't
loaded, /sys/fs/kdbus is missing).

Fix this by returning 0 from mount_one() if path_is_mount_point()
returned -ENOENT.

build-sys: fix typo

update TODO

btrfs: add support for recursive btrfs snapshotting

btrfs: missing endian conversion fix

btrfs: make btrfs_subvol_snapshot() parameters a flags field

btrfs: support recursively removing btrfs snapshots

machined: drop btrfs subvolumes when removing container images

path-util: don't eat up ENOENT in path_is_mount_point()

There's no reason to eat up ENOENT, it should be OK to simply report the
error back.

path-util: make sure fd_is_mount_point() returns true for root directory