Issue #1198 has been updated by Larry Fast.
Just wanted to add my +1 to this issue. I'm currently trying to track down the source of this: Error: Failed to apply catalog: Parameter require failed on File[C:/apache-flume-1.3.0-bin/]: No title provided and "File[]" is not a valid resource reference Any chance of at least raising this from Low to Medium? ---------------------------------------- Feature #1198: alter parser to throw an error on use of an undefined, unquoted, variable. https://projects.puppetlabs.com/issues/1198#change-90249 * Author: Mike Pountney * Status: Accepted * Priority: Low * Assignee: * Category: language * Target version: * Affected Puppet version: 0.25.4 * Keywords: * Branch: ---------------------------------------- Currently, the parser treats the following cases as equivalent: <pre> file {"/tmp/testfile": content => $content } file {"/tmp/testfile": content => "$content" } </pre> I think it would be beneficial to throw a compile error in the first case if $content is not defined, eg fail('Attempt to use unquoted, undefined variable $content'). This would sensibly catch many cases where I have: * typo'd on variable names, * misjudged scoping, * forgotten to define a variable that I am using in a defined type, * broken a facter fact. The second form provides a means for people to continue with the current logic of 'all variables are an empty string if undefined' for such things as 'if "$myvar"' statements. Anyway, please feel free to tell me that this ain't gonna happen, just wanted to throw it out there... -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/puppet-bugs?hl=en. For more options, visit https://groups.google.com/groups/opt_out.
