Skip to content

Gnupg2 Homebrew install on macOS provides gpg command not gpg2.#14236

Closed
gartung wants to merge 4 commits intospack:developfrom
gartung:gartung-gpg-macos
Closed

Gnupg2 Homebrew install on macOS provides gpg command not gpg2.#14236
gartung wants to merge 4 commits intospack:developfrom
gartung:gartung-gpg-macos

Conversation

@gartung
Copy link
Copy Markdown
Member

@gartung gartung commented Dec 19, 2019

Using Homebrew to install gnupg2 on macOS provides symbolic link /usr/local/bin/gpg but not /usr/local/bin/gpg2. On macOS check for gpg in addition to gpg2.

This might break test where keys are created because gnupg has a limit on the length of the path to the keystore. It may require running

spack test --basetemp /tmp/spack

on macOS

@adamjstewart
Copy link
Copy Markdown
Member

You could do something like we do for Python where we search for python3.7, then python3, then python if the first two aren't found. We do something similar for our default EDITOR search.

Copy link
Copy Markdown
Member

@adamjstewart adamjstewart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the following change is the simplest way to do things.

@adamjstewart adamjstewart added buildcache macOS tests General test capability(ies) labels Jan 31, 2020
# SPDX-License-Identifier: (Apache-2.0 OR MIT)

import os
import platform
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this import can be removed to

@gartung
Copy link
Copy Markdown
Member Author

gartung commented Jan 31, 2020

After merging with develop there are no changes.

@gartung gartung closed this Jan 31, 2020
@gartung gartung deleted the gartung-gpg-macos branch January 31, 2020 20:31
@gartung gartung removed the request for review from scheibelp January 31, 2020 20:31
@gartung
Copy link
Copy Markdown
Member Author

gartung commented Jan 31, 2020

Looks like @tgamblin came up with a better implementation anyway in #14594

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buildcache macOS tests General test capability(ies)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants