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

Fix appending .erb to template filename. #83

Merged
merged 1 commit into from
Jun 27, 2014
Merged

Conversation

mcquin
Copy link
Contributor

@mcquin mcquin commented Jun 27, 2014

Blocking bug for Learn Chef tutorials -- and anyone using chef generate template, heh.

Not sure why this issue wasn't turning up in spec tests. My best guess is that mixlib/cli's option parsing returns frozen strings and our stubs in spec/unit/command/generator_commands_spec.rb don't touch the option parsing code. Perhaps we should add functional/integration tests to chef-dk. In the meantime, inserted a comment to (hopefully) prevent regression of this particular case.

Checked other file generators and none try to directly modify new_file_basename.

\cc @opscode/client-eng

@btm
Copy link
Contributor

btm commented Jun 27, 2014

👍

mcquin added a commit that referenced this pull request Jun 27, 2014
Fix appending .erb to template filename.
@mcquin mcquin merged commit 9e7ca40 into master Jun 27, 2014
ksubrama pushed a commit that referenced this pull request Jan 11, 2016
Find the path of the chef gem correctly while installing chef-client service via MSI.
@charlesjohnson charlesjohnson deleted the mcquin/template-generator branch August 8, 2016 18:29
@chef-boneyard chef-boneyard locked and limited conversation to collaborators Feb 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants