cpu-set-util: Accept commas as separators in parse_cpu_set_and_warn

Tested CPUAffinity settings on both a service unit and in system.conf
and confirmed they work as expected.

Added a new test to confirm that trailing commas and spaces work and to
prevent any regressions in that area.

Merge pull request #1698 from poettering/pid-is-unwaited-fix

fixup for previous pr

service: exiting early is nicer than deeply-indented code blocks

core: all unit types that watch control PIDs should use the same logic

When coldplugging the unit state, make sure to follow the same basic
logic for all unit types: always verify whether the control PID is still
a waitable process before proceeding.

core: don't pass uninitialzed PIDs to pid_is_unwaited()

Since 5fd9b2c5467b0a42ccdabc7eb8e516d512609a8e passing a pid of 0 to
pid_is_unwaited() and pid_is_live() is considered as a request on the
current process, similar how the other calls in process-util.c handle a
PID of 0. This broke service.c, which passes a 0 PID and expects it to
be considered an unwaited process.

This fix make sure we can boot again.

Merge pull request #1695 from evverx/fix-cap-bounding-merging

core: fix CapabilityBoundingSet merging

Merge pull request #1691 from poettering/util-lib-3

split up util.h and macro.h into even more bits

util-lib: move inotify-related definitions to fs-util.[ch]

util-lib: move fdname_is_valid() to fd-util.[ch]

automount: move generically userful call repeat_mount() into mount-util.[ch]

process-util: move a couple of process-related calls over

process-util: make some minor corrections to PID live detection

process-util: rename get_parent_of_pid() → get_process_ppid()

In order to match the other get_process_xyz() calls.

process-util: actually move rename_process() over

The prototype was moved long ago, actually move the definition over now,
too.

mount-util: move fstype_is_network() and name_to_handle_at() definitions over

update TODO

util-lib: split out allocation calls into alloc-util.[ch]

util-lib: split out globbing related calls into glob-util.[ch]

stat-util.h: move F_TYPE_EQUAL() macro definition to stat-util.h

util-lib: split out printf() helpers to stdio-util.h

macro.h: move definition of MODE_INVALID to parse-util.h

macro: move DECIMAL_STR_WIDTH() into macro.h

So that it sets next to DECIMAL_STR_MAX(), where it belongs.

socket-util: move CMSG_FOREACH() from macro.h to socket-util.h

io-util.h: move iovec stuff from macro.h to io-util.h

process-util: move PID related macros from macro.h to process-util.h

user-util: move UID/GID related macros from macro.h to user-util.h

util-lib: move web-related calls into web-util.[ch]

util-lib: split out syslog-related calls into syslog-util.[ch]

detect-virt: add new --chroot switch to detect chroot() environments

util-lib: move running_in_chroot() into virt.[ch]

It's a very weak kind of virtualization, after all.

util-lib: move /proc/cmdline parsing code to proc-cmdline.[ch]

util-lib: move is_main_thread() to process-util.[ch]

src/basic: rename audit.[ch] → audit-util.[ch] and capability.[ch] → capability-util.[ch]

The files are named too generically, so that they might conflict with
the upstream project headers. Hence, let's add a "-util" suffix, to
clarify that this are just our utility headers and not any official
upstream headers.

util-lib: split out umask-related code to umask-util.h

util-lib: move more locale-related calls to locale-util.[ch]

util-lib: move status printing stuff into src/core/

It's very specific to the core, and not used elsewhere, hence move it
out of terminal-util.[ch].

util-lib: move string table stuff into its own string-table.[ch]

util-lib: split stat()/statfs()/stavfs() related calls into stat-util.[ch]

util: move string_is_safe() to string-util.[ch]

util-lib: move a number of fs operations into fs-util.[ch]

util-lib: split out file attribute calls to chattr-util.[ch]

util-lib: split xattr-related calls into xattr-util.[ch]

util-lib: introduce dirent-util.[ch] for directory entry calls

Also, move a couple of more path-related functions to path-util.c.

util-lib: split out resource limits related calls into rlimit-util.[ch]

util-lib: move take_password_lock() to user-util.[ch]

Also, rename it take_etc_passwd_lock(), in order to make it more
expressive.

util: move filename_is_valid() and path_is_safe() to path-util.[ch]

util-lib: move mount related utility calls to mount-util.[ch]

util-lib: move fstab_node_to_udev_node() to fstab-util.[ch]

util-lib: move more file I/O related calls into fileio.[ch]

util: remove path_get_parent(), in favour of dirname_malloc()

We don't need two functions that do essentialy the same, hence drop
path_get_parent(), and stick to dirname_malloc(), but move it to
path-util.[ch].

util-lib: split out hex/dec/oct encoding/decoding into its own file

util-lib: split string parsing related calls from util.[ch] into parse-util.[ch]

core: fix CapabilityBoundingSet merging

Fixes: #1221

Merge pull request #1693 from ssahani/word

timesysnd: port to extract_first_word

resolved: port to extract_first_word

timesysnd: port to extract_first_word

Merge pull request #1688 from phomes/typo-fix

path-util: do not return NULL as int

path-util: do not return NULL as int

strv_split will only return NULL on oom so we should return -ENOMEM
instead.

Looks like an oversight from the changes in 0f474365

Merge pull request #1679 from evverx/refuse-manual-start-by-reload-or-restart

core: don't allow manual start with reload-or-restart too

Merge pull request #1686 from medhefgo/remount-ro-fix

unmount: Pass in mount options when remounting read-only

unmount: Pass in mount options when remounting read-only

man 2 mount says that the mountflags and data parameteres should
match the original values except for the desired changes. We only
bother with the mount options since the only flags we can change
are MS_RDONLY, MS_SYNCHRONOUS and MS_MANDLOCK; which shouldn't
matter too much.

Fixes: #351

Merge pull request #1676 from poettering/util-lib-2

split up util.[ch] into more pieces, and other stuff

journal: add missing include

Merge pull request #1681 from ssahani/journal

journald-server: port to extract_first_word

journald-server: port to extract_first_word

core: don't allow manual start with reload-or-restart too

fix bug: systemctl reload-or-restart starts a service with RefuseManualStart=yes

Merge pull request #1675 from evverx/run-protect-home

systemd-run can launch units with ProtectHome

process-util: move more process related calls to process-util.[ch]

socket-util: move remaining socket-related calls from util.[ch] to socket-util.[ch]

build-sys: add missing makefile symlink

util-lib: move ether_addr_to_string() into ether-addr-util.c

path-util: minor coding style fix

We usually avoid relying on C's degrade-to-boolean functionality when
comparing numerical variables with 0. We use it only for pointers and
actual booleans.

core: rename SmackFileSystemRoot= to SmackFileSystemRootLabel=

That way it's in sync with the other SMACK label settings.

https://github.com/systemd/systemd/pull/1664#issuecomment-150891270

user-util: never hand out or accept invalid UIDs

libc isn't that strict, but it's a good idea if we are, to not create
confusion around invalid user ids.

util: remove lookup_uid(), replace by uid_to_name()

So far we had two pretty much identical calls in user-util.[ch]:
lookup_uid() and uid_to_name(). Get rid of the former, in favour of the
latter, and while we are at it, rewrite it, to use getpwuid_r()
correctly, inside an allocation loop, as POSIX intended.

user-util: simplify uid parsing a bit

util-lib: split out user/group/uid/gid calls into user-util.[ch]

util-lib: split out IO related calls to io-util.[ch]

shell-completion: systemd-run: add the property ProtectHome

run: can launch units with ProtectHome

Merge pull request #1674 from evverx/run-fix-couldnot-find-executable

run: fix "couldn't find executable" for the existing executables

Merge pull request #1673 from mustrumr/date-fixes

Date fixes

run: fix "couldn't find executable" for the existing executables

Fixes: #1672

man: fix systemd.time RAS syndrome and line length

"UTC time" is a RAP phrase (redundant acronym phrase phrase).

test: "today UTC" is not always "today UTC"

On Oct 25 2015 in EET/EEST there is a UTC+3->UTC+2 transition. This
means that the representation of "today UTC" as local time is ambiguous.

basic: use the return value of endswith

It returns the position where the suffix begins, which can be used for
strndup to extract the prefix without calling strlen.

Merge pull request #1671 from keszybz/zsh-journalctl-identifier

zsh-completion: journalctl --identifier

zsh-completion: journalctl --identifier

Fixes #1549.

Merge pull request #1668 from ssahani/net1

networkd: fix asserts

Merge pull request #1654 from poettering/util-lib

Various changes to src/basic/

Merge pull request #1667 from evverx/run-protect-system

systemd-run can launch units with ProtectSystem

util-lib: move formats-util.h from shared/ to basic/

It's only a header file, definining format strings for basic system
types, hence it should be in src/basic/, not src/shared/.

util-lib: split out fd-related operations into fd-util.[ch]

There are more than enough to deserve their own .c file, hence move them
over.

Merge pull request #1663 from poettering/journal-compress-fix

journal: fix error handling when compressing journal objects

vxlan: fix assert

veth: fix assert

ipvlan: fix assert

bond: fix assert

vlan: fix assert

shell-completion: systemd-run: add the property ProtectSystem