fix: update images that had missing arrows on them

docs: move media into media/ subdirectory

Seeing as selective rendering doesn't work on another branch. Seems host
dependent: relative paths like this work, and GitHub has this example
working:

    https://github.blog/changelog/2021-11-24-specify-theme-context-for-images-in-markdown/

using this host:

    https://user-images.githubusercontent.com/

I can't say I am thrilled about the `gh-` prefix on this thing, because
it means the README is going to look good on GitHub, but not on other
mirrors, like GitLab, unless they decide to also support this particular
"vendor prefix".

docs: fix dark/light images

This:

    https://github.blog/changelog/2021-11-24-specify-theme-context-for-images-in-markdown/

shows an example working on:

    https://user-images.githubusercontent.com/

Not sure if there is some host-specific check going on here, so trying a
local path instead.

docs: add dark-mode variant of example topology diagram

chore: add media to .gitignore

Ignores worktree created with:

    git worktree add media media

chore: update .mailmap

According to `git shortlog -s -e`, some discrepant email addresses had
crept in.

docs: improve formatting of email address link in Markdown

docs: add info on OSC-52

docs: fix missing whitespace in README.md

Closes: https://github.com/wincent/clipper/issues/25

docs: add diagram showing example

Makes it a little easier to explain what this thing does.

chore: move from "master" to "main"

docs: add another reference to vim-clipper

docs: add tip to use `-N` switch with `nc` if required

As suggested here:

https://github.com/wincent/vim-clipper/pull/3#issuecomment-646926371

Merge branch 'pull/21'

Closes: https://github.com/wincent/clipper/pull/21
* pull/21:
  docs: update "Authors" and "History" sections
  refactor: make log file directory on all platforms
  feat: automatically create the default log directory if required

docs: update "Authors" and "History" sections

refactor: make log file directory on all platforms

I don't see any reason why this should be limited to Linux, so remove
the platform check.

Additionally, check the error condition and blow up if we can't make the
directory. Finally, remove "config" from the variable name because it
may not be a `~/.config` directory at all.

And finally, avoid calling `expandPath` twice.

feat: automatically create the default log directory if required

style: format Markdown with Prettier

With: `npx prettier --write README.md`

docs: fix linebreaks in "Authors" section of README

Document use of socat

As noted in the related issue, this can be useful (or necessary) on
systems without `nc` or with version of `nc` (GNU, for example) that
doesn't support the `-U` switch.

Closes: https://github.com/wincent/clipper/issues/19

Prepare for v2.0.0 release

Bumping the major version because there *may* be somebody out there
relying on the old behavior (ie. they have an empty `--flags` value and
are getting the default).

Overhaul flag/config handling to deal with empty values

We were previously conflating non-emptiness of flags and config values
with being-set. This meant that you couldn't explicitly set a value to
an empty string, because we would interpret that as the value not being
set and we would fall back to the default. This is a problem if you
want, for example, to suppress all `--flags` being sent to the
underlying `--executable`.

Fix this by replacing our simple string and int variables with struct
wrappers that contain both the value and an explicit boolean that
indicates whether the value was provided. This requires us to implement
the `flag.Value` interface on our wrapper structs, and also the
`json.Unmarsheler` interface so that we can unpack from the JSON config
file into these custom structs.

In addition to fixing the issue, these changes make the code in the
`mergeSettings()` function look simpler and more fool-proof now. Only
downside is we need a bit of boilerplate-ish code to implement those
interfaces. Error messages when providing malformed JSON are about as
good as they used to be:

    clipper: 2018/10/04 13:15:31 json: cannot unmarshal object into Go
    struct field Options.Address of type string

Closes: https://github.com/wincent/clipper/issues/15

Prepare for 1.0 release

Automatically clean up stale socket files at startup

These can get left behind after a crash, for example.

doc: make it clear that StreamLocalBindUnlink should be set on the server

I've never found it to work on the client, but it does work on the
server. It wasn't clear to me from the docs that this would be the case
because the wording is almost identical for both `sshd_config`:

  StreamLocalBindUnlink
          Specifies whether to remove an existing Unix-domain
          socket file for local or remote port for- warding before
          creating a new one.  If the socket file already exists and
          StreamLocalBindUnlink is not enabled, sshd will be unable to
          forward the port to the Unix- domain socket file.  This option
          is only used for port forwarding to a Unix-domain socket file.

          The argument must be yes or no.  The default is no.

and `ssh_config`:

  StreamLocalBindUnlink
          Specifies whether to remove an existing Unix-domain
          socket file for local or remote port for- warding before
          creating a new one.  If the socket file already exists and
          StreamLocalBindUnlink is not enabled, ssh will be unable to
          forward the port to the Unix- domain socket file.  This option
          is only used for port forwarding to a Unix-domain socket file.

          The argument must be yes or no (the default).

I'm thinking that perhaps the client side config only applies to
port-forwarding (or the socket on the local side), and the server config
applies to reverse port forwarding (or the the socket on the remote
side).

Merge branch 'pull/14'

Closes: https://github.com/wincent/clipper/pull/14
* pull/14:
  doc: update AUTHORS section
  fix readme typo

doc: update AUTHORS section

fix readme typo

Add note about use of 'TextYankPost' in vim-clipper

doc: Note changes in tmux post-v2.5

Currently only on the master branch, but will be in 2.6:

https://github.com/tmux/tmux/commit/78352fdd328b611d93

Merge branch 'pull/12'

Closes: https://github.com/wincent/clipper/pull/12
* pull/12:
  doc: Update "Authors" section
  Library/Logs instead of Library/Log in README

doc: Update "Authors" section

Library/Logs instead of Library/Log in README

doc: Add troubleshooting tip about `nc -N`

Closes: https://github.com/wincent/clipper/issues/11

doc: apply updates for compatibility with tmux 2.4

Closes: https://github.com/wincent/clipper/issues/10

Fix variable expansion in Makefile tag target

Update docs for 0.4.2 release

Don't do pass by reference

According to this:

    http://stackoverflow.com/a/39993797/2103996

Slices are passed by value, but the value is just a pointer to the
underlying array, which does not change, so the listeners are not
copied.

Add a filter function to keep main body cleaner

So I've now done this three ways to see which one sucks less. The lack
of generics makes this one feel a bit ridiculous, but at least the main
body reads nicely now.

Note also the pass-by-reference shennanigans to avoid making a copy of
the listener objects (I don't even know what that would mean, although I
want to try it out and find out: maybe Go only makes a copy of the
slice, not the slice items, when you pass a slice by value).

Implement separate loop for determining `listenerCount`

Still ugly, because we still need the `nil` check in the second loop.

Bail if we failed to set up any listeners

Normally we only try to set up one listener.

But in the case where we try to listen on the IPv4 and IPv6 loopback
addresses, we're fine with up to one of those failing, but not both.
This `listenerCount` variable is ugly, but I don't know Go idioms. I
think I might just pull it out into a separate loop for clarity (in a
language with a generic `filter` function I would just use that to strip
out `nil` and then look at the `len`).

Reduce duplication of listener set-up

Note if somebody passes a crazy port-number (like a negative one)
`net.Listen` will blow up for us, but it would have done that before
anyway.

Fold two loops together

Remove unnecessary anonymous goroutine

Don't listen on all interfaces

This is the dumb fix for the face-palm bug reported here:

  https://github.com/wincent/clipper/issues/9

wherein our attempt to listening on IPv4 and IPv6 loopback interfaces
when available was actually causing us to listen on all interfaces
(ugh).

I'll remove the duplication in the next commit, and also try to
gracefully handle systems which might not have IPv6 interfaces up.

doc: Fix typo in README

Create socket with tighter permissions

doc: Note that Apple has reverted to pre-Yosemite behavior in Sierra

doc: trim down description in README

"or as a process" was grammatically invalid; in fixing it, decided to
make the whole sentence more concise.

contrib: Drop --address and --port from sample config

Seeing as these are the defaults, and the options are voluminously
documented elsewhere anyway, may as well drop them from the config.

Merge branch 'pull/4'

Closes: https://github.com/wincent/clipper/pull/4
* pull/4:
  doc: Engage in word-smithery
  doc: Fix Linux set-up instructions
  Move Linux contrib files under contrib/linux
  update linux service script
  add systemd script support for linux

doc: Engage in word-smithery

- Add some explicit headings.
- Slight reorder of some things.
- Minor edits for flow.

doc: Fix Linux set-up instructions

- Typo: "cliper" -> "clipper".
- Fix paths: service expects clipper to be in `/usr/local/bin`, not
  `~/bin`.
- Note likely need to use `sudo`.
- Kill off unnecessary blank lines.

Move Linux contrib files under contrib/linux

Mirroring what we did for Darwin in 40b0f3a.

update linux service script

add systemd script support for linux

Move Darwin contrib files into contrib/darwin

Because we already have a pending PR (#4) for Linux, we'll want to
organize these by platform.

Add .mailmap file

Update usage sample in docs

Fix variable references in Makefile

Prepare for 0.4 release

Add Git-powered version number at build time

As demonstrated here:

- https://gist.github.com/TheHippo/7e4d9ec4b7ed4c0d7a39839e6800cc16
- https://github.com/Masterminds/glide/blob/master/Makefile

Drop "shorthand" from usage

Mostly so I can add "default" information to the `--address`/`-a`
option, even though the actual default is an empty string and so usually
wouldn't show.

Listen on IPv4 and IPv6 by default

As noted [here](https://github.com/golang/go/issues/9334#issuecomment-67098831), the "correct way" to listen on both IPv4 and IPv6 loopback interfaces is to omit the address part.

Closes: https://github.com/wincent/clipper/issues/2
And will also close: https://github.com/wincent/vim-clipper/issues/1

Fix broken HTML entity in README

Merge branch 'pull/3'

Closes: https://github.com/wincent/clipper/pull/3
* pull/3:
  Rename --args/-r to --flags/-f
  Add --executable and --args flags
  Rename ClipApp{,Args} to Executable,Args
  doc: update for OS X -> macOS, and Linux support
  Treat ClipAppArgs independently of ClipApp
  Handle variable whitespace in ClipAppArgs
  Reformat some comments for consistency
  Kill an excess empty line in the Makefile
  doc: Update "Authors" and "History" sections
  add support for linux clipboard

Rename --args/-r to --flags/-f

I don't want to reimplement `flag.PrintDefaults` in order to get
appropriately grouped usage information, so I am renaming this flag to
so that the long and short forms both begin with the same letter.

"Flag" is unfortunately pretty overloaded at this point, and we already
have a bit of a cluster going on with "Settings", "Options", "Defaults",
and "Flags", but this is the way things are for now: because of the way
JSON marshalling works, we have to have names which match the desired
options.

Add --executable and --args flags

Alas, `--args` must break with the pattern of having the short option
start with the same letter as the long option (ie. `-a`); instead it
uses `-r`. This makes the `--help`/`-h` output a bit "meh"; not sure if
there's a way to fix that. If I can't find one I might need to rename
this option to `--flags`/`-f` or `--options`/`-o`.

Rename ClipApp{,Args} to Executable,Args

Going to expose these as command-line options, so want to get names
right.

doc: update for OS X -> macOS, and Linux support

Not sure whether `clipApp` and `clipAppArgs` should be exposed on the
command-line as well, for consistency. Will think about it.

Treat ClipAppArgs independently of ClipApp

You might conceivably want to set one without setting the other. For
example, on OS X, you might want to keep `pbcopy`, but instead use
`-pboard find` (for some crazy reason).

This isn't exposed as a command-line flag right now, so the only way you
could do such a thing would be to provide a custom JSON config file.

Handle variable whitespace in ClipAppArgs

- Trim leading/trailing whitespace
- Split on runs of consecutive whitespace, so that "x     y" will split
  the same as "x y".

Reformat some comments for consistency

Kill an excess empty line in the Makefile

doc: Update "Authors" and "History" sections

add support for linux clipboard

Prepare for 0.3 release

Fix documentation issues discovered through proof-reading

Update docs with domain socket info

Haven't proof-read this yet, so it is probably a bit rough.

Add real -h and --help flags

You would get help output before this if you typed "-h" or "--help" (or "-help")
but only because you were seeing the failure output. In other words,
"--random-crap" would show you exactly the same output.

Let's use real flags here instead, and exit cleanly with a 0 exit code.

Split plist files into two

Homebrew has it's own custom fork:

  https://github.com/Homebrew/homebrew-core/blob/409a67fbf587e4346235cb7ee71eab0d65adc88f/Formula/clipper.rb

So I am not worried about breaking that, but I do want to show a couple of clear
examples of how to set this up via TCP ports or UNIX domain sockets.

Note that I also updated the paths to use `/usr/local/bin` instead of `/usr/bin`
or a relative PATH because of Apple's SIP (System Integrity Protection) that
makes it much less likely that people will be able to write to the latter.

Complain if given --port with UNIX domain socket --address

Rejig settings handling to get desired precedence

Flags beat config file which beats defaults.

This is very verbose, but this is the Go way, I think.

Make option overrides a little more predictable

Switching to structs instead of independent variables to make it more obvious
what's going on with overrides. Note the motivating use case here, which is
making sure that only things which look like paths end up getting converted to
absolute paths and used.

Update links in README

Promote GitHub project page to "official" status, simplify `go build`
invocatioe, mention other mirrors, and drop references to binary builds, which I
am unlikely to offer moving forward, given the prevalence of Homebrew in the
target market for which those executables existed (OS X power users).

Optionally listen on a UNIX domain socket

If `--address` (or "address" in the config file) looks like a path (ie. starts
with "/" or "~") then Clipper will listen on a UNIX domain socket at that path
rather than on a port.

Motivation: better security, because the socket can be protected via filesystem
permissions, whereas the TCP port is accessible to any user that has access to
localhost.

Netcat can be configured to talk to a UNIX socket (via `-U`), and it's possible
to forward local sockets using SSH as well, given a sufficiently recent version
of SSH, so everything can continue to work as before.

I'm going to run with this instead of my half-made passphrase-based solution,
for now, because the latter has a bunch of issues (need to exchange a shared
secret at start-time, and updating it may be tricky in the presence of
long-lived tmux sessions etc).

This is still a WIP, however, as we have some issues to work through. For
example, command-line options should trump config file options (we currently do
the opposite) and it all needs to be documented still. Also, the code is
frickin' ugly (it's been so long since I programmed in Go, I had to Google "how
do I X?" about 20 times while preparing this tiny change).

Catch signals in order to perform exit cleanup

Want to do this because I'm about to add support for listening on UNIX domain
sockets, and those need to be cleaned up at exit in order to avoid havoc.

Based on:

  http://stackoverflow.com/a/25242964/2103996

Future-proof copyright year ranges

doc: Mention changes to execution context in Yosemite

Note that the stuff about execution context only applies pre-Yosemite.
Clipper is still useful, however, even on Yosemite, because it provides
you with a uniform interface to the clipboard for both local and remote
hosts.

doc: Fix Markdown fail

doc: Mention the vim-clipper plug-in

Remove Donations section from README

Update copyright headers and author information

Crop logo

Add logo to README

Add "Reverse" Clipper section to README

doc: Remove references to Gitorious

Soon to be defunct:

  https://about.gitlab.com/2015/03/03/gitlab-acquires-gitorious/

Add section about Mosh to the docs

This commit adds some tips on getting Mosh, SSH and Clipper playing
nicely together. I include the section about the various `Control*`
settings because they are a complicating variable which needs to be
dealt with.

(In general, the `Control*` settings are a great time saver, but they
can make trouble-shooting difficult and we can simplify the business of
setting up a "pure-forwarder" for use with Mosh by keeping them out of
the picture.)

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add a tip for the bold and lazy to the docs

A slightly bolder and more lazy tip that I'm not including is that you
can just do:

  killall sshd

Unless you are foolhardily running as root, in which case that will
lock you out of your server. Let's not encourage people to be too
heavy-handed.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Fix oversize heading

This heading was a little horrid to look at with the non-wrapping
pre-formatted text. Make it smaller; hopefully it will wrap.

Signed-off-by: Wincent Colaiuta <win@wincent.com>