whitespace observed during a read-through

Replace Id: tags with OpenBSD: tags
OK deraadt@

If pledge() or unveil() fails error out with ERR_IPC. Also error out in
the receiver when intial setup fails because of filesystem errors (unable
to open or create the base directory) or on memory fa

If pledge() or unveil() fails error out with ERR_IPC. Also error out in
the receiver when intial setup fails because of filesystem errors (unable
to open or create the base directory) or on memory failures.
OK benno@

show more ...

remove sess argument from all functions that only used it for logging
functions.
ok deraadt@

remove sess argument from log functions. ok deraadt@

indents and reflows around ERR*() and LOG*(); ok benno

repair exit codes, slightly broken after last commit (please do not go
to an enum, that's just alien). Documents the return values in each
function and unifies the protocol incompatibility error mes

repair exit codes, slightly broken after last commit (please do not go
to an enum, that's just alien). Documents the return values in each
function and unifies the protocol incompatibility error message.
from kristaps

show more ...

Simplify startup code, various phases are heading straight towards exit
so they don't need to be constructed to return errors.
ok benno florian

revert previous

sync with kristaps up to Sun Feb 17 2019

339cf5998c0c022623cd68de50722b6c14543952 Push "error trail" further into code.

baf58ce5fe1bc6ce431b3b0ac8264b83ae8c7d02 Document all arguments. Add
commo

sync with kristaps up to Sun Feb 17 2019

339cf5998c0c022623cd68de50722b6c14543952 Push "error trail" further into code.

baf58ce5fe1bc6ce431b3b0ac8264b83ae8c7d02 Document all arguments. Add
common -av usage. Remove bits about not supporting anything but
files/dirs.

821a811a8c80e52fb56b241fc65a16cae1b4fb2c Disambiguate as prodded by deraadt@

6c4475b8f226e9031ec0ec1b3f14f7d347132c87 Add -h to usage string

4d344ae6156873b44c95de0c1ed629e637c2d7ab Clarify error message
language, use service name instead of port, specify that the socket is
SOCK_STREAM. From deraadt@. Tweaked for lowercase messages.

f3ec049e76257fc96bcdc872f1d3b967b98f3eb6 In consideration to benno@'s
comments, let the mktemp functions propogate an errno handled by the
caller. Also keep the original line lengths. While in mktemp.c, make
some defines into an enum.

e116c2bd00e634b56e4276120135915ceaa31cf2 Put the FSM of the sender
into its own function. Put dry_run ack and end of phase ack into the
send buffer too, further reducing the possibility of deadlock.

c7745aa4c7394ca89d841f8ee76782256d694340 Make the sender write loop be
fully non-blocking. This frees us of deadlocking the protocol because
the sender will always be able to pull down data.

93c7b4843e80aeac2ec6ae6ffc395df4deaf4a31 Remove "yoda" notation to be
more in tune with OpenBSD. Most found by deraadt@.

show more ...

cleanup protocl warning, and return exit code upwards.

swap comparison arguments as needed, to match idiomatic C most of us
are familiar with. Proofed by florian and benno

cleanup weird spaces around !. (We normalize source-code to a standard
idiom because it is less error prone for a larger team. kristaps idiom
is highly divergent)
ok benno

fix whitespace

Import Kristaps' openrsync into the tree.
OK deraadt@