Update TODO

machinectl: add "enable" and "disable" verbs for enabling/disabling systemd-nspawn for containers

This is basically just a shortcut for "systemctl enable
systemd-nspawn@<foobar>.service", but does escaping.

machinectl: add new "start" verb to start a container as a service in nspawn

units: rework systemd-nspawn@.service unit

- Unescape instance name so that we can take almost anything as instance
  name.

- Introduce "machines.target" which consists of all enabled nspawns and
  can be used to start/stop them altogether

- Look for container directory using -M instead of harcoding the path in
  /var/lib/container

units: make graphical.target dependencies more complete and similar to those of multi-user.target

bus-proxy: fix sd_bus_reply_*() usage

We *must* not use sd_bus_reply_*() as it does not set the sender field
correctly. Use the synthetic_reply_*() helpers instead!

capability: use /proc/sys/kernel/cap_last_cap

This file was introduced with linux-3.2, use it instead of probing for it
via prctl(PR_CAPBSET_READ).

For now, keep the old code for backwards compat. We can drop it once 3.2
is our lowest requirement.

The test-cap-list code is extended to verify cap_last_cap() is the same as
we'd get via prctl probing and /proc.

core: loopback - correctly fail the loopback_check if somehow the rtnl calls fail

bus: fix typo

Drop spurious 'we'.

rtnl: recv_message - don't enforce sender uid

All we care about is that the kernel (pid==0) sent the message. Verifying the sender uid
seems to break when using userns.

Reported by Stéphane Graber.

test: loopback - parse logging env var

sd-rtnl: rtnl_call - don't dispatch wqueue after timeout has passed

Only a minor change as the timeout would be hit soon thereafetr at the next loop.

sd-rtnl: rtnl_poll - fix typo

This caused rtnl_poll to always return true immediately in sd_rtnl_call().

sd-rtnl: recv_message - drop message when peeking fails

Read the message form the socket or we will loop trying to read the
same message repeatedly.

sd-rtnl: recv_message - don't fail on interrupt

We should just try again instead.

sd-rtnl: recv_message - log when dropping message

We drop messages received from the wrong uid/pid, log this at debug level.

core: loopback - simplify check_loopback()

We no longer configure the addresses on the loopback interface, but simply bring it up
and let the kernel do the rest. Also change the check to only check if the interface
is up, rather than checking for the IPv4 loopback address.

machined: Fix MarkReadOnly method's name on bus

libsystemd: Fix minor typo in comment

machined: don't look for images on each property get, but cache the image object inbetween

update TODO

util: treat -1 as special size in format_bytes()

machined: add support for reporting image size via btrfs quota

tmpfiles.d: upgrade a couple of directories we create at boot to subvolumes

In particular we upgrade /var/lib/container, /var/tmp and /tmp to
subvolumes.

tmpfiles: add new line type 'v' for creating btrfs subvolumes

machinectl/machined: implement "rename", "clone", "read-only" verbs for machine images

machined: add "machinectl remove" for removing images

nspawn: use the same image discovery logic in nspawn as in machined

machined: Move image discovery logic into src/shared, so that we can make use of it from nspawn

Fix check_loopback()

Add missing htonl() so that check_loopback() actually tests for 127.0.0.1
instead of 1.0.0.127 on little-endian machines.

loginctl: reindent --help text

loginctl: add more --help sections

machinectl: add status commands

machined: fix search patch magic for '.host' image

machined: make image read-only check indepenednt on own privs

machinectl: mark read-only images when listing in red

update TODO

import: make image root directory configurable, instead of hardcoding /var/lib/container

machined: when discovering images, implicitly add ".host" as pseudo image referring to the host's own directory tree

machined: fix image search path iteration

machined: let's also check machine directories in /usr and /usr/local

import: properly remove pre-existing images if --force is used

import: beef up gpt importer to optionally make writable copy of read-only vendor image

import: minor improvements to dkr importer

util: always override crtime xattr

machinectl: left-align times

copy: try top copy atime/time/xattrs when copying files

test: improve btrfs test case

machined: be more thorough when checking whether an image is writable or not

journald: always allocate space for object fields

If OBJECT_PID= came as the last field, we would not reallocate the iovec to bigger size,
and fail the assertion later on in dispatch_message_real().

journald: fix off by one in native transport

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

hwdb: mouse - add Razer Abyssus

man: mark hwdb related man pages as conditional

Follow-up fix for commit fe659612e40e6a0cf91a73b10aa45eea958b747e.

man: typo in sd_watchdog_enabled notes

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

test: wait for cloned thread to exit

In test_raw_clone, make sure the cloned thread calls _exit() and in the parent
thread call waitpid(..., __WCLONE) to wait for the child thread to terminate,
otherwise there is a race condition where the child thread will log to the
console after the test process has already exited and the assertion from the
child thread might not be enforced.

The absence of this patch might also create problems for other tests that would
be added after this one, since potentially both parent and child would run
those tests as the child would continue running.

Tested by confirming that the logs from the child are printed before the test
terminates and that a false assertion in the child aborts the test with a core
dump.

[zj: also add check for the return value.]

test: only use assert_se in test_raw_clone

The asserts used in the tests should never be allowed to be optimized away.

configure.ac: keep posix compat for string tests

'==' and '=' are equivalent in /bin/bash, but POSIX compliant shells do
not understand '==.'

man: add a note why %U,%h,%s are mostly useless

The reasons were already given in the description of %U, but the
are easy to overlook, and it's better to be explicit to avoid
confusion.

run: uninitialized variable

ata_id: remove temp variable to kill warning

src/udev/ata_id/ata_id.c:503:24: warning: assignment from incompatible pointer type
         identify_words = &identify.wyde;
                        ^

ata_id: modernize

pam_systemd: remove spurious include of <sys/capability.h>

It does not use any functions or constants from libcap directly.

Tested that "pam_systemd.la" builds cleanly and works after this change.

timedated: remove spurious include of <sys/capability.h>

It does not use any functions from libcap directly. The CAP_SYS_TIME constant
in use by this file comes from <linux/capability.h> imported through "missing.h".

Tested that "systemd-timedated" builds cleanly and works after this change.

localed: remove spurious include of <sys/capability.h>

It does not use any functions from libcap directly. The CAP_SYS_ADMIN constant
in use by this file comes from <linux/capability.h> imported through "missing.h".

Tested that "systemd-localed" builds cleanly and works after this change.

bus: remove spurious include of <sys/capability.h>

They do not use any functions from libcap directly. The CAP_SYS_ADMIN constant
in use by bus-objects.c comes from <linux/capability.h> imported through
"missing.h". The "missing.h" header is imported through "util.h" which gets
imported in "bus-util.h".

Tested that everything builds cleanly after this change.

machined: remove spurious include of <sys/capability.h>

They do not use any functions from libcap directly. The CAP_KILL constant in
use by these files comes from <linux/capability.h> imported through
"missing.h".

Tested that "systemd-machined" builds cleanly and works after this change.

hostnamed: remove spurious include of <sys/capability.h>

It does not use any functions from libcap directly. The CAP_SYS_ADMIN constant
in use by this file comes from <linux/capability.h> imported through "missing.h".

Tested that "systemd-hostnamed" builds cleanly and works after this change.

tmpfiles: remove spurious include of <sys/capability.h>

It does not use any functions from libcap directly. The CAP_MKNOD constant in
use by this file comes from <linux/capability.h> imported through "missing.h".

Tested that "systemd-tmpfiles" builds cleanly and works after this change.

logind: remove spurious include of <sys/capability.h>

They do not use any functions from libcap directly. The CAP_* constants in use
through these files come from "missing.h" which will import <linux/capability.h>
and complement it with CAP_* constants not defined by the current kernel
headers. The "missing.h" header is imported through "util.h" which gets
imported in "logind.h".

Tested that "systemd-logind" builds cleanly and works after this change.

nspawn: remove spurious include of <sys/capability.h>

It does not use any functions from libcap directly. The CAP_* constants in use
through this file come from "missing.h" which will import <linux/capability.h>
and complement it with CAP_* constants not defined by the current kernel
headers.

Add an explicit import of our "capability.h" since it does use the function
capability_bounding_set_drop from that header file. Previously, that header was
implicitly imported through through "cap-list.h".

Tested that "systemd-nspawn" builds cleanly and works after this change.

doc: os-release: mention all shell characters that must be escaped

Since the manpage already talks about shell-compatibility, it should be
more accurate about what needs to be escaped and how.

build-sys: remove references to dbusinterfacedir

This directory is not used by systemd.

Tested by running a full build, running `make install` and comparing the file
list in the target trees and making sure that `make distcheck` still works.

build-sys: do not use pkgconfig dbus-1.pc to find dbus directories

Do not use the dbus-1.pc pkgconfig settings to determine dbus directories. Use
directories relative to ${sysconfdir} and ${datadir} instead.

This approach was suggested by Simon McVittie in:
http://lists.freedesktop.org/archives/systemd-devel/2014-October/024388.html

Tested by building and installing systemd without the dbus-devel installed.
Without this patch, the dbus files and directories end up in the root of the
filesystem. With this patch, they end up in the same locations as previously
(assuming default ${sysconfdir} and ${datadir}) whether dbus-devel is present
or not. Also made sure that `make check` works without dbus-devel installed.

timesync: remove square(), use pow instead

In any case, the compiler generates the same code inline and never
actually calls the library function.

test: do not use last cap from kernel in test-cap-list

The new test-cap-list introduced in commit 2822da4fb7f891 uses the included
table of capabilities. However, it uses cap_last_cap() which probes the kernel
for the last available capability. On an older kernel (e.g. 3.10 from RHEL 7)
that causes the test to fail with the following message:

    Assertion '!capability_to_name(cap_last_cap()+1)' failed at src/test/test-cap-list.c:30, function main(). Aborting.

Fix it by exporting the size of the static table and using it in the test
instead of the dynamic one from the current kernel.

Tested by successfully running ./test-cap-list and the whole `make check` test
suite with this patch on a RHEL 7 host.

udev: fix another strict aliasing issue

libudev: fix strict aliasing violation

build-sys: update path in reference to sd-lldp.h

The file was moved from src/libsystemd-network to src/systemd in commit
7a6f1457462840 ("sd-lldp: minor header cleanup").

This fixes "make distcheck".

machined: add org.freedesktop.machine1.policy.in to POTFILES.in

The new polkit file was introduced in commit d04c1fb8e21560 ("machined:
introduce polkit for OpenLogin() call").

update TODO

machined: beef up machined image listing with creation/modification times of subvolumes

We make use of the btrfs subvol crtime for this, and for gpt images of a
manually managed xattr, if we can.

import: prefer usec_t over time_t

update TODO

update TODO

import: add a new "pull-gpt" verb for downloading GPT disk images from the internet

import: three minor fixes

import: print friendly error messages on errors

import: remember when we were finished importing

import: make the dkr import URL a part of the import object, not the import name object

machined: fix writability check for GPT images

util: make creation time xattr logic more generic

sd-bus: rename sd_bus_open_system_container() to sd_bus_open_system_machine()

Pretty much everywhere else we use the generic term "machine" when
referring to containers in API, so let's do though in sd-bus too. In
particular, since the concept of a "container" exists in sd-bus too, but
as part of the marshalling system.

util: fix strict aliasing violations in use of struct inotify_event v5

There is alot of cleanup that will have to happen to turn on
-fstrict-aliasing, but I think our code should be "correct" to the rule.

hwdb: add rule and first entry for PS/2 mice

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

hwdb: Microsoft Wireless Laser Mouse 8000 DPI data

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

update TODO

shell-completion: add full support for completing busctl command lines, with services, objects, interfaces, members, and signatures

busctl: when introspecting objects, optionally limit output by interface name

gitignore: hide test-lldp files

machined: introduce polkit for OpenLogin() call

This way "machinectl login" can be opened up to run without privileges.