Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

ssh-copy-id: Added HEAD configuration #49952

Closed
wants to merge 1 commit into from

Conversation

kevinastone
Copy link
Contributor

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Formulae Submissions:

(not a new formula, but...)

  • Does your submission pass
    brew audit --strict --online <formula> (where <formula> is the name of the formula you're submitting)?
  • Have you built your formula locally prior to submission with brew install <formula>?

The HEAD will checkout openssh-portable and copy the contrib script just
like the binary.

HEAD currently has fixes for non-posix shells (like fish that I use)

@@ -6,6 +6,10 @@ class SshCopyId < Formula
version "7.1p1"
sha256 "fc0a6d2d1d063d5c66dffd952493d0cda256cad204f681de0f84ef85b2ad8428"

head do
Copy link
Member

Choose a reason for hiding this comment

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

This can just be head "https://github.com/openssh/openssh-portable.git"

The HEAD will checkout openssh-portable and copy the contrib script just
like the binary.

HEAD currently has fixes for non-posix shells (like fish that I use)
@DomT4
Copy link
Member

DomT4 commented Mar 10, 2016

Merged in 74a39f8. Thank you for your contribution to Homebrew @kevinastone; we appreciate it! 😺

@DomT4 DomT4 closed this in 74a39f8 Mar 10, 2016
xu-cheng pushed a commit to Homebrew/homebrew-core that referenced this pull request Mar 11, 2016
The HEAD will checkout openssh-portable and copy the contrib script just
like the binary.

HEAD currently has fixes for non-posix shells (like fish that I use)

Closes Homebrew/legacy-homebrew#49952.

Signed-off-by: Dominyk Tiller <dominyktiller@gmail.com>
@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants