Issue #5081 has been updated by Matt Robinson. Status changed from Accepted to Available In Testing Branch Assignee deleted (Nigel Kersten)
commit:616986da3751012cf526ad75fd250abc93e6c52a ---------------------------------------- Bug #5081: Syntax checking not working with --ignoreimport https://projects.puppetlabs.com/issues/5081 Author: Dan Carley Status: Available In Testing Branch Priority: High Assignee: Category: executables Target version: 2.6.5 Affected Puppet version: 2.6.1 Keywords: iteration_2010-12-01 Branch: The use of `--ignoreimport` with `--parseonly` isn't raising errors on bad syntax. I've traced the last time this was working back to the tag of `2.6.0rc4`. This is important for those of us who use syntax checks in commit hooks, in order to prevent buggy manifests from going out. It can be observed with the following: <pre> class test { file { "/tmp/foo": content => "bar", some invalid text } } </pre> <pre> d...@dan-macpro:~/projects/puppet$ git describe 2.6.0rc4 d...@dan-macpro:~/projects/puppet$ RUBYLIB=./lib ./bin/puppet --parseonly --ignoreimport ../test.pp; echo $? err: Could not parse for environment production: Syntax error at 'invalid'; expected '}' at /home/dan/projects/test.pp:4 1 </pre> <pre> d...@dan-macpro:~/projects/puppet$ git describe 2.6.3rc1-105-gb1ef091 d...@dan-macpro:~/projects/puppet$ RUBYLIB=./lib ./bin/puppet --parseonly --ignoreimport ../test.pp d...@dan-macpro:~/projects/puppet$ echo $? 0 </pre> -- 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 post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
