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

tig: add zsh completion installation for head version #43421

Closed
wants to merge 1 commit into from

Conversation

wkentaro
Copy link
Contributor

See also jonas/tig#433

@wkentaro wkentaro force-pushed the tig-zsh-completion branch 3 times, most recently from 635fb25 to 6adf53c Compare August 30, 2015 19:52
@dunn
Copy link
Contributor

dunn commented Aug 31, 2015

Does the completion rely on any HEAD-only functionality or can it be back-ported to stable?

@wkentaro
Copy link
Contributor Author

the completion file is there on HEAD-only. Not yet released.

@dunn
Copy link
Contributor

dunn commented Aug 31, 2015

We can take the commit and use it as a patch in a stable block:

  stable do
    url "http://jonas.nitro.dk/tig/releases/tig-2.1.1.tar.gz"
    sha256 "50c5179fd564b829b6b2cec087e66f10cf8799601de19350df0772ae77e4852f"

    # Merged in HEAD; remove in next stable release
    patch do
      url "https://github.com/jonas/tig/commit/91912eb97da4f6907015dab41ef9bba315730854.diff"
      sha256 "263e86b25f788eb158bdd667e112bc839debe9e3fe788cbc39cc9654b65b6c8a"
    end
  end

@wkentaro
Copy link
Contributor Author

instead of

head "https://github.com/jonas/tig.git"

?

@dunn
Copy link
Contributor

dunn commented Aug 31, 2015

Oh no, in addition. That way head and stable builds both get zsh completion.

@wkentaro
Copy link
Contributor Author

I see thanks. I added.

@dunn
Copy link
Contributor

dunn commented Aug 31, 2015

Thanks so much! Merged in 472a7f0.

@dunn dunn closed this in 472a7f0 Aug 31, 2015
ybogdanov pushed a commit to ybogdanov/homebrew that referenced this pull request Sep 1, 2015
See also jonas/tig#433

Closes Homebrew#43421.

Signed-off-by: Alex Dunn <dunn.alex@gmail.com>
@wkentaro wkentaro deleted the tig-zsh-completion branch November 29, 2015 13:43
@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.

None yet

2 participants