configure: allow setting EFI_CC

efi: use EFI_CC

factory: install to datadir

udev: restore udevadm settle timeout

Commit 9ea28c55a2 (udev: remove seqnum API and all assumptions about
seqnums) introduced a regresion, ignoring the timeout option when
waiting until the event queue is empty.

Previously, if the udev event queue was not empty when the timeout was
expired, udevadm settle was returning with exit code 1.  To check if the
queue is empty, you could invoke udevadm settle with timeout=0. This
patch restores the previous behavior.

(David: fixed timeout==0 handling and dropped redundant assignment)

rules: fix tests for removable state

We only care about whether our direct parent is removable, not whether any
further points up the tree are - the kernel will take care of policy for
those itself. This enables autosuspend on devices where the root hub reports
that its removable state is unknown.

README: glibc version 2.16 is required for IP_UNICAST_IF

https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=be08eda5
https://bugs.gentoo.org/show_bug.cgi?id=546194

TODO: add 13790add4 as blocker for v220

build: add support for AARCH64 EFI

Aarch64 and ARM32 lack an EFI capable objcopy, so use the ldflags + -O
binary trick gnu-efi and the Red Hat shimloader are using.

(David: rebase to systemd-git and added EFI_ prefixes)

boot/util: add ticks_read() stub

build: add AARCH64 efi support

This is just plumbing to add ARCH_AARCH64 EFI support for makefile tests
and defining the machine name.

boot/util: use x86 ASM only on x86 platforms.

build: support non-x86 EFI builds

Move the no-mmx/no-sse CFLAGS to X86-64 and IA32 defines in preparation
for ARM32 and Aarch64 support.

hwdb: set the resolution for a couple of bcm5974 touchpads

Verified for the 5,1 Macbook, the others are guesses based on the list of
supported devices of the moshi trackpad protector.
http://www.moshi.com/trackpad-protector-trackguard-macbook-pro#silver

Resolution calculated based on the min/max settings set in the kernel driver,
divided by the physical size. This is probably slightly off, but still better
than no resolution at all.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

udev: builtin-keyboard: add support for EVDEV_ABS_*

Parse properties in the form
EVDEV_ABS_00="<min>:<max>:<res>:<fuzz>:<flat>"

and apply them to the kernel device. Future processes that open that device
will see the updated EV_ABS range.

This is particularly useful for touchpads that don't provide a resolution in
the kernel driver but can be fixed up through hwdb entries (e.g. bcm5974).

All values in the property are optional, e.g. a string of "::45" is valid to
set the resolution to 45.

The order intentionally orders resolution before fuzz and flat despite it
being the last element in the absinfo struct. The use-case for setting
fuzz/flat is almost non-existent, resolution is probably the most common case
we'll need.

To avoid multiple hwdb invocations for the same device, replace the
hwdb "keyboard:" prefix with "evdev:" and drop the separate 60-keyboard.rules
file. The new 60-evdev.rules is called for all event nodes
anyway, we don't need a separate rules file and second callout to the hwdb
builtin.

udev: builtin-keyboard: invert a condition

No functional changes, just to make the next patch easier to review

udev: builtin-keyboard: move actual key mapping to a helper function

No changes in the mapping, but previously we opened the device only on
successful parsing. Now we open the mapping as soon as we have a value that
looks interesting. Since errors are supposed to be the exception, not the
rule, this is probably fine.

udev: builtin-keyboard: immediately EVIOCSKEYCODE when we have a pair

Rather than building a map and looping through the map, immediately call the
ioctl when we have a successfully parsed property.

This has a side-effect: before the maximum number of ioctls was limited to the
size of the map (1024), now it is unlimited.

udev: builtin-keyboard: move fetching the device node up

No point parsing the properties if we can't get the devnode to apply them
later. Plus, this makes future additions easier to slot in.

shared: move replace_env* from util to env-util

shared: add terminal-util.[ch]

shared: add random-util.[ch]

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.