docker-credential-helpers: pass variant

[glensc]

so, docker/docker-credential-helpers has also pass variant.

how to package this? to existing formula?

1. https://github.com/Homebrew/homebrew-core/blob/9b37cb405386ed11af4cb68130d56ceb5335d413/Formula/docker-credential-helper.rb
2. create new formula: docker-credential-helper-pass?

as would be accurate to also install the pass dependency that pass requires...

[glensc]

i think separate formula is way to go, because also need to link and overwrite if installing docker-for-mac. and without link doesn't get any program from the formula, not just conflicting ones

...
Error: The `brew link` step did not complete successfully
The formula built, but is not symlinked into /usr/local
Could not symlink bin/docker-credential-osxkeychain
Target /usr/local/bin/docker-credential-osxkeychain
already exists. You may want to remove it:
  rm '/usr/local/bin/docker-credential-osxkeychain'

To force the link and overwrite all conflicting files:
  brew link --overwrite docker-credential-helper

To list all files that would be deleted:
  brew link --overwrite --dry-run docker-credential-helper

Possible conflicting files are:
/usr/local/bin/docker-credential-osxkeychain -> /Applications/Docker.app/Contents/Resources/bin/docker-credential-osxkeychain
==> Summary

[zbeekman]

@glensc: Can docker-crednetial-helper be built against both osx-keychain and pass at the same time? Or does it require a different formula?

Also, I saw your issue on their repo that they do not currently support using pass on macOS. If they don't support using pass on macOS, then I'm going to close this as "won't fix" and we can revisit it once docker-credential-helpers supports pass on macOS.

[glensc]

@zbeekman yes, it can be built two variants from same formula. and to make build need to copy (or rename) two files:

• pass variant macos support docker/docker-credential-helpers#146 (comment)

however, I think it should be separate formula due to usability issues:

• docker-credential-helpers: pass variant #39517 (comment)