Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add brew cask cat command #4200

Merged
merged 1 commit into from May 7, 2014
Merged

Add brew cask cat command #4200

merged 1 commit into from May 7, 2014

Conversation

linc01n
Copy link
Contributor

@linc01n linc01n commented May 5, 2014

Port brew cat to homebrew-cask.

Mentioned in #4018

@linc01n linc01n changed the title Add brew cask cat command mentioned in #4018 Add brew cask cat command May 5, 2014
@rolandwalker
Copy link
Contributor

Thanks @linc01n !

rolandwalker added a commit that referenced this pull request May 7, 2014
Add `brew cask cat` command
@rolandwalker rolandwalker merged commit e1ee7b1 into Homebrew:master May 7, 2014
@rolandwalker
Copy link
Contributor

I don't agree that multiple Casks is unparseable! It seems that comment came in from Homebrew.

I'm going to tweak it a little bit so that it exits with an error when multiple Casks are given.

@linc01n linc01n deleted the cli_cat branch May 7, 2014 12:35
@linc01n
Copy link
Contributor Author

linc01n commented May 7, 2014

I copy right from homebrew. Just tweak the exec "cat" to File.open to pass the test. I think one cask a time is a simpler solution. eg. the first cast is correct and second cask spell wrong/not found.

For now it takes the first cask and drop the remaining.

@Homebrew Homebrew locked and limited conversation to collaborators May 8, 2018
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