Skip to content

Handle simpler and invalid values for config overrides#177

Closed
jgawor wants to merge 1 commit intocloudfoundry:masterfrom
jgawor:config_override_input
Closed

Handle simpler and invalid values for config overrides#177
jgawor wants to merge 1 commit intocloudfoundry:masterfrom
jgawor:config_override_input

Conversation

@jgawor
Copy link
Copy Markdown
Contributor

@jgawor jgawor commented Apr 21, 2015

Two key changes:

  1. Handle simpler overrides (no need for array for single overrides). For example instead of JBP_CONFIG_OPEN_JDK_JRE '[version: 1.7.0_+]' handle JBP_CONFIG_OPEN_JDK_JRE 'version: 1.7.0_+'
  2. Handle invalid values better. For example, for invalid input such as JBP_CONFIG_OPEN_JDK_JRE 'version 1.7.0_+' generate a nicer error message instead of Ruby errors (e.g. NoMethodError).

@cfdreddbot
Copy link
Copy Markdown

Hey jgawor!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you've already signed the CLA.

nebhale added a commit that referenced this pull request Apr 21, 2015
@nebhale nebhale closed this Apr 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants