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

DSL: add stanza stage_only (new spelling of caskroom_only) #7365

Merged
merged 1 commit into from
Nov 18, 2014

Conversation

rolandwalker
Copy link
Contributor

caskroom_only was never documented. Its original purpose was obsoleted in #4865, and its use has been recently been reduced to two Casks.

This PR

  • continues the rationalization of naming by changing caskroom_only to stage_only. "stage" is the verb for "make a copy under the caskroom directory"
  • documents stage_only
  • adds tests for stage_only
  • validates the argument to stage_only
  • gives sensible output in brew cask info for stage_only Casks
  • enforces that stage_only cannot coexist with any activatable artifacts

caskroom_only is still supported for backward compatibility, but should be removed before 0.50.0.

@rolandwalker rolandwalker added the core Issue with Homebrew itself rather than with a specific cask. label Nov 15, 2014
`caskroom_only` was never documented.  Its original purpose was
obsoleted in Homebrew#4865, and its use has been recently been reduced to
two Casks.

This PR
* continues the rationalization of naming by changing `caskroom_only`
  to `stage_only`. "stage" is the verb for "make a copy under the
  caskroom directory"
* documents `stage_only`
* adds tests for `stage_only`
* validates the argument to `stage_only`
* gives sensible output in `brew cask info` for `stage_only` Casks
* enforces that `stage_only` cannot coexist with any activatable
  artifacts

`caskroom_only` is still supported for backward compatibility,
but should be removed before 0.50.0.
@tapeinosyne
Copy link
Contributor

Looks good.

tapeinosyne pushed a commit that referenced this pull request Nov 18, 2014
DSL: add stanza `stage_only` (new spelling of `caskroom_only`)
@tapeinosyne tapeinosyne merged commit a21dbc8 into Homebrew:master Nov 18, 2014
@rolandwalker rolandwalker deleted the add_stage_only branch November 18, 2014 13:10
@miccal miccal removed the core Issue with Homebrew itself rather than with a specific cask. label Dec 23, 2016
@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.

3 participants