Symlinks inside password-store make sense when using passff in firefox, and
have the same LDAP password in different websites.

In order to have passff working correctly then, we need to strip the .gpg
extension from symlinks as well. Example output of `pass find site1` or `pass
| grep site1` before this commit:

    site1.job.com.gpg -> site2.job.com

and after:

    site1.job.com -> site2.job.com

xclip will return a non-zero exit code if $DISPLAY is not set, which
might happen, for example, if you're running pass in a tmux session
remotely or something like that. xclip itself will also show an error
message ("Can't open display") which provides enough context to figure
out what the problem is.

There might be other reasons for xclip to return a non-zero exit code,
but the man page doesn't list them.

Quote shell arguments for insert so that it handles passwords and entry
names that contain special characters.

The latest gpg-agent from the 2.1 series no longer requires these
environment variables and will instead start gpg-agent as it needs.

Signed-off-by: David Adam <zanchey@ucc.gu.uwa.edu.au>

Some users may use MacPorts instead of Homebrew.  If brew fails to
return a prefix for getopt, check if the user has 'port' installed and
use the MacPort ${prefix} /opt/local.

Avoids an issue where pass terminates because getopt cannot be resolved
in the assumed location /usr/local/bin.  Provide support to resolve
absolute path of getopt from a MacPorts installation.

Signed-off-by: Jamie Couture <jamie.couture@gmail.com>

When autocompleting from `pass <TAB>', sometimes the following errors
appear:

  _values:compvalues:10: not enough arguments
  find: `/home/user/.password-store': No such file or directory
  _values:compvalues:10: not enough arguments
  find: `/home/user/.password-store': No such file or directory

The `_values' error happens when there is no password-store folder *or*
there are no passwords in pass; the `find' error only when there is no
password-store folder.

We can trace it back to line 108, which contains the only `_values'
statement that is executed when we autocomplete from pass. We confirm
this by following the trail of execution, which is

  _pass -> _pass_cmd_show -> _pass_complete_entries ->
        -> _pass_complete_entries_helper

If we try running the command inside `$()' on line 104, we see that it
returns nothing and the output is blank. This means that `_values' only
receives 1 of its 2 mandatory parameters, therefore the above error is
triggered (not enough arguments).

That is unless we don't have a password-store folder, in which case the
`find: [...] no such file or directory' error is *also* triggered.

We solve the first error by supplying a default value of "" if the
command outputs nothing, using the zsh construct ${var:-else}.

We solve the second error by redirecting the find command's stderr output
to /dev/null, so the error is effectively suppressed.

* * * *

This patch also fixes the first tab completion, which currently only
loads the completion function definition.

We do this by adding a `_pass' statement at the end of the file, which
runs the `_pass' completion function after loading its definition.
This is the standard way an autoloaded function works; for other examples
look at zsh's official completion files.

Suggested-by: Christoph Egger <christoph@christoph-egger.org>

In the move away from extensive global variables and improved cleanup
routines, we forgot to fix the darwin platform file, which means
temporary ramdisks never got unmounted. This patch cleans up the general
cleanup trap logic routines.

This is required by most graphical editors, as the shell command they
provide needs to be told to wait until the file has been closed.

This reverts '311c98c9'. We thus use
PATH so that the tests still pass in directories with spaces.

If keepassx2pass.py is given an xml file containing passwords with an empty
title, like <title></title>, ElementTree.text returns None. This commit
substitutes an empty string; which will produce a password with name '_',
instead of raising AtttributeError, as shown in this exception::

    Traceback (most recent call last):
      File "contrib/importers/keepassx2pass.py", line 80, in <module>
        main(sys.argv[1])
      File "contrib/importers/keepassx2pass.py", line 77, in main
        import_group(group)
      File "contrib/importers/keepassx2pass.py", line 71, in import_group
        import_entry(entry, npath)
      File "contrib/importers/keepassx2pass.py", line 58, in import_entry
        print "Importing " + path_for(element, path)
      File "contrib/importers/keepassx2pass.py", line 37, in path_for
        title = cleanTitle(space_to_camelcase(element.find('title').text))
      File "contrib/importers/keepassx2pass.py", line 16, in space_to_camelcase
        for word in value.split(" "):
    AttributeError: 'NoneType' object has no attribute 'split'

Timeout password after PASSWORD_STORE_CLIP_TIME seconds, if set.
Otherwise timeout after 45 seconds. These are the setting used by pass.

Key discovery parses stderr output of gnupg. The secmem warning
interferes with parsing.